After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 338527 - Python detection: redux
Python detection: redux
Status: RESOLVED FIXED
Product: libxslt
Classification: Platform
Component: general
1.1.15
Other Linux
: Normal enhancement
: ---
Assigned To: Daniel Veillard
libxml QA maintainers
Depends on:
Blocks:
 
 
Reported: 2006-04-14 19:48 UTC by Joseph Sacco
Modified: 2006-04-28 16:52 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch for python path detection (5.52 KB, patch)
2006-04-14 19:49 UTC, Joseph Sacco
none Details | Review
patch for python path detection (2.17 KB, patch)
2006-04-22 16:25 UTC, Joseph Sacco
reviewed Details | Review

Description Joseph Sacco 2006-04-14 19:48:36 UTC
Some GARNOME users have encountered problems building libxslt because of python
detection. These users typically are running older linux distos and have
installed a newer version of python in some location other than /usr, for
example, /opt/python-2.4.3.

libxslt is to be built using this newer version of python. The libxslt python
site_packages files are to be installed in the GARNOME tree, not the python
tree.

To make this happen two issues must be address:

Python detection
-----------------
Python should be detected in this order:
* user specifies path to python executable as a configuration argument
* user specifies path to python executable as an environment variable
* configure script hunts in the usual places

Python site_package installation
--------------------------------
Python site_packages should be installed in 

   $exec_prefix/lib/pythonX.Y/site-packages

Note:

   in general: $exec_prefix != $python_exec_prefix


Attached is a proposed patch for libxslt-1.1.15 that will resolve these issues.


-Joseph
Comment 1 Joseph Sacco 2006-04-14 19:49:22 UTC
Created attachment 63500 [details] [review]
patch for python path detection
Comment 2 Joseph Sacco 2006-04-22 16:25:30 UTC
Created attachment 64110 [details] [review]
patch for python path detection

Daniel,

Attached is a patch for configure.in that (1) works within existing code and
semantics of configure flags, and (2) resolves the issues of python detection
and site-package installation referenced above.

Note that in addition, some quoting foo has been cleaned up within the python
section.

Hindsight is an exact science... If the python section were to be rewritten
from scratch, using the stock macro definitions provided by aclocal in
python.m4 would be a good choice.
Comment 3 Daniel Veillard 2006-04-25 13:14:26 UTC
The patch didn't applied cleanly to CVS, but I think I managed to
apply it, and commited to CVS,

  thanks a lot,

Daniel
Comment 4 Daniel Veillard 2006-04-28 16:52:01 UTC
I cursed very much when I realized at release time that your patch
broke RPM build, when I use PYTHONXXX='$(prefix)/...'
it's that I meant it, not PYTHONXXX=$prefix/...

Fixed in CVS, but one lost hour...

Daniel