search

xolox / python-deb-pkg-tools

Debian packaging tools
https://pypi.python.org/pypi/deb-pkg-tools
MIT License
42 stars 11 forks source link

Get rid of as much dependencies as possible and replace some existing dependencies with alternatives which may be better #13

Closed KOLANICH closed 5 years ago

KOLANICH commented 5 years ago

Hi. First of all, thank you for an awesome lib.

But it has some drawbacks. I mean its mandatory dependencies.

  1. python-memcached - do I really need memcached to work with deb packages using this lib? Can this be made optional?
  2. humanfriendly and coloredlogs - could these be made optional?
  3. sh is more pythonic alternative to executor.
xolox commented 5 years ago

Hi and thanks for the feedback. In deb-pkg-tools 6.0 (just released) I've changed python-memcached to an optional dependency, indeed most users of deb-pkg-tools won't need this (even though I definitely do). I'm not going to change the other dependencies though, because deb-pkg-tools isn't just a library but also a command line program, and I'm not inclined to split the two responsibilities up into separate Python packages just so one or two (lightweight) libraries can be made optional.

KOLANICH commented 5 years ago

because deb-pkg-tools isn't just a library but also a command line program, and I'm not inclined to split the two responsibilities up into separate Python packages just so one or two (lightweight) libraries can be made optional.

I guess it is OK to make them an extra with the name CLI and instruct a user to install them on the first run of the CLI if anything is missing