Adds python-distutils-extra as a bin dependency.

Bug #643655 reported by Owais Lone
22
This bug affects 5 people
Affects Status Importance Assigned to Milestone
python-distutils-extra
Fix Released
Undecided
Unassigned

Bug Description

 python-distutils-extra is added as depends as well as under build-depends.

 python-distutils-extra is not required to run the package.

Owais Lone (loneowais)
description: updated
Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

ok, from what I see "the guilty commit" is http://bazaar.launchpad.net/~python-distutils-extra-hackers/python-distutils-extra/debian/revision/199
The old module took python-distutis-extra as part of the python base module, hence the fact it wasn't added as a binary dependency.

As setup.py is scanned by the install_egg_info call, depends from there are added to the egg file, and so, as bin dep.

The ideal situation and smartest way would be:
- dep in setup.py -> build-dep
- dep everywhere else -> dep

but the egg files contains both and there is no distinction AFAIK. Looking up at distutils content, it seems to be deep in dist.py requires… Not sure there is a way without touching it to tell "please ignore that file". We can workaround it in ignore p-d-e ourself, but what if the application really needs it?

scanning again ourself the tree can be suboptimal as well, not really sure what we should do for that (wait for distribute? ;))

Changed in python-distutils-extra:
status: New → Triaged
Revision history for this message
Michael Terry (mterry) wrote :

I just proposed https://code.launchpad.net/~mterry/python-distutils-extra/skip-setup.py for merging. It basically just skips a file in the toplevel directory called 'setup.py' when calculating module requirements.

However, Didier did bring up a good point that build-deps are manual right now and it would be nice if distutils-extra could also calculate those from setup.py. I'll open a separate wishlist bug for that.

Revision history for this message
Michael Terry (mterry) wrote :

Build time dependency request filed as bug 652162.

Revision history for this message
Martin Pitt (pitti) wrote :

Merged with the accompanying test case update. Thanks Mike!

Changed in python-distutils-extra:
status: Triaged → Fix Committed
Revision history for this message
Owais Lone (loneowais) wrote :

Thanks Guys. These dependencies turned even tiny 50kb apps into mega bytes.

Martin Pitt (pitti)
Changed in python-distutils-extra:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.