Mvdb/harvest command use argparse

[m-vdb]

Hi @gabrielfalcao,

Another PR related to Django 1.9: all commands now use argparse by default. So this PR switches the command from optparse to argparse. The bad thing about that is that this break compatibility with Django 1.7. The good thing is that your command will be instantly compatible with 1.10 (almost out ;)).

LMK if you want to merge it or wait a little bit

[m-vdb]

this line gives an error on Django==1.8, so that's why I replaced it below with something smarter

[nikolas]

This clause is unnecessary - we can just use requires_system_checks with Django 1.8 as well as Django 1.9.

[m-vdb]

also, it works like a charm with Django 1.10.1 😄

[m-vdb]

@gabrielfalcao @nikolas do you think we could merge that?

[kairatomurbek2]

Hi @gabrielfalcao, manage.py harvest now working in django 10. Why does not it work ?

[kairatomurbek2]

Hi @m-vdb, manage.py harvest now working in django 10. Why does not it work ?

[m-vdb]

@kairatomurbek2 it works well for us with Django 1.10.1. open an issue on this project with sufficient debugging information if you need help.

[kairatomurbek2]

@m-vdb Yes, I need help, I use Django 1.10.2 I have this error goes here
File "/home/kayrat/projects/marketplace/virtualenv/local/lib/python2.7/site-packages/lettuce/django/management/commands/harvest.py", line 43, in Command
option_list = BaseCommand.option_list + (
AttributeError: type object 'BaseCommand' has no attribute 'option_lis
comand ./manage.py harvest acceptance_tests/features --settings=main.settings_test --debug-mode

[kairatomurbek2]

Yes I need help, I have a Django 1.10.2 I run, but for some reason does not
work. Now this error
File
"/home/kayrat/projects/marketplace/virtualenv/local/lib/python2.7/site-packages/lettuce/django/management/commands/harvest.py",
line 43, in Command
option_list = BaseCommand.option_list + (
AttributeError: type object 'BaseCommand' has no attribute 'option_list'

./manage.py harvest acceptance_tests/features --settings=main.settings_test
--debug-mode

On Fri, Oct 21, 2016 at 12:27 AM, Maxime Vdb [email protected]
wrote:

@kairatomurbek2 https://github.com/kairatomurbek2 it works well for us
with Django 1.10.1. open an issue on this project with sufficient debugging
information if you need help.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#527 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFrePgc5I8nuooF-clqFTJWqk3d2r1rAks5q17KpgaJpZM4JXsd8
.

С уважением !
Омурбек уулу Кайрат
+996(0700)13-96-80
e-mail: [email protected]
skype: lolakt1

[m-vdb]

as you can see in the code of this repo, you won't find any reference to BaseCommand.option_list here. My guess is that you don't have the latest version of lettuce.

[kairatomurbek2]

@m-vdb I have the latest version, I created a test project to test volatile, check please. https://github.com/kairatomurbek2/for_lettuce

[kairatomurbek2]

@m-vdb I'm still the same error comes