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

 
 

patch #7979: AX_BOOST_PYTHON: use correct Python version

Submitted by:  Daniel Müllner <muellner>
Submitted on:  Mon 25 Mar 2013 06:21:42 PM UTC  
 
Priority: 5 - NormalStatus: Done
Assigned to: Peter Simons <simons>Open/Closed: Closed

Add a New Comment (Rich MarkupRich Markup):
   

You are not logged in

Please log in, so followups can be emailed to you.

 

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>
Project AdministratorIn charge of this item.
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 File(s):
   
   
Comment:
   

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

 

Depends on the following items: None found

Items that depend on this one: None found

 

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

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 5 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Thu 04 Apr 2013 10:12:06 AM UTCsimonsStatusReady For Test=>Done
      Open/ClosedOpen=>Closed
    Mon 25 Mar 2013 10:52:58 PM UTCsimonsStatusNone=>Ready For Test
      Assigned toNone=>simons
    Mon 25 Mar 2013 06:21:42 PM UTCmuellnerAttached File-=>Added ax_boost_python.m4, #27684

    Back to the top


    Powered by Savane 3.1-cleanup