Bug 168751

Summary: rhpl hardcodes the python version, making it harder to backport
Product: [Fedora] Fedora Reporter: Toshio Kuratomi <toshio>
Component: rhplAssignee: Jeremy Katz <katzj>
Status: CLOSED RAWHIDE QA Contact:
Severity: medium Docs Contact:
Priority: medium    
Version: 4   
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2006-02-02 23:05:10 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 150222    
Attachments:
Description Flags
Patch to let rhpl's Makefile detect the python version none

Description Toshio Kuratomi 2005-09-20 01:58:00 UTC 
From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.7.10) Gecko/20050909 Fedora/1.0.6-1.2.fc4 Firefox/1.0.6

Description of problem:
rhpl hardcodes the version of python that it builds with in the Makefile.inc file.  This does not appear to be because it can only use python2.4 but because python2.4 is the version that is available in the Fedora Core version for which it is released.  This makes it harder to port the library to a distribution (for instance FC3) that uses a different version of python.

Additionally, it requires that the python interpreter value be updated by the rhpl maintainer whenever the Core python package changes.

Version-Release number of selected component (if applicable):
rhpl-0.173-1

How reproducible:
Always

Steps to Reproduce:
1. mock -r fedora-3-x86_64-core rhpl-0.173-1.src.rpm
2.
3.
  

Actual Results:  Mock failed to rebuild the rpm for FC3.

Expected Results:  Generate rpms for FC3.

Additional info:

This was talked about in a fedora-packaging thread[1] as part of why rpm needs more macros related to python.  For this problem it seemed it was more useful to fix it in the tarball than in the spec file.  Attaching a patch to fix it in the Makefile.inc.

[1] https://www.redhat.com/archives/fedora-packaging/2005-September/thread.html
Comment 1 Toshio Kuratomi 2005-09-20 02:05:21 UTC 
Created attachment 119008 [details]
Patch to let rhpl's Makefile detect the python version

This is a simple patch to Makefile.inc.  It runs the same python version
detection code as rpm's pythondeps.sh script to determine which version of
python we're building with so the package will be built with the correct
includes and installed to the correct directory.

Notes:	This works as long as the build system has the version of python
installed in the path that the package is to be compiled to.  This is true for
the Extras build system but I don't know if it's the case for the current Core
buildsystem.
Comment 2 Jeremy Katz 2005-09-20 21:47:24 UTC 
rhpl is about to undergo a variety of changes (to make it more sane for me to
maintain).  One of the changes is probably going to involve me switching it to
using distutils.

So I'm going to hold off on applying this for now.  I'll stick it on FC5Blocker,
though, so that it doesn't get lost.
Comment 3 Toshio Kuratomi 2005-09-20 22:55:37 UTC 
Nice.

FWIW, my first attempt to make this sane was to create a distutils setup script
but I realized that distutils doesn't have a "target" for message catalogs so
the po directory would require extra work to build.
Comment 4 Jeremy Katz 2006-02-02 23:05:10 UTC 
Didn't get all I wanted to done with rhpl for FC5, but went ahead and applied
this in CVS