bad packaging about impacket HOT 9 CLOSED

Hrm.  Looking at the setup.py for the 0.9.9.9 release it seems that you failed 
to update the setup(version) prior to doing the builds.  This seems to be a 
chronic issue as it appears to be the case on svn/tags/impacket_0_9_8_0/ and 
svn/tags/impacket_0_9_7_0/ also.  You really need to make these things all 
match up.

You cannot simply rename the resulting .tar.gz and expect everything to be good.

Hey Brian:

Thanks for the report and sorry for the messed up package. We'll be fixing it 
these days.

Stay tunned.

Hi.

To be clear, you need to adjust the version in setup.py to reflect what you 
want the resulting release to ship as, before you do the release process so 
that things are all named/versioned correctly.  I would suggest that that 
version given should match the release branch/tag name.

I've modified some of the properties of the setup.py file. This new version is 
attached, does this modifications fix the issues you have to create the package.

Well, that setup.py is fine if what you want as a result is 
impacket-0.9.9.9.tar.gz.  But you have already released 0.9.9.9 at:

http://corelabs.coresecurity.com/index.php?module=Wiki&action=attachment&type=to
ol&page=Impacket&file=impacket-0.9.9.9.tar.gz

Surely you don't actually want to re-release 0.9.9.9 right?  If you don't, then 
you need to decide what the version of the next release will be and set the 
version property to that value.

Currently on trunk the version is 1.0.0.0-dev.  That seems like a reasonable 
version while trunk is in development given that the previous release was 
0.9.9.9.  But at the point where you are ready to do an official release, you 
would want to set that version to "1.0.0.0" so that the tarball you get is 
impacket-1.0.0.0.tar.gz (the zip file will be similar).  It's that tarball (and 
zip file) that you make available as your release to your users, _WITHOUT_ 
doing anything to them to change the version in their name.  That's the 
important part.  Once "setup bdist" (or whatever variant you use) has created 
and named the tarball and/or zip files, you cannot change the version (nor 
should you really change the name) in the file name.

Additionally at the point in the source tree where you did the "setup bdist" to 
make your 1.0.0.0 release (with the setup.py version="1.0.0.0" you would create 
a tag "impacket_1_0_0_0" so that you have perfect alignment of setup.py 
version= value, the names on the released packages and the SCCS tag.

Hey Brian..

Thanks for all your points.. we will not be changing 0.9.9.9 since, as you 
said, it's already out there. 

We will incorporate these changes whenever we release the next version.

thanks again,
beto

We just published 0.9.10.. Hopefully those problems are solved.