chiark / gitweb /
Merge branch 'replace_optparse_with_argparse' into 'master'
authorDaniel Martí <mvdan@mvdan.cc>
Thu, 10 Sep 2015 02:08:24 +0000 (02:08 +0000)
committerDaniel Martí <mvdan@mvdan.cc>
Thu, 10 Sep 2015 02:08:24 +0000 (02:08 +0000)
replace deprecated optparse with argparse

squashed and rebased merge request fdroid/fdroidserver!74

following guidelines from:
https://docs.python.org/2/library/argparse.html#upgrading-optparse-code
except, still using option = parse.parse_args() instead of args = ...

- using the following script in folder fdroidserver:
```
for i in *.py; do
sed -i -e 's/optparse/argparse/' \
-e 's/OptionParser/ArgumentParser/' \
-e 's/OptionError/ArgumentError/' \
-e 's/add_option/add_argument/' \
-e 's/(options, args) = parser/options = parser/' \
-e 's/options, args = parser/options = parser/' \
-e 's/Usage: %prog/%(prog)s/' $i;
done
```
- use ArgumentParser argument to replace (option, args) = parser.parse()
  call
- use parser.error(msg) instead of raise ArgumentException as suggested
  in https://docs.python.org/2/library/argparse.html#exiting-methods
- in fdroid catch ArgumentError instead of OptionError

See merge request !75

1  2 
fdroidserver/build.py
fdroidserver/checkupdates.py
fdroidserver/lint.py
fdroidserver/rewritemeta.py

Simple merge
Simple merge
Simple merge
Simple merge