patchGNU Autoconf Archive - Patches: patch #7979, AX_BOOST_PYTHON: use correct...

 
 

patch #7979: AX_BOOST_PYTHON: use correct Python version

Submitter:  Daniel Müllner <muellner>
Submitted:  Mon 25 Mar 2013 06:21:42 PM UTC
   
 
Priority:  5 - Normal Status:  Done
Assigned to:  simons Open/Closed:  Closed
* Mandatory Fields

Add a New Comment Rich Markup
   

Mon 25 Mar 2013 10:52:58 PM UTC, comment #1: 

Pushed in commit 7df1624efd4985e44a65566afcff68f6de58dfe1. Thank you very much for the patch!

Peter Simons <simons>
Group administrator
Mon 25 Mar 2013 06:21:42 PM UTC, original submission:  

The AX_BOOST_PYTHON macro uses AX_PYTHON internally to find the include directory for the Python headers. I propose to switch to AX_PYTHON_DEVEL. Aside from being the more canonical choice for the job, this has the following crucial advantage:

Unfortunately, AX_PYTHON always reports the first Python interpreter it finds. As far as I know, there is no way to tell AX_PYTHON which Python version to use on systems where both Python2 and Python3 are installed. AX_PYTHON_DEVEL is better in this respect, as it takes the variable PYTHON_VERSION into account (which is set by the macro AM_PATH_PYTHON). As a consequence, a call to "./configure PYTHON=python2" with an autoconf script containing

AM_PATH_PYTHON
AX_PYTHON_DEVEL
AX_BOOST_PYTHON

sets the header and include directories for Python2 but still links to libboost_python3 instead of libboost_python.

This is fixed in the attached patch. Required changes are very minor. I do not see anything that could cause incompatibilities with old code.

Daniel Müllner <muellner>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #27684:  ax_boost_python.m4 added by muellner (4KiB - application/x-m4)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by simons (Posted a comment)
  • -email is unavailable- added by muellner (Submitted the item)
  •  

    Follow 5 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2013-04-04 simons StatusReady For Test Done
        Open/ClosedOpen Closed
    2013-03-25 simons StatusNone Ready For Test
        Assigned toNone simons
    2013-03-25 muellner Attached File- Added ax_boost_python.m4, #27684

    Back to the top

    Powered by Savane 3.13-f8d8.
    Corresponding source code