DEB pytqt: drop python2 support. #176

Merged
MicheleC merged 1 commits from deb/python-tqt/drop-python2 into master 1 year ago
Owner

Requires PR #175.

Local test of examples with python 3 works fine.

Requires PR #175. Local test of examples with python 3 works fine.
MicheleC added a new dependency 1 year ago
MicheleC added this to the R14.1.0 release milestone 1 year ago
MicheleC requested review from SlavekB 1 year ago
Poster
Owner

One question is whether to rename the packages python-tqt-{dev,doc} into python3-tqt-{dev,doc} (with the 3 in the name like other packages).

One question is whether to rename the packages `python-tqt-{dev,doc}` into `python3-tqt-{dev,doc}` (with the 3 in the name like other packages).
Poster
Owner

Like for sip4-tqt, the second commit suggests names that should make the migration easier from both R14.0.x and R14.1.0-dev.

Commits to be squashed before merging.

Like for sip4-tqt, the second commit suggests names that should make the migration easier from both R14.0.x and R14.1.0-dev. Commits to be squashed before merging.
MicheleC force-pushed deb/python-tqt/drop-python2 from 411d3cddf9 to 7e6633b297 1 year ago
Poster
Owner

PR further updated following changes on sip4-tqt PR

PR further updated following changes on sip4-tqt PR
SlavekB requested changes 1 year ago
SlavekB left a comment
Owner

There are some small things to solve – see comments.

There are some small things to solve – see comments.
XSBC-Original-Maintainer: Debian Python Modules Team <python-modules-team@lists.alioth.debian.org>
Uploaders: Torsten Marek <shlomme@debian.org>, Bernd Zeimetz <bzed@debian.org>, Michael Casadevall <mcasadevall@debian.org>
Build-Depends: debhelper (>= 9~), python-all-dev (>= 2.3.5-10), python-all-dbg, python3-all-dev, python3-all-dbg, libtqtinterface-dev, tqt3-dev-tools, tqt3-apps-dev, python3-sip-tqt-dev, python3-sip-tqt-dbg, python-sip-tqt-dev, python-sip-tqt-dbg, libtqscintilla-dev, python-support (>= 0.7.1) | dh-python
Build-Depends: debhelper (>= 9~), python3-all-dev, python3-all-dbg, libtqtinterface-dev, tqt3-dev-tools, tqt3-apps-dev, python3-sip-tqt-dev, sip-tqt-dev, python3-sip-tqt-dbg, libtqscintilla-dev, dh-python
SlavekB commented 1 year ago
Owner

sip-tqt-dev does not need to be listed as it is a required dependency for python3-sip-tqt-dev.

`sip-tqt-dev` does not need to be listed as it is a required dependency for `python3-sip-tqt-dev`.
Poster
Owner

Ok, makes sense.

Ok, makes sense.
MicheleC marked this conversation as resolved
Package: python3-tqt
Architecture: any
Depends: ${shlibs:Depends}, ${python3:Depends}, ${sip3-tqt:Depends}, ${misc:Depends}
SlavekB commented 1 year ago
Owner

${py3sip-tqt:Depends} should be used here instead of ${sip3-tqt:Depends} to ensure a dependency on the corresponding API version of SIP-TQt – see change of dh_py3sip_tqt in #175.

`${py3sip-tqt:Depends}` should be used here instead of `${sip3-tqt:Depends}` to ensure a dependency on the corresponding API version of SIP-TQt – see change of `dh_py3sip_tqt` in #175.
Poster
Owner

Good catch, I missed that.

Good catch, I missed that.
MicheleC marked this conversation as resolved
Linguist translation files from Python code.
Package: python-tqt-dev
SlavekB commented 1 year ago
Owner

The question arises whether to keep the names python-tqt-dev and python-tqt-doc, which may give the impression of a dependency on Python 2, or whether to shorten the package names to pytqt-dev and pytqt-doc when we consider using PyTQt as the source name?

The question arises whether to keep the names `python-tqt-dev` and `python-tqt-doc`, which may give the impression of a dependency on Python 2, or whether to shorten the package names to `pytqt-dev` and `pytqt-doc` when we consider using PyTQt as the source name?
Poster
Owner

I suggest we do this when we actually rename the repository. What do you think?

I suggest we do this when we actually rename the repository. What do you think?
SlavekB commented 1 year ago
Owner

Yes, we can consider renaming the source package and also git repository. In any case, you may notice that the pytqt-tools binary package is already being built in this package, so it's not the first binary package with a name in this format.

Yes, we can consider renaming the source package and also git repository. In any case, you may notice that the `pytqt-tools` binary package is already being built in this package, so it's not the first binary package with a name in this format.
Poster
Owner

How about renaming the packages as follow?

  1. python3-tqt* to python3-pytqt*
  2. python-tqt-{doc,dev} to pytqt-{doc,dev}
How about renaming the packages as follow? 1. python3-tqt* to python3-pytqt* 2. python-tqt-{doc,dev} to pytqt-{doc,dev}
SlavekB commented 1 year ago
Owner

Yes, that makes sense. At the same time, however, we should probably go one step further and rename the python module as such from python_tqt to PyTQt.

Yes, that makes sense. At the same time, however, we should probably go one step further and rename the python module as such from `python_tqt` to `PyTQt`.
Poster
Owner

Yes, good idea.

Yes, good idea.
Poster
Owner

package names updated in 2nd commit. When all good, we can squash them.

package names updated in 2nd commit. When all good, we can squash them.
SlavekB marked this conversation as resolved
MicheleC force-pushed deb/python-tqt/drop-python2 from 7e6633b297 to dcc725f4de 1 year ago
Poster
Owner

PR updated as per feedback

PR updated as per feedback
SlavekB reviewed 1 year ago
XSBC-Original-Maintainer: Debian Python Modules Team <python-modules-team@lists.alioth.debian.org>
Uploaders: Torsten Marek <shlomme@debian.org>, Bernd Zeimetz <bzed@debian.org>, Michael Casadevall <mcasadevall@debian.org>
Build-Depends: debhelper (>= 9~), python-all-dev (>= 2.3.5-10), python-all-dbg [!solaris-any], python3-all-dev, python3-all-dbg [!solaris-any], libtqtinterface-dev, tqt3-dev-tools, tqt3-apps-dev, python3-sip-tqt-dev, python3-sip-tqt-dbg, python-sip-tqt-dev, python-sip-tqt-dbg [!solaris-any], libtqscintilla-dev, python-support (>= 0.7.1) | dh-python
Build-Depends: debhelper (>= 9~), python3-all-dev, python3-all-dbg [!solaris-any], libtqtinterface-dev, tqt3-dev-tools, tqt3-apps-dev, python3-sip-tqt-dev, python3-sip-tqt-dbg, libtqscintilla-dev, dh-python
SlavekB commented 1 year ago
Owner

Note: For -dbg packages in DilOS is used [!solaris-any]. So it should be used for the package python3-sip-tqt-dbg as well.

Note: For `-dbg` packages in DilOS is used `[!solaris-any]`. So it should be used for the package `python3-sip-tqt-dbg` as well.
Poster
Owner

Ah, my bad. Somehow I had overwritten that unintentionally. Fixed.

Ah, my bad. Somehow I had overwritten that unintentionally. Fixed.
Poster
Owner

Actually I checked the code in master and the [!solaris-any] was missing from python3-sip-tqt-dbg in first place :-)

Actually I checked the code in master and the `[!solaris-any]` was missing from `python3-sip-tqt-dbg` in first place :-)
SlavekB commented 1 year ago
Owner

thanks for correcting my earlier mistake :)

thanks for correcting my earlier mistake :)
MicheleC marked this conversation as resolved
MicheleC force-pushed deb/python-tqt/drop-python2 from dcc725f4de to dedcd09059 1 year ago
MicheleC requested review from SlavekB 1 year ago
SlavekB force-pushed deb/python-tqt/drop-python2 from dedcd09059 to de3998cda4 1 year ago
MicheleC force-pushed deb/python-tqt/drop-python2 from de3998cda4 to 0edfaf2f43 1 year ago
SlavekB force-pushed deb/python-tqt/drop-python2 from 0edfaf2f43 to e083d551c1 1 year ago
SlavekB commented 1 year ago
Owner

Rebased to the current head. The second (temporary) commit is no longer needed because the renaming was done in the previous one.

Rebased to the current head. The second (temporary) commit is no longer needed because the renaming was done in the previous one.
Poster
Owner

Nice, thanks for updating this. I was planning to do it myself but nice to find the job done already ;-)

Nice, thanks for updating this. I was planning to do it myself but nice to find the job done already ;-)
MicheleC force-pushed deb/python-tqt/drop-python2 from e083d551c1 to 6b383d6222 1 year ago
Poster
Owner

There was one more change required in debian/rules which was causing FTBFS. Commit updated.

There was one more change required in debian/rules which was causing FTBFS. Commit updated.
MicheleC force-pushed deb/python-tqt/drop-python2 from 6b383d6222 to 3b5942e7b3 1 year ago
SlavekB approved these changes 1 year ago
SlavekB left a comment
Owner

Everything looks good.

Everything looks good.
MicheleC changed title from DEB python-tqt: drop python2 support. to WIP: (READY) DEB python-tqt: drop python2 support. 1 year ago
MicheleC force-pushed deb/python-tqt/drop-python2 from 3b5942e7b3 to 61555e4f9e 1 year ago
MicheleC force-pushed deb/python-tqt/drop-python2 from 61555e4f9e to f72276dd66 1 year ago
MicheleC force-pushed deb/python-tqt/drop-python2 from f72276dd66 to f963f7aac5 1 year ago
MicheleC changed title from WIP: (READY) DEB python-tqt: drop python2 support. to DEB python-tqt: drop python2 support. 1 year ago
MicheleC changed title from DEB python-tqt: drop python2 support. to WIP: DEB python-tqt: drop python2 support. 1 year ago
MicheleC changed title from WIP: DEB python-tqt: drop python2 support. to DEB python-tqt: drop python2 support. 1 year ago
MicheleC merged commit f963f7aac5 into master 1 year ago
MicheleC deleted branch deb/python-tqt/drop-python2 1 year ago
MicheleC changed title from DEB python-tqt: drop python2 support. to DEB pytqt: drop python2 support. 1 year ago

Reviewers

SlavekB approved these changes 1 year ago
The pull request has been merged as f963f7aac5.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Depends on
Reference: TDE/tde-packaging#176
Loading…
There is no content yet.