Re: [Rdkit-devel] [PATCH 2/2] Install libraries in ${RDKit_LibDir}

[Greg L. <gre...@gm...>]

Dear Gianluca,

On Mon, Jul 19, 2010 at 1:07 AM, Gianluca Sforna <gi...@gm...> wrote:
>
> On Wed, Jul 14, 2010 at 5:43 PM, Greg Landrum <gre...@gm...> wrote:
>
>> If there's some way to remove it that works on linux and windows (the
>> mac normally works if linux does), and there's still a way to use
>> $RDBASE if it's set, I think it would be excellent if $RDBASE were no
>> longer required.
>
> A short term solution could be to hardcode the RDBASE value in
> RDConfig.py at build time (since we know where stuff will land in the
> installed system) while allowing override from the env variable at
> runtime.

I guess I don't see how we're going to know at build time where things
will be installed. That's not known until the user does "make install"
(or, in the case of the package, installs the package). Are you
suggesting we take sensible defaults (by operating system) and just
use those?

> Lastly, I pulled the packaging branch from SVN and attempted the
> build; I noticed the destination directories are repeated twice as in:
> /usr/share/RDKit/Data/Data/SmartsLib/RLewis_smarts.txt

doh!

Fixed in svn... thanks.

-greg