--curonly still requires mwclient

[jrbray1]

both when you omit --xmlrevisions and use --curronly it still wants mwclient

python2 ./dumpgenerator.py --api=http://wiki.johnbray.org.uk/api.php --xml --curonly
Please install the mwclient module if you want to use --xmlrevisions.
...
)
File "./dumpgenerator.py", line 391, in getPageTitles
for title in titles:
File "./dumpgenerator.py", line 261, in getPageTitlesAPI
site = mwclient.Site(apiurl.netloc, apiurl.path.replace("api.php", ""), scheme=apiurl.scheme)

If mwclient is needed in all circumstances, it needs to say so

[cooperdk]

If mwclient is needed in all circumstances, it needs to say so

No, it doesn't. Take a look at requirements.txt where it is included. If you do what you're supposed when using a Python script, you won't get the error.