[Bug 611040] Review Request: python-importlib - Backport of importlib.import_module() from Python 2.7

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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=611040

Thomas Spura <tomspur@xxxxxxxxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |dmalcolm@xxxxxxxxxx,
                   |                            |tomspur@xxxxxxxxxxxxxxxxx
         AssignedTo|nobody@xxxxxxxxxxxxxxxxx    |tomspur@xxxxxxxxxxxxxxxxx
               Flag|                            |fedora-review?

--- Comment #1 from Thomas Spura <tomspur@xxxxxxxxxxxxxxxxx> 2010-07-05 09:45:12 EDT ---
Don't you need at least a Conflicts with python(abi) = 2.7?

In python-celery you currently commented out the requires on this package, this
could be resolved by e.g.:

%{!?pyver: %global pyver %(%{__python} -c "import sys ; print
sys.version[:3]")}
%if pyver < 2.7
Requires:  python-importlib
%endif

This way, when rebuilding all packages with python2.7 (or above) this will be
cone automatic.

Furthermore, when using a conflicts on python2.7, this package won't being
build in >= 2.7 and everyone will notice, that this can be a dead.package.


Do you (=Fabian) have another resolution for this, or David, what's your
preference?


(Anything else looks fine at the first sign.)

-- 
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.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]