Message39178
Logged In: YES
user_id=21627
The patch looks quite good. There are a number of remaining
issues that need to be resolved, though:
- please regenerate the patch against the current CVS. As
is, it fails to apply; parts of it are already in the CVS
(the thr_create changes)
- I think the SOVERSION should be 1.0, atleast initially:
for most Python releases, there will be only a single
release of the shared library, which should be named 1.0.
- Why do you think that no rpath is needed on Linux? It is
not needed if prefix is /usr, and on many installations, it
is also not needed if prefix is /usr/local. For all other
configurations, you still need a rpath on Linux.
- IMO, there could be a default case, assuming SysV-ish
configurations. |
|
Date |
User |
Action |
Args |
2007-08-23 15:11:29 | admin | link | issue527027 messages |
2007-08-23 15:11:29 | admin | create | |
|