Re: [boost] Review request: auto-index tool

Subject: Re: [boost] Review request: auto-index tool
From: Daniel James (dnljms_at_[hidden])
Date: 2011-01-21 17:14:06

On 21 January 2011 16:45, Dave Abrahams <dave_at_[hidden]> wrote:
> At Fri, 21 Jan 2011 09:01:39 -0500,
> Bryce Lelbach wrote:
>>
>> While auto-index is still not a part of boost, I think you should check for the
>> environmental variable BOOST_ROOT so that users can checkout auto-index and use
>> it outside of their boost-trunk source tree. I see that you had:
>>
>>  local boost = [ modules.peek : BOOST ] ;
>>
>> I've never seen the above done before to locate boost. I usually use:
>>
>>  import os ;
>>  local BOOST_ROOT = [ os.environ BOOST_ROOT ] ;
>>
>> I'm sure volodya could tell us which is more portable.
>
> Either one can work, but John's way is better.

We could be more consistent about environment variables - sometimes
it's BOOST, sometimes it's BOOST_ROOT.

Daniel