Boost logo

Boost :

Subject: Re: [boost] [Boost-users] Boost 1.65.0 Beta 1 [boost.python] issue
From: Rene Rivera (grafikrobot_at_[hidden])
Date: 2017-07-14 23:14:19


On Fri, Jul 14, 2017 at 7:23 AM, Raffi Enficiaud via Boost-users <
boost-users_at_[hidden]> wrote:

> As I said in a previous post, I cannot get boost.python to build on
> Windows: the target is just ignored (see stack trace below).
>

Missed that post :-(

> I think there is a bug in .\tools\build\src\tools\python.jam

A highly probable statement all on its own :-)

> concerning the version computation: the variable "major-minor" is not set
> properly if "version" is optional.
>
> In this block:
>
> ============= ("rule configure")
> if $(version)
> {
> if ! $(version) in [ feature.values python ]
> {
> feature.extend python : $(version) ;
> py$(major-minor[1])-version ?= $(version) ;
> =============
>
> The variable "major-minor" is empty.
>
> The "if" in the following block prevents from computing major-minor as
> "version" is optional and is not set:
>
> =============
> local major-minor ;
> if $(version)
> {
> major-minor = [ split-version $(version) ] ;
> version = $(major-minor:J=.) ;
> }
> =============
>
> I believe this can be fixed by recomputing "major-minor" in this block,
> after "version" has been inferred.
>
> =============
> if $(cmd)
> {
> debug-message ...requested configuration matched! ;
>
> exec-prefix = $(sys.exec_prefix) ;
>
> compute-default-paths $(target-os) : $(sys.version) :
> $(sys.prefix) : $(sys.exec_prefix) ;
>
> version = $(sys.version) ;
> interpreter-cmd ?= $(cmd) ;
> cmds-to-try = ; # All done.
> }
> =============
>

Seems reasonable.. Can you put up a PR for that?

-- 
-- Rene Rivera
-- Grafik - Don't Assume Anything
-- Robot Dreams - http://robot-dreams.net
-- rrivera/acm.org (msn) - grafikrobot/aim,yahoo,skype,efnet,gmail

Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk