[Twisted-Python] latest release & solaris

Hi just a couple of notes of about the latest release 1) the tarball (gz) for unix seems to have some funny permissions/symlinks/umask? in it. This barfed using solaris tar, with an error about not being able to create the file, so I downloaded the windows version and unzipped that instead, and chmod -R to something sensible. 2) the C extension module cBanana.c actually has some C++ style comments in it. I think this is fine for gcc, but solaris' cc compiler is less forgiving. I just edited the file. other than those two gotchas, it seems to work. thanks Jon

From: Jon Dyte <jon@totient.demon.co.uk> Subject: [Twisted-Python] latest release & solaris Date: Thu, 6 Jun 2002 22:30:37 +0100 (BST)
Hi
just a couple of notes of about the latest release
1) the tarball (gz) for unix seems to have some funny permissions/symlinks/umask? in it. This barfed using solaris tar, with an error about not being able to create the file, so I downloaded the windows version and unzipped that instead, and chmod -R to something sensible.
The only permissions I see that are "funny" are that the directories are sgid. This is a common convention on our development server, so it snuck in there... We'll try not to do that next release. However, please use GNU tar to unpack this archive; as I understand it, solaris tar is buggy and uses an antiquated fileformat that can't handle long names, among other things.
2) the C extension module cBanana.c actually has some C++ style comments in it. I think this is fine for gcc, but solaris' cc compiler is less forgiving. I just edited the file.
Thanks for the bug report.
other than those two gotchas, it seems to work.
Good to know! -- | <`'> | Glyph Lefkowitz: Traveling Sorcerer | | < _/ > | Lead Developer, the Twisted project | | < ___/ > | http://www.twistedmatrix.com |

Jon Dyte wrote:
2) the C extension module cBanana.c actually has some C++ style comments in it. I think this is fine for gcc, but solaris' cc compiler is less forgiving. I just edited the file.
OK, this is fixed in CVS. Thanks for the bug report. BTW, if you could run our test suite and see all tests pass that'd be cool - they should, but it can't hurt to get tested on more platforms: Run in text mode: $ admin/runtests -t Run in text mode with poll() reactor: $ admin/runtests -t poll
participants (3)
-
Glyph Lefkowitz
-
Itamar Shtull-Trauring
-
Jon Dyte