[Bug 526126] Review Request: python3 - Python 3.x (backwards incompatible version)

bugzilla at redhat.com bugzilla at redhat.com
Fri Jan 8 02:12:38 UTC 2010


Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=526126





--- Comment #56 from Dave Malcolm <dmalcolm at redhat.com>  2010-01-07 21:12:33 EDT ---
Replying to comment #55:
Thanks!  I've added a %check to the specfile.  I used
  LD_LIBRARY_PATH=$(pwd)
rather than
  LD_LIBRARY_PATH=.
as I ran into problems that appeared to occur when a subprocess had changed
working directory and could no longer find libpython.  See the diff below for
more info.

Updated specfile: http://dmalcolm.fedorapeople.org/python3.spec
Updated SRPM: http://dmalcolm.fedorapeople.org/python3-3.1.1-11.fc12.src.rpm
Diff: http://dmalcolm.fedorapeople.org/python3-from-3.1.1-10-to-3.1.1.11.diff
rpmlint output as before (see comment #27, comment #28, and comment #38)
Koji scratch build into dist-f13 here:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1908523

Following up on comment #46:

Issues addressed in release 10:
>  - fixup the factual errors in the comment describing redefinition of
>__os_install_post
>  - cosmetic issue: exec vs find | xargs noted in comment #36

Remaining issues:
>  - a reviewer needs to go through the full review guidelines on this
TODO
>  - perhaps fixup rpm-build to avoid needing
> find-provides-without-python-sonames.sh
>  - fixup macros.python3 to bake in the definitions, avoid invoking python3
> each time
>  - verify the script in comment #17 still works and that it verifies the 2 and
> 3 packages are independent
>  - what files are affected when modifying shebangs, and how (see commment #39)
>  - anything else I've missed

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.




More information about the Fedora-package-review mailing list