Re: [Rdkit-devel] [PATCH 2/2] Install libraries in ${RDKit_LibDir}
Open-Source Cheminformatics and Machine Learning
Brought to you by:
glandrum
|
From: Gianluca S. <gi...@gm...> - 2010-07-14 14:19:09
|
On Wed, Jul 14, 2010 at 6:38 AM, Greg Landrum <gre...@gm...> wrote: > Dear Gianluca, > > I made a couple of modifications to the packaging scripts to be sure > that some of the "extra" directories like $RDBASE/Data also get > installed. These are in svn on the packaging branch. Great, I just cloned the code from SVN to have a look the latest changes > > At the moment the system still needs the RDBASE environment variable > so that the python code can find some of its extra data; I'm not quite > sure what to do about this. Yeah, RDBASE is my next target. While technically nothing prevents packages from adding env variables, it's better to not pollute the system if we can avoid it. Other than for tests (where I guess it's fine to leave it as is) do you know what RDBASE is used for at runtime? I already started patching the code to remove the runtime dependency, but I'd like to hear your opinion on the topic before digging further in this task. Cheers G. -- Gianluca Sforna http://morefedora.blogspot.com http://www.linkedin.com/in/gianlucasforna |