Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

move to python2 package naming #1258

Closed
totaam opened this issue Jul 16, 2016 · 15 comments
Closed

move to python2 package naming #1258

totaam opened this issue Jul 16, 2016 · 15 comments

Comments

@totaam
Copy link
Collaborator

totaam commented Jul 16, 2016

Issue migrated from trac ticket # 1258

component: packaging | priority: major | resolution: fixed | keywords: rpm python

2016-07-16 22:16:42: antoine created the issue


The Python 2.x packages for Fedora are now called python2-XXX instead of just python-XXX as before.

Doing this will prevent update conflicts.
It should just be a matter of renaming the package and adding Obsoletes and Provides to the spec file. This should be backwards compatible for centos 6.x
We can reference the new package names in the main xpra spec file.
Maybe this is a good time to merge the pyopengl and accelerate package to match fedora packaging.

Somewhat related to #1253.

See:

@totaam
Copy link
Collaborator Author

totaam commented Jul 16, 2016

2016-07-16 22:17:57: antoine changed status from new to assigned

@totaam
Copy link
Collaborator Author

totaam commented Jul 16, 2016

2016-07-16 22:17:57: antoine commented


Done for rencode in r13031, will use this for testing.

@totaam
Copy link
Collaborator Author

totaam commented Jul 17, 2016

2016-07-17 17:42:57: antoine edited the issue description

@totaam
Copy link
Collaborator Author

totaam commented Jul 17, 2016

2016-07-17 17:42:57: antoine commented


Done in r13032 for:

  • netifaces
  • pillow
  • pycuda
  • pytools
  • srp

Still todo: pyopengl merge.

@totaam
Copy link
Collaborator Author

totaam commented Jul 18, 2016

2016-07-18 12:59:49: antoine changed status from assigned to new

@totaam
Copy link
Collaborator Author

totaam commented Jul 18, 2016

2016-07-18 12:59:49: antoine changed owner from antoine to smo

@totaam
Copy link
Collaborator Author

totaam commented Jul 18, 2016

2016-07-18 12:59:49: antoine commented


Onto pyopengl: the Fedora python-pyopengl package (now named python2-pyopengl for python2 in Fedora as per PyOpenGL renamed to python-pyopengl) includes both PyOpenGL and PyOpenGL_accelerate in a single RPM but without conflicting with the PyOpenGL-accelerate packages we had already published, causing all sorts of problems including distro version upgrades failing, and that's bad!

Fixing this pyopengl packaging mess (done in r13033 + r13034) was hard because I don't want to have to maintain multiple versions of the spec file and the rpm version in centos6 is severely limited compared to the one in Fedora.
The whole point of our pyopengl package on Fedora is only to add the Obsoletes: PyOpenGL-accelerate... nothing else. PITA.
Eventually we could drop it altogether, but there doesn't seem to by an new pyopengl release on the horizon, so this could take a while.

r13035 fixes the same problem with python-lz4: Fedora decided to package it as python2-lz4 but without an Obsoletes: python-lz4 clause, and so we can't depend on this new name because this would bring the Fedora package, which then conflicts with the already installed xpra repository provided package python-lz4.. So we'll have to push the updated package first. PITA.

r13038 also updates the xpra spec file to use the new "python2" package names where applicable.

@totaam
Copy link
Collaborator Author

totaam commented Jul 28, 2016

2016-07-28 09:23:05: antoine commented


FWIW: a clean Fedora 24 install worked fine as of r13105 (missing conf file) + r13104 (remove rpmfusion dependency). r13110 makes sure that we override the Fedora version because 4>2-1, obviously!

@totaam
Copy link
Collaborator Author

totaam commented Aug 1, 2016

2016-08-01 10:27:13: antoine commented


Needed a few more tweaks (and a few attempts) for centos6: r13145 + r13146 + r13147 + r13148.

Now we can update from the beta repo and get:

=========================================================================================================================####  Package                       Arch               Version                                   Repository                  Size=========================================================================================================================
Installing:
 python-pyopengl               x86_64             3.1.1a1-4.1xpra2.el6_8                    winswitch-beta             2.4 M
     replacing  PyOpenGL.noarch 3.1.1a1r1-1.el6_8
     replacing  PyOpenGL-accelerate.x86_64 3.1.1a1r1-1.el6_8
 python2-lz4                   x86_64             0.8.2-2.el6_8                             winswitch-beta             9.2 k
     replacing  python-lz4.x86_64 0.8.2-1.el6_8
 python2-netifaces             x86_64             0.10.4-6.el6_8                            winswitch-beta              16 k
     replacing  python-netifaces.x86_64 0.5-1.el6
 python2-pillow                x86_64             3.3.0-2.el6_8                             winswitch-beta             579 k
     replacing  python-pillow.x86_64 3.2.0-2.el6_8
Updating:
 libvpx-xpra                   x86_64             1.6.0-1.el6_8                             winswitch-beta             697 k
 xpra                          x86_64             1.0-0.20160728r13114.el6_8                winswitch-beta             2.8 M
 xpra-common                   noarch             1.0-0.20160728r13114.el6_8                winswitch-beta             557 k

@totaam
Copy link
Collaborator Author

totaam commented Aug 5, 2016

2016-08-05 15:48:20: antoine commented


Fedora 23 fixes: r13228, r13229.

@totaam
Copy link
Collaborator Author

totaam commented Oct 27, 2016

2016-10-27 22:28:22: smo changed status from new to closed

@totaam
Copy link
Collaborator Author

totaam commented Oct 27, 2016

2016-10-27 22:28:22: smo set resolution to fixed

@totaam
Copy link
Collaborator Author

totaam commented Oct 27, 2016

2016-10-27 22:28:22: smo commented


No issues reported

@totaam totaam closed this as completed Oct 27, 2016
@totaam
Copy link
Collaborator Author

totaam commented Nov 24, 2016

2016-11-24 07:32:27: antoine commented


Added Cython in r14483, needed by Fedora 25.

@totaam
Copy link
Collaborator Author

totaam commented Jul 4, 2018

2018-07-04 07:24:55: antoine commented


See Python RPM Porting Guide : New Python package naming scheme.

This caused some problems: #1897

This was referenced Jan 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant