[Pytables-users] setup.py detection of hdf5 runtim
Hi, Would it be possible to patch the setup.py so that it searches for the runtime libraries in the provided path for hdf5 rather than just a set of default directories? We frequently have to install on systems where we don't have access to the default directories and have to append to LD_LIBRARY_PATH/DYLD_LIBRARY_PATH. The current version setup.py always generates a warning for us (and is actually finding the wrong hdf5 runtime when it DOESN'T generate a warning). Here's what I'm proposing: --- a/setup.py +++ b/setup.py @@ -155,7 +155,7 @@ class Package(object): # An explicit path can not be provided for runtime libraries. # (The argument is accepted for compatibility with previous methods.) return _find_file_path( -self.runtime_name, default_runtime_dirs, +self.runtime_name, locations, self._runtime_prefixes, self._runtime_suffixes ) -Chris -- Special Offer -- Download ArcSight Logger for FREE! Finally, a world-class log management solution at an even better price-free! And you'll get a free Love Thy Logs t-shirt when you download Logger. Secure your free ArcSight Logger TODAY! http://p.sf.net/sfu/arcsisghtdev2dev ___ Pytables-users mailing list Pytables-users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/pytables-users
Re: [Pytables-users] setup.py detection of hdf5 runtim
Hi Kris, There have been some other build path issues recently as well. It would be good to consider this. I know it is only a one-line change, but would you mind submitting this as a pull request on github? Thanks, Anthony On Fri, Sep 2, 2011 at 9:47 AM, Chris Kees cek...@gmail.com wrote: Hi, Would it be possible to patch the setup.py so that it searches for the runtime libraries in the provided path for hdf5 rather than just a set of default directories? We frequently have to install on systems where we don't have access to the default directories and have to append to LD_LIBRARY_PATH/DYLD_LIBRARY_PATH. The current version setup.py always generates a warning for us (and is actually finding the wrong hdf5 runtime when it DOESN'T generate a warning). Here's what I'm proposing: --- a/setup.py +++ b/setup.py @@ -155,7 +155,7 @@ class Package(object): # An explicit path can not be provided for runtime libraries. # (The argument is accepted for compatibility with previous methods.) return _find_file_path( -self.runtime_name, default_runtime_dirs, +self.runtime_name, locations, self._runtime_prefixes, self._runtime_suffixes ) -Chris -- Special Offer -- Download ArcSight Logger for FREE! Finally, a world-class log management solution at an even better price-free! And you'll get a free Love Thy Logs t-shirt when you download Logger. Secure your free ArcSight Logger TODAY! http://p.sf.net/sfu/arcsisghtdev2dev ___ Pytables-users mailing list Pytables-users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/pytables-users -- Special Offer -- Download ArcSight Logger for FREE! Finally, a world-class log management solution at an even better price-free! And you'll get a free Love Thy Logs t-shirt when you download Logger. Secure your free ArcSight Logger TODAY! http://p.sf.net/sfu/arcsisghtdev2dev___ Pytables-users mailing list Pytables-users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/pytables-users
Re: [Pytables-users] setup.py detection of hdf5 runtim
Hi Chris, Il 02/09/2011 16:47, Chris Kees ha scritto: Hi, Would it be possible to patch the setup.py so that it searches for the runtime libraries in the provided path for hdf5 rather than just a set of default directories? We frequently have to install on systems where we don't have access to the default directories and have to append to LD_LIBRARY_PATH/DYLD_LIBRARY_PATH. The current version setup.py always generates a warning for us (and is actually finding the wrong hdf5 runtime when it DOESN'T generate a warning). Here's what I'm proposing: With The current version setup.py do you mean the one from the PyTables 2.2 package tarball? In case, it would be nice if you could give a try to the latest version currently in master on github https://github.com/PyTables/PyTables/ There is another small patch on my personal area https://github.com/avalentino/PyTables/commit/6e6c9ad7a09bfd97310a2ef8ed1ea526b11acc1e that could be useful in this case. Also it would be useful if you could give us more details (e.g. the exact command line, setup.py output, relevant environment and so on). --- a/setup.py +++ b/setup.py @@ -155,7 +155,7 @@ class Package(object): # An explicit path can not be provided for runtime libraries. # (The argument is accepted for compatibility with previous methods.) return _find_file_path( -self.runtime_name, default_runtime_dirs, +self.runtime_name, locations, self._runtime_prefixes, self._runtime_suffixes ) -Chris Thanks -- Antonio Valentino -- Special Offer -- Download ArcSight Logger for FREE! Finally, a world-class log management solution at an even better price-free! And you'll get a free Love Thy Logs t-shirt when you download Logger. Secure your free ArcSight Logger TODAY! http://p.sf.net/sfu/arcsisghtdev2dev ___ Pytables-users mailing list Pytables-users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/pytables-users
