Re: [boost] [python] patch for python.jam for setting suffix onHP-UX

From: Boris Gubenko (Boris.Gubenko_at_[hidden])
Date: 2007-11-16 13:40:30

David Abrahams wrote:
> I don't think this is really the recommended way to determine the
> platform, is it?  Could you check with Volodya?

I "stole" it from tools/build/v2/util/os.jam :

.platform = [ modules.peek : OSPLAT ] ;

I need to distinguish two different architectures of HP-UX: ia64 and
PA-RISC.

I'm copying Volodya on the reply, in case he does not watch this
thread.

Thanks,
  Boris

----- Original Message ----- 
From: "David Abrahams" <dave_at_[hidden]>
To: <boost_at_[hidden]>
Sent: Friday, November 16, 2007 1:20 PM
Subject: Re: [boost] [python] patch for python.jam for setting suffix onHP-UX

>
> on Mon Nov 12 2007, "Boris Gubenko" <Boris.Gubenko-AT-hp.com> wrote:
>
>> There is a logic in python.jam setting suffix for Boost.Python extension
>> files on HP-UX depending on version of Python interpreter. This logic does
>> not take into account the fact that on HP-UX PA-RISC, the suffix must be
>> ".sl", regardless of the version of Python interpreter (I myself was not
>> aware of this fact before yesterday when I saw the results of Boost.Python
>> tests on HP-UX_pa_risc_gcc with Python 2.5: the tests compile, but fail
>> at run time).
>>
>> Attached patch fixes the problem. Verified on HP-UX PA-RISC with Python
>> 2.5. Verified on HP-UX ia64 with Python 2.3 and 2.5 that it does not cause
>> any regression. The patch relies on bjam setting correctly OSPLAT variable,
>> so, for bjam compiled with aC++ on HP-UX ia64, the patch for jam.h in
>>
>>   http://lists.boost.org/boost-build/2007/11/17975.php
>>
>> should be committed first.
>>
>> Ok to commit?
>>
>> Thanks,
>>   Boris
>>
>> Index: tools/python.jam
>> ===================================================================
>> --- tools/python.jam    (revision 41021)
>> +++ tools/python.jam    (working copy)
>> @@ -991,7 +991,9 @@
>>        case cygwin : suffix = dll ;
>>        case hpux :
>>        {
>> -          if [ feature.get-values python : $(condition) ] in 1.5 1.6 2.0 2.1
>> 2.2 2.3 2.4
>> +          platform = [ modules.peek : OSPLAT ] ;
>> +          if $(platform) != IA64 ||
>> +             [ feature.get-values python : $(condition) ] in 1.5 1.6 2.0 2.1
>> 2.2 2.3 2.4
>>            {
>>                suffix = sl ;
>>            }
>
> Boris,
>
> I don't think this is really the recommended way to determine the
> platform, is it?  Could you check with Volodya?
>
> Thanks,
>
> -- 
> Dave Abrahams
> Boost Consulting
> http://www.boost-consulting.com
>
> _______________________________________________
> Unsubscribe & other changes: http://lists.boost.org/mailman/listinfo.cgi/boost
>