search

yaqwsx / PcbDraw

Convert your KiCAD board into a nicely looking 2D drawing suitable for pinout diagrams
MIT License
1.13k stars 90 forks source link

Obsolete argparse dependency #35

Closed set-soft closed 4 years ago

set-soft commented 4 years ago

The current setup.py asks for argparse package.

But this functionality became part of standard Python on v3.2. The current version is 3.8.3, and even the conservative Debian stable is using 3.7.3 (released on march 2019). Python 3.2 was released in february 2011.

So I think this dependency isn't really needed. I guess it got there when PcbDraw was Python 2, but for Python 3 there is no need to install argparse as a separated package.

BTW: I'm currently using docopt, well in fact a patched version to support args.argument (in addition to args["argument"]). This package gets the command line specification from the doctring.

yaqwsx commented 4 years ago

You are right, fixed in d20258f

About docopt - I prefer to use click in my other projects. Unfortunately, when I wrote the first version of PcbDraw, I wasn't aware about it. That's why I used argparse.