please use Python3 for all scripts

Bug #1440564 reported by Matthias Klose
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
xdiagnose (Ubuntu)
Fix Released
Undecided
Didier Roche-Tolomelli

Bug Description

There is still the goal to remove Python2 from the desktop images, and to use Python3 only. Please convert the remaining scripts and drop the dependency on python.

Tags: patch python3
Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

Here is the python3 version. Note, we still build with python2 distutils auto debhelper integration which wasn't ported to python3, but this one isn't installed on the runtime.

Changed in xdiagnose (Ubuntu):
status: New → Fix Committed
assignee: nobody → Didier Roche (didrocks)
tags: added: patch
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package xdiagnose - 3.8

---------------
xdiagnose (3.8) wily; urgency=medium

  [ Didier Roche ]
  * Port and ensure scripts are running with python3 (LP: #1440564)

 -- Steve Langasek <email address hidden> Wed, 10 Jun 2015 13:47:00 -0700

Changed in xdiagnose (Ubuntu):
status: Fix Committed → Fix Released
Revision history for this message
Barry Warsaw (barry) wrote :

As it turns out, xdiagnose was still Depending on Python 2. I completed the port by switching to --buildsystem=pybuild and dropping all Python 2-isms. I have a diff, attached here, that completes the port to Python 3 (hence why I re-opened this bug). Before I upload it though, perhaps Didier could give it a sanity check?

Changed in xdiagnose (Ubuntu):
status: Fix Released → New
Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

LGTM, thanks Barry!
+1

Revision history for this message
Barry Warsaw (barry) wrote :

Thanks! Uploaded. BTW, lp:xdiagnose seemed terribly out of date so I didn't modify it. I'm not sure if there's a current upstream branch for xdiagnose.

Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

same, it was already out of date, I didn't dare touching it ;)

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package xdiagnose - 3.8.3

---------------
xdiagnose (3.8.3) xenial; urgency=medium

  * Complete the port to Python 3-only. (LP: #1440564)
  * d/control: Update Build-Depends to use pybuild and drop Python 2.
  * d/rules: Switch to --buildsystem pybuild and simplify rules.
  * d/compat: Bump to version 9.

 -- Barry Warsaw <email address hidden> Fri, 19 Feb 2016 08:41:53 -0500

Changed in xdiagnose (Ubuntu):
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.