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
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 (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  Edit
Everyone can see this information.

Duplicates of this bug

Other bug subscribers