#32 Restructure the package, so we can build python3 + python3-libs from python39
Merged 4 years ago by churchyard. Opened 4 years ago by churchyard.
rpms/ churchyard/python39 main_from_nonmain  into  master

file modified
+102 -85
@@ -28,21 +28,23 @@ 

  # Note that the bcond macros are named for the CLI option they create.

  # "%%bcond_without" means "ENABLE by default and create a --without option"

  

- 

- # Flat package, i.e. python36, python37, python38 for tox etc.

- # WARNING: This also influences the main_python bcond below.

- # in Fedora, never turn this on for the python3 package

- # and always keep it on for python37 etc.

- # WARNING: This does not change the package name and summary above.

- %bcond_without flatpackage

- 

  # Main Python, i.e. whether this is the main Python version in the distribution

  # that owns /usr/bin/python3 and other unique paths

- # Default: if this is a flatpackage -> it is not the main Python

- %if %{with flatpackage}

+ # This also means the built subpackages are called python3 rather than python3X

+ # WARNING: This also influences the flatpackage bcond below.

+ # By default, this is determined by the %%__default_python3_pkgversion value

+ %if 0%{?__default_python3_pkgversion} == %pyshortver

+ %bcond_without main_python

+ %else

  %bcond_with main_python

+ %endif

+ 

+ # Flat package, i.e. no separate subpackages

+ # Default (in Fedora): if this is a main Python, it is not a flatpackage

+ %if %{with main_python}

+ %bcond_with flatpackage

  %else

- %bcond_without main_python

+ %bcond_without flatpackage

  %endif

  

  # When bootstrapping python3, we need to build setuptools.
@@ -97,6 +99,13 @@ 

  # =====================

  # General global macros

  # =====================

+ %if %{with main_python}

+ %global pkgname python3

+ %global exename python3

+ %else

+ %global pkgname python%{pyshortver}

+ %global exename python%{pybasever}

+ %endif

  

  %global pylibdir %{_libdir}/python%{pybasever}

  %global dynload_dir %{pylibdir}/lib-dynload
@@ -139,6 +148,12 @@ 

  # on files that test invalid syntax.

  %undefine py_auto_byte_compile

  

+ # When a main_python build is attempted despite the %%__default_python3_pkgversion value

+ # We undefine %%python_provide so the python3-... package does not provide wrong python3X-...

+ %if %{with main_python} && (0%{?__default_python3_pkgversion} != %pyshortver)

+ %undefine python_provide

+ %{warn:Doing a main_python build with wrong %%__default_python3_pkgversion (0%{?__default_python3_pkgversion}, but this is %pyshortver)}

+ %endif

  

  # =======================

  # Build-time requirements
@@ -195,9 +210,6 @@ 

  # workaround http://bugs.python.org/issue19804 (test_uuid requires ifconfig)

  BuildRequires: /usr/sbin/ifconfig

  

- # For %%python_provide

- BuildRequires: python-rpm-macros

- 

  %if %{with rpmwheels}

  BuildRequires: python-setuptools-wheel

  BuildRequires: python-pip-wheel
@@ -206,6 +218,8 @@ 

  %if %{without bootstrap}

  # for make regen-all and distutils.tests.test_bdist_rpm

  BuildRequires: python%{pyshortver}

+ # for proper automatic provides

+ BuildRequires: python3-rpm-generators

  %endif

  

  # =======================
@@ -283,17 +297,33 @@ 

  # Descriptions, and metadata for subpackages

  # ==========================================

  

+ # this if branch is ~300 lines long and contains subpackages' definitions

+ %if %{without flatpackage}

+ %if %{with main_python}

+ # Description for the python3X SRPM only:

+ %description

+ Python %{pybasever} is an accessible, high-level, dynamically typed, interpreted

+ programming language, designed with an emphasis on code readability.

+ It includes an extensive standard library, and has a vast ecosystem of

+ third-party libraries.

+ 

+ %package -n %{pkgname}

+ Summary: Python %{pybasever} interpreter

+ 

  # People might want to dnf install pythonX.Y instead of pythonXY;

  # we enable this in both flat and nonflat package.

  Provides: python%{pybasever} = %{version}-%{release}

- 

- %if %{without flatpackage}

+ %else

+ # Provide python3X from python3

+ Provides: python%{pyshortver} = %{version}-%{release}

+ Provides: python%{pyshortver}%{?_isa} = %{version}-%{release}

+ %endif

  

  # Packages with Python modules in standard locations automatically

  # depend on python(abi). Provide that here.

  Provides: python(abi) = %{pybasever}

  

- Requires: %{name}-libs%{?_isa} = %{version}-%{release}

+ Requires: %{pkgname}-libs%{?_isa} = %{version}-%{release}

  

  # In order to support multiple Python interpreters for development purposes,

  # packages with the naming scheme flatpackage (e.g. python35) exist for
@@ -331,29 +361,27 @@ 

  # /usr/bin/python3*. Granularity per subpackage is impossible.

  # It's intended for the libs package not to drag in the interpreter, see

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

- # All others require %%{name} anyway.

+ # All others require %%{pkgname} anyway.

  %global __requires_exclude ^/usr/bin/python3

  

- 

- # The description used both for the SRPM and the main `python3` subpackage:

- %description

- Python is an accessible, high-level, dynamically typed, interpreted programming

- language, designed with an emphasis on code readability.

+ %description -n %{pkgname}

+ Python %{pybasever} is an accessible, high-level, dynamically typed, interpreted

+ programming language, designed with an emphasis on code readability.

  It includes an extensive standard library, and has a vast ecosystem of

  third-party libraries.

  

- The %{name} package provides the "python3" executable: the reference

+ The %{pkgname} package provides the "%{exename}" executable: the reference

  interpreter for the Python language, version 3.

- The majority of its standard library is provided in the %{name}-libs package,

- which should be installed automatically along with %{name}.

+ The majority of its standard library is provided in the %{pkgname}-libs package,

+ which should be installed automatically along with %{pkgname}.

  The remaining parts of the Python standard library are broken out into the

- %{name}-tkinter and %{name}-test packages, which may need to be installed

+ %{pkgname}-tkinter and %{pkgname}-test packages, which may need to be installed

  separately.

  

- Documentation for Python is provided in the %{name}-docs package.

+ Documentation for Python is provided in the %{pkgname}-docs package.

  

  Packages containing additional libraries for Python are generally named with

- the "%{name}-" prefix.

+ the "%{pkgname}-" prefix.

  

  

  %if %{with main_python}
@@ -375,7 +403,7 @@ 

  %endif # with main_python

  

  

- %package libs

+ %package -n %{pkgname}-libs

  Summary:        Python runtime libraries

  

  %if %{with rpmwheels}
@@ -386,18 +414,16 @@ 

  Provides: bundled(python3-setuptools) = 41.2.0

  %endif

  

- %{?python_provide:%python_provide python3-libs}

- 

  # There are files in the standard library that have python shebang.

  # We've filtered the automatic requirement out so libs are installable without

  # the main package. This however makes it pulled in by default.

  # See https://bugzilla.redhat.com/show_bug.cgi?id=1547131

- Recommends: %{name}%{?_isa} = %{version}-%{release}

+ Recommends: %{pkgname}%{?_isa} = %{version}-%{release}

  

  # tkinter is part of the standard library,

  # but it is torn out to save an unwanted dependency on tk and X11.

  # we recommend it when tk is already installed (for better UX)

- Recommends: (%{name}-tkinter%{?_isa} = %{version}-%{release} if tk%{?_isa})

+ Recommends: (%{pkgname}-tkinter%{?_isa} = %{version}-%{release} if tk%{?_isa})

  

  # https://fedoraproject.org/wiki/Changes/Move_usr_bin_python_into_separate_package

  # In Fedora 31, several "unversioned" files like /usr/bin/pydoc and all the
@@ -411,18 +437,17 @@ 

  # old Python 2 builds that still provided unversioned Python are handled.)

  

  

- %description libs

+ %description -n %{pkgname}-libs

  This package contains runtime libraries for use by Python:

  - the majority of the Python standard library

  - a dynamically linked library for use by applications that embed Python as

-   a scripting language, and by the main "python3" executable

+   a scripting language, and by the main "%{exename}" executable

  

  

- %package devel

+ %package -n %{pkgname}-devel

  Summary: Libraries and header files needed for Python development

- Requires: %{name} = %{version}-%{release}

- Requires: %{name}-libs%{?_isa} = %{version}-%{release}

- BuildRequires: python-rpm-macros

+ Requires: %{pkgname} = %{version}-%{release}

+ Requires: %{pkgname}-libs%{?_isa} = %{version}-%{release}

  # The RPM related dependencies bring nothing to a non-RPM Python developer

  # But we want them when packages BuildRequire python3-devel

  Requires: (python-rpm-macros if rpm-build)
@@ -441,18 +466,16 @@ 

  Requires: (python3-rpm-generators if rpm-build)

  %endif

  

- %{?python_provide:%python_provide python3-devel}

- 

- Provides: %{name}-2to3 = %{version}-%{release}

+ Provides: %{pkgname}-2to3 = %{version}-%{release}

  Provides: 2to3 = %{version}-%{release}

  

- Conflicts: %{name} < %{version}-%{release}

+ Conflicts: %{pkgname} < %{version}-%{release}

  

  # In Fedora 31, several "unversioned" files were moved here from Python 2:

  # pygettext.py, msgfmt.py, python-config, python.pc

  Conflicts: python-devel < 3

  

- %description devel

+ %description -n %{pkgname}-devel

  This package contains the header files and configuration needed to compile

  Python extension modules (typically written in C or C++), to embed Python

  into other programs, and to make binary distributions for Python libraries.
@@ -461,24 +484,22 @@ 

  and 2to3 tool, an automatic source converter from Python 2.X.

  

  

- %package idle

+ %package -n %{pkgname}-idle

  Summary: A basic graphical development environment for Python

- Requires: %{name} = %{version}-%{release}

- Requires: %{name}-tkinter = %{version}-%{release}

+ Requires: %{pkgname} = %{version}-%{release}

+ Requires: %{pkgname}-tkinter = %{version}-%{release}

  

  Provides: idle3 = %{version}-%{release}

  Provides: idle = %{version}-%{release}

  

- Provides: %{name}-tools = %{version}-%{release}

- Provides: %{name}-tools%{?_isa} = %{version}-%{release}

- Obsoletes: %{name}-tools < %{version}-%{release}

+ Provides: %{pkgname}-tools = %{version}-%{release}

+ Provides: %{pkgname}-tools%{?_isa} = %{version}-%{release}

+ Obsoletes: %{pkgname}-tools < %{version}-%{release}

  

  # In Fedora 31, /usr/bin/idle was moved here from Python 2.

  Conflicts: python-tools < 3

  

- %{?python_provide:%python_provide python3-idle}

- 

- %description idle

+ %description -n %{pkgname}-idle

  IDLE is Python’s Integrated Development and Learning Environment.

  

  IDLE has the following features: Python shell window (interactive
@@ -491,52 +512,46 @@ 

  configuration, browsers, and other dialogs.

  

  

- %package tkinter

+ %package -n %{pkgname}-tkinter

  Summary: A GUI toolkit for Python

- Requires: %{name} = %{version}-%{release}

- 

- %{?python_provide:%python_provide python3-tkinter}

+ Requires: %{pkgname} = %{version}-%{release}

  

- %description tkinter

+ %description -n %{pkgname}-tkinter

  The Tkinter (Tk interface) library is a graphical user interface toolkit for

  the Python programming language.

  

  

- %package test

+ %package -n %{pkgname}-test

  Summary: The self-test suite for the main python3 package

- Requires: %{name} = %{version}-%{release}

- Requires: %{name}-libs%{?_isa} = %{version}-%{release}

+ Requires: %{pkgname} = %{version}-%{release}

+ Requires: %{pkgname}-libs%{?_isa} = %{version}-%{release}

  

- %{?python_provide:%python_provide python3-test}

- 

- %description test

+ %description -n %{pkgname}-test

  The self-test suite for the Python interpreter.

  

  This is only useful to test Python itself. For testing general Python code,

- you should use the unittest module from %{name}-libs, or a library such as

- %{name}-pytest or %{name}-nose.

+ you should use the unittest module from %{pkgname}-libs, or a library such as

+ %{pkgname}-pytest.

  

  

  %if %{with debug_build}

- %package debug

+ %package -n %{pkgname}-debug

  Summary: Debug version of the Python runtime

  

  # The debug build is an all-in-one package version of the regular build, and

  # shares the same .py/.pyc files and directories as the regular build. Hence

  # we depend on all of the subpackages of the regular build:

- Requires: %{name}%{?_isa} = %{version}-%{release}

- Requires: %{name}-libs%{?_isa} = %{version}-%{release}

- Requires: %{name}-devel%{?_isa} = %{version}-%{release}

- Requires: %{name}-test%{?_isa} = %{version}-%{release}

- Requires: %{name}-tkinter%{?_isa} = %{version}-%{release}

- Requires: %{name}-idle%{?_isa} = %{version}-%{release}

+ Requires: %{pkgname}%{?_isa} = %{version}-%{release}

+ Requires: %{pkgname}-libs%{?_isa} = %{version}-%{release}

+ Requires: %{pkgname}-devel%{?_isa} = %{version}-%{release}

+ Requires: %{pkgname}-test%{?_isa} = %{version}-%{release}

+ Requires: %{pkgname}-tkinter%{?_isa} = %{version}-%{release}

+ Requires: %{pkgname}-idle%{?_isa} = %{version}-%{release}

  

  # In Fedora 31, /usr/bin/python-debug was moved here from Python 2.

  Conflicts: python-debug < 3

  

- %{?python_provide:%python_provide python3-debug}

- 

- %description debug

+ %description -n %{pkgname}-debug

  python3-debug provides a version of the Python runtime with numerous debugging

  features enabled, aimed at advanced Python users such as developers of Python

  extension modules.
@@ -560,6 +575,8 @@ 

  %global __requires_exclude ^python\\(abi\\) = 3\\..$

  %global __provides_exclude ^python\\(abi\\) = 3\\..$

  

+ Provides: python%{pybasever} = %{version}-%{release}

+ 

  %if %{with rpmwheels}

  Requires: python-setuptools-wheel

  Requires: python-pip-wheel
@@ -568,7 +585,7 @@ 

  Provides: bundled(python3-setuptools) = 41.2.0

  %endif

  

- # The description for the flat package

+ # The description for the flat package (SRPM and built)

  %description

  Python %{pybasever} package for developers.

  
@@ -1062,7 +1079,7 @@ 

  %endif # with tests

  

  

- %files

+ %files -n %{pkgname}

  %doc README.rst

  

  %if %{with main_python}
@@ -1086,7 +1103,7 @@ 

  %endif

  

  %if %{without flatpackage}

- %files libs

+ %files -n %{pkgname}-libs

  %doc README.rst

  %endif

  
@@ -1307,7 +1324,7 @@ 

  

  

  %if %{without flatpackage}

- %files devel

+ %files -n %{pkgname}-devel

  %endif

  

  %if %{with main_python}
@@ -1351,7 +1368,7 @@ 

  

  

  %if %{without flatpackage}

- %files idle

+ %files -n %{pkgname}-idle

  %endif

  

  %if %{with main_python}
@@ -1369,7 +1386,7 @@ 

  %endif

  

  %if %{without flatpackage}

- %files tkinter

+ %files -n %{pkgname}-tkinter

  %endif

  

  %{pylibdir}/tkinter
@@ -1387,7 +1404,7 @@ 

  

  

  %if %{without flatpackage}

- %files test

+ %files -n %{pkgname}-test

  %endif

  

  %{pylibdir}/ctypes/test
@@ -1413,7 +1430,7 @@ 

  

  %if %{with debug_build}

  %if %{without flatpackage}

- %files debug

+ %files -n %{pkgname}-debug

  %endif

  

  %if %{with main_python}

The --without flatpackage --without main_python build will currently fail due to %python_provide not recognizing python39- prefixes. That should be fixed in https://src.fedoraproject.org/rpms/python-rpm-macros/pull-request/48.

The --without flatpackage --with main_python build will currently not provide "python39-" due to %python_provide not doing that + we are not yet 3.9 in python-rpm-macros even with that PR linked from above.

The --with flatpackage --without main_python (the default) should be rather unchanged.

The --with flatpackage --with main_python doesn't really make much sense, but should be rather unchanged.

The build will fail in %check due to recent openssl changes. When testing, disable the tests.

Build failed.

Later, we can also make the main_python default to whether this is or isn't %__default_python3_version from https://src.fedoraproject.org/rpms/python-rpm-macros/pull-request/48 to only define the information in once place.

rebased onto 2b7b773de92b475a95163a0a72840f6eaaf08956

4 years ago

The --without flatpackage --without main_python build will currently fail due to %python_provide not recognizing python39- prefixes. That should be fixed in https://src.fedoraproject.org/rpms/python-rpm-macros/pull-request/48.

that was merged and will not be backported to non-rawhide. This means the --without flatpackage --without main_python build will always fail on Fedora 30/31/32. I can live with that. WDYT?

Build failed.

The diff is kinda complex and hard to read/verify but I haven't found anything suspicious. One minor change I'd do is to add a comment to the line 288 where "%if %{without flatpackage}" starts saying that this branch is almost 300 lines long and contains subpackages' definitions. The else branch of this condition is much smaller but the first one is hard to read and parse in brain :)

I've done a build for the first three combinations of flat package and main python. All three builds pass in mock even with tests enabled.

I've checked:
- build logs
- all RPM info (rpm -qi)
- installability of all RPMs in rawhide (co-existence with Python 3.8 or upgrade in the case on the main python (with enabled @python/python3.9 COPR))
- provides
- requires
- list of files

And for the combination we currently have in COPR, I've tested also rpmdiff.

Combination --with flatpackage and --without main_python

Contains a lot of warnings in build log like these

warning: File listed twice: /usr/bin/python3.9
warning: File listed twice: /usr/bin/python3.9
warning: File listed twice: /usr/bin/python3.9-config
warning: File listed twice: /usr/bin/python3.9-config
warning: File listed twice: /usr/include/python3.9/pyconfig-64.h
warning: File listed twice: /usr/include/python3.9/pyconfig-64.h
…

Other combinations also contain some of these warnings but not that many as this one.

Package python39-3.9.0~a5-1.fc33.x86_64.rpm requires /usr/bin/python3.9 is that ok?

Combination --without flatpackage and --with main_python

The build produces python39-debuginfo-3.9.0~a5-1.fc33.x86_64.rpm and python39-debugsource-3.9.0~a5-1.fc33.x86_64.rpm which does not happen in COPR — the same packages there have python3- prefix so it seems that it's correct because the name comes from the name of SRPM.

python3-debug-3.9.0~a5-1.fc33.x86_64.rpm provides python38-debug = 3.9.0~a5-1.fc33 and python38-debug(x86-64) = 3.9.0~a5-1.fc33 — the same problem exists for python3-[devel,idle,libs,test,tkinter]

Suspicious parts of rpmdiff between my builds and COPR

##### python3
S.5.....    SUMMARY
S.5.....    DESCRIPTION
S.5.......T /usr/bin/python3.9
##### debug
added       PROVIDES python-debug(x86-64) = 3.9.0~a5-1.fc33
added       PROVIDES python38-debug = 3.9.0~a5-1.fc33
added       PROVIDES python38-debug(x86-64) = 3.9.0~a5-1.fc33
S.......... /usr/lib64/libpython3.9d.so.1.0
S.......... /usr/lib64/python3.9/lib-dynload/_bz2.cpython-39d-x86_64-linux-gnu.so
S.......... /usr/lib64/python3.9/lib-dynload/_codecs_cn.cpython-39d-x86_64-linux-gnu.so
S.......... /usr/lib64/python3.9/lib-dynload/_contextvars.cpython-39d-x86_64-linux-gnu.so
S.......... /usr/lib64/python3.9/lib-dynload/_curses.cpython-39d-x86_64-linux-gnu.so
S.......... /usr/lib64/python3.9/lib-dynload/_datetime.cpython-39d-x86_64-linux-gnu.so
S.......... /usr/lib64/python3.9/lib-dynload/_decimal.cpython-39d-x86_64-linux-gnu.so
S.......... /usr/lib64/python3.9/lib-dynload/_sqlite3.cpython-39d-x86_64-linux-gnu.so
S.......... /usr/lib64/python3.9/lib-dynload/_testcapi.cpython-39d-x86_64-linux-gnu.so
S.......... /usr/lib64/python3.9/lib-dynload/_uuid.cpython-39d-x86_64-linux-gnu.so
S.......... /usr/lib64/python3.9/lib-dynload/_xxsubinterpreters.cpython-39d-x86_64-linux-gnu.so
S.......... /usr/lib64/python3.9/lib-dynload/fcntl.cpython-39d-x86_64-linux-gnu.so
S.......... /usr/lib64/python3.9/lib-dynload/pyexpat.cpython-39d-x86_64-linux-gnu.so
S.......... /usr/lib64/python3.9/lib-dynload/select.cpython-39d-x86_64-linux-gnu.so
S.......... /usr/lib64/python3.9/lib-dynload/termios.cpython-39d-x86_64-linux-gnu.so
##### devel
added       PROVIDES python-devel(x86-64) = 3.9.0~a5-1.fc33
added       PROVIDES python38-devel = 3.9.0~a5-1.fc33
added       PROVIDES python38-devel(x86-64) = 3.9.0~a5-1.fc33
##### idle
added       PROVIDES python-idle(x86-64) = 3.9.0~a5-1.fc33
added       PROVIDES python38-idle = 3.9.0~a5-1.fc33
added       PROVIDES python38-idle(x86-64) = 3.9.0~a5-1.fc33
.M......... /usr/share/metainfo/idle3.appdata.xml
##### libs
added       PROVIDES python38-libs = 3.9.0~a5-1.fc33
S.......... /usr/lib64/libpython3.9.so.1.0
S.......... /usr/lib64/python3.9/lib-dynload/_codecs_iso2022.cpython-39-x86_64-linux-gnu.so
S.......... /usr/lib64/python3.9/lib-dynload/_contextvars.cpython-39-x86_64-linux-gnu.so
S.......... /usr/lib64/python3.9/lib-dynload/_ctypes.cpython-39-x86_64-linux-gnu.so
S.......... /usr/lib64/python3.9/lib-dynload/_decimal.cpython-39-x86_64-linux-gnu.so
S.......... /usr/lib64/python3.9/lib-dynload/_elementtree.cpython-39-x86_64-linux-gnu.so
S.......... /usr/lib64/python3.9/lib-dynload/_heapq.cpython-39-x86_64-linux-gnu.so
S.......... /usr/lib64/python3.9/lib-dynload/_json.cpython-39-x86_64-linux-gnu.so
S.......... /usr/lib64/python3.9/lib-dynload/_pickle.cpython-39-x86_64-linux-gnu.so
S.......... /usr/lib64/python3.9/lib-dynload/_queue.cpython-39-x86_64-linux-gnu.so
S.......... /usr/lib64/python3.9/lib-dynload/_sqlite3.cpython-39-x86_64-linux-gnu.so
S.......... /usr/lib64/python3.9/lib-dynload/syslog.cpython-39-x86_64-linux-gnu.so
S.......... /usr/lib64/python3.9/lib-dynload/unicodedata.cpython-39-x86_64-linux-gnu.so
##### test
S.5.....    DESCRIPTION
added       PROVIDES python-test(x86-64) = 3.9.0~a5-1.fc33
added       PROVIDES python38-test = 3.9.0~a5-1.fc33
added       PROVIDES python38-test(x86-64) = 3.9.0~a5-1.fc33
##### tkinter
added       PROVIDES python-tkinter(x86-64) = 3.9.0~a5-1.fc33
added       PROVIDES python38-tkinter = 3.9.0~a5-1.fc33
added       PROVIDES python38-tkinter(x86-64) = 3.9.0~a5-1.fc33

Combination --without flatpackage and --without main_python

python39-3.9.0~a5-1.fc33.x86_64.rpm have in the description "The python39 package provides the "python3" executable".

It might be a good idea to unify the description for all combinations so we don't have to check it or change it every time we switch the package from one combination to another. It might also simplify the transition from Fedora to RHEL.

python39-libs-3.9.0~a5-1.fc33.x86_64.rpm provides python-libs = 3.9.0~a5-1.fc33 and python38-libs = 3.9.0~a5-1.fc33

Some of the findings above might not be a real problem but I mentioned all I have found so we can discuss them. The biggest issues IMHO are these python38- provides but I still don't know where these come from.

That's all I've found. I have all the results collected so feel free to ask for some more tests.

Thanks @lbalhar. This was done very thoroughly.

Here are some quick notes.

One minor change I'd do is to add a comment to the line 288...

Good idea.

File listed twice were already the case and we decided to keep it that way. I think it's OK. If we wan to get rid of those, we can do that later (but only if it doesn't make the spec even more complicated).

The debuginfo pkg names change is consistent with our proposal -- it is indeed based on the SRPM name and there is no need to preserve an upgrade path there.

python3-debug-3.9.0~a5-1.fc33.x86_64.rpm provides python38-debug = 3.9.0~a5-1.fc33 and python38-debug(x86-64) = 3.9.0~a5-1.fc33 — the same problem exists for python3-[devel,idle,libs,test,tkinter]

This is a problem indeed, it's based on the fact that the provides are being taken care of by an external macro (%python_provide) that does "oh, this is called 'python3', let's provide 'python38' as well, that is the current Python version". We can "fix" that by deriving the main_python bcond from the same source of data and put a huge warning next to it that turning it on without changing that source is going to do this. Alternatively, we can also undefine %python_provide when that happens.

The sizes differences in rpmdiff happen all the time, as gcc and other things are heavily updated in rawhide those days.

python39-3.9.0~a5-1.fc33.x86_64.rpm have in the description "The python39 package provides the "python3" executable".

Good catch.

It might be a good idea to unify the description for all combinations so we don't have to check it or change it every time we switch the package from one combination to another. It might also simplify the transition from Fedora to RHEL.

I wonder how we'll do that. But at least we can see what parts of the description are repeated and put them to a common macro. We can also have a section near the top with just the descriptions, one after another, so they are easier to spot and edit. Should we do that in a followup PR?

rebased onto 17ce962c3b7ab99aa24d4537c566b798a414fed7

4 years ago

Addressed some of your concerns in individual commits. I plan to rebase this later, but I want you to be able to see the differences.

Build succeeded.

The latest fixups looks good to me.

1 new commit added

  • Remove manual %python_provides calls
4 years ago

rebased onto 5be40cd3a261e5788ded86ce85c4f4eee91dee3e

4 years ago

Build succeeded.

rebased onto d799859

4 years ago

I'm shipping this. We can do followup fixups.

Pull-Request has been merged by churchyard

4 years ago

Build succeeded.