[Bug 1269609] Review Request: ari-backup - A wrapper around rdiff-backup

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

 



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



--- Comment #9 from Richard Shaw <hobbes1069@xxxxxxxxx> ---
(In reply to Randy Barlow from comment #5)
> > - Package contains BR: python2-devel or python3-devel
> > 
> > I'm questioning this one, assuming this package is both python 2 and 3
> > compatible, and you're using %{_python}, then you would want the build
> > requirement to be whichever python is the default for that system. If it's
> > not python3 compatible we'll need to specify python2 everywhere.
> 
> I think I'm missing something here. I don't see the string "devel" in my
> spec file. Am I doing something that is bringing this requirement in
> automatically? This package is only Python 2 currently - do I need to do
> something different than I currently am doing to make that clear?

Ok, python-setuptools is pulling in python2-devel for you. Since it appears
that Python 3 is not supported at this time then you must specifically require
Python 2 in two different places:

1. In the build requirements
BuildRequires: python2-devel

2. When calling python:

%{__python} -> %{__python2}

This is required because at some point Fedora will switch to Python 3 by
default and the definition of %{__python} will change.

https://fedoraproject.org/wiki/Packaging:Python

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
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]