setup.py clean should delete SOURCES.txt

setup.py clean doesn't delete foo.egg-info/SOURCES.txt. As a result Debian does this in each packages manually (otherwise as SOURCES.txt changes during the build it gets included into the debian's source-diff). I think the proper place to delete it is by distribute/setuptools itself.

The clean command is a Distutils command, meaning that it is not aware of the existence of the .egg-info directory created by Distribute/Setuptools.

I'd be happy to include a custom one in Distribute. One question though: why removing only SOURCES.txt ? the "foo.egg-info" directory is fully created at build time, meaning that we can safely remove it by calling clean.

Thanks for the quick reply. Sure that would be great. Delete the directory. SOURCES.txt was just the file that was causing problems.

Thanks for the quick response, Tarek.

I wonder, though, what use the egg-info directory has in the source distribution at all? It seems bizarre to create it for distribution, then throw it away when actually building or installing.

Would a better solution be to simply not *create* the egg-info directory for an sdist command?

The foo.egg-info directory is specified as a place where module authors can store their own metadata (e.g. http://peak.telecommunity.com/DevCenter/setuptools#adding-new-egg-info-files ). So deleting all files in it (that are not autogenerated by distutils/setuptools/distribute) seems a bad idea to me.

@Ben Chester Finney

That wouldn't solve the problem (it would actually make it worse). Then SOURCES.txt and other generated files would be included into debian's diff (if not explicitly deleted in each debian package; on second builds). If you consider foo.egg-info in sdist to be a problem (I don't) please file a separate bug report.

@anonymous : those custom files *are* autogenerated by the egg_info command each time sdist is called. It uses the writers registered as entry poitns for that. So fully removing this directory is not a problem as far as I can see.

So if everyone agrees, I propose to remove .egg-info altogether when the clean command is called.

More then a month passed and nobody objected. I think this should count as everyone agrees ;-). Sorry if this feels a bit pushy; I just don't want this to be stalled, unnecessarily.

I am currently very busy on distutils, and I can't devote time right now on this. If you have some time to do it, I'd be happy to review then include a patch

Do you have a Pointer for extending distribute? I tried to add a command called cleaned by copying command/register.py to command/cleand.py; renamed class register to class cleaned in cleaned.py and added 'cleaned' to all in init.py.

However:

$ python2.6 setup.py cleaned
[helpmessage]
Error: invalid command 'cleaned'

"$ python2.6 setup.py register" works and cleaned does import:

>>> from setuptools.command import cleaned
>>> dir(cleaned)
['__builtins__', '__doc__', '__file__', '__name__', '__package__', '_register', 'cleaned']

So I am very stuck.

Are there any news on this? One year has passed, but "myname.egg-info" dir still annoys by its existence after `setup.py clean`.

http://www.timberland-discountshoes.com/Bestsellers Cheap Timberland Boots http://www.timberland-discountshoes.com/sale/Timberland-Womens-6-Inch-Premium-Boot-Pink-244.html Timberland 6 inches http://www.timberland-discountshoes.com/sale/Timberland-Classic-2-Eye-Mens-Lug-Sole-Boat-Shoes-Chocolate-206.html Timberland Classic 2 Eye Mens Lug Sole Boat Shoes http://www.timberland-discountshoes.com/sale/Timberland-Roll-Top-Kids-Boots-In-Wheat-159.html Timberland boots http://www.timberland-discountshoes.com/Timberland-Womens-Chukka-Boots Timberland womens Chukka Boots http://www.timberland-discountshoes.com/sale/Timberland-Men-Beach-Sandals-Yellow-274.html Timberland shoes

http://www.cheapmonclerssale.com/Discount-Moncler-Safran-Long-Down-Women-Jackets-Beige-269/ Discount Moncler Mengs Women Down Jackets http://www.cheapmonclerssale.com/Moncler-Kids-c89/ Moncler kids Jackets http://www.cheapmonclerssale.com/Discount-Moncler-Branson-Down-Jacket--Purple-247/ Moncler Men Low Casual Shoes http://www.cheapmonclerssale.com/Moncler-Couple-Jackets-c88/ moncler women coats