Last Comment Bug 761105 - egg-info directories checked in the source tree
: egg-info directories checked in the source tree
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: Build Config (show other bugs)
: unspecified
: All All
: -- normal (vote)
: ---
Assigned To: Mike Hommey [:glandium]
:
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-06-04 03:07 PDT by Mike Hommey [:glandium]
Modified: 2012-06-04 04:54 PDT (History)
3 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Remove build/pylib/blessings/blessings.egg-info and other-licenses/virtualenv/virtualenv.egg-info (65.79 KB, patch)
2012-06-04 03:08 PDT, Mike Hommey [:glandium]
ted: review+
Details | Diff | Splinter Review

Description Mike Hommey [:glandium] 2012-06-04 03:07:32 PDT
Thanks to bug 661908, egg-info directories are created in the source tree (bug 758823). But there's something even better: there are egg-info directories checked in mercurial:
- build/pylib/blessings/blessings.egg-info
- other-licenses/virtualenv/virtualenv.egg-info

The latter comes from bug 661908 itself, but the former pre-dates it. So running configure may modify files under build/pylib/blessings/blessings.egg-info (and it happened to me in https://hg.mozilla.org/try/rev/7c9f4fa9c5a1 )
Comment 1 Mike Hommey [:glandium] 2012-06-04 03:08:49 PDT
Created attachment 629736 [details] [diff] [review]
Remove build/pylib/blessings/blessings.egg-info and other-licenses/virtualenv/virtualenv.egg-info
Comment 2 Gregory Szorc [:gps] 2012-06-04 03:37:05 PDT
I just downloaded both of these packages from PyPI and both tar.gz archives contain egg-info directories.

I thought egg-info directories from source packages provided (possibly important) metadata. What would removing them accomplish?

As far as the blessings metadata diff, it is listed as version 1.0 in the upstream package. When I checked in the virtualenv integration a few days ago, I checked in the bump to 1.1. However, it appears it was regressed (by accident, likely) in 0d736d4fa057. *sigh*

To further complicate matters, my work in bug 751795 may require further changes to how the Python environment is configured. For that to be a true front-end to the build system, it needs to wrap configure, which is where virtualenv is being created currently. Ugh. But, that's for a different time, luckily.

Why does Python packaging have to be so complex?
Comment 3 Ted Mielczarek [:ted.mielczarek] 2012-06-04 04:20:29 PDT
Beats me, man.
Comment 4 Mike Hommey [:glandium] 2012-06-04 04:21:15 PDT
(In reply to Gregory Szorc [:gps] from comment #2)
> I just downloaded both of these packages from PyPI and both tar.gz archives
> contain egg-info directories.
> 
> I thought egg-info directories from source packages provided (possibly
> important) metadata. What would removing them accomplish?

Avoiding this:
> When I checked in the virtualenv integration a few days
> ago, I checked in the bump to 1.1. However, it appears it was regressed (by
> accident, likely) in 0d736d4fa057. *sigh*


> Why does Python packaging have to be so complex?

The alternative question is: do we really have to run setup.py on these things when we never had to? It might be easier for virtualenv, but it really feels we'd have much less problems if we "installed" that stuff in the virtualenv "manually".
Comment 5 Mike Hommey [:glandium] 2012-06-04 04:54:45 PDT
https://hg.mozilla.org/mozilla-central/rev/77f792cd4483

Note You need to log in before you can comment on or make changes to this bug.