Bug #15391

Stop shipping python-qt4 that is no longer required

Added by intrigeri 2018-03-09 08:33:34 . Updated 2018-05-10 10:59:51 .

Status:
Resolved
Priority:
Elevated
Assignee:
Category:
Target version:
Tails_3.7
Start date:
2018-03-09
Due date:
% Done:

100%

Feature Branch:
bugfix/15391-stop-installing-useless-python-packages
Type of work:
Code
Blueprint:

Starter:
Affected tool:
Electrum
Deliverable for:

Description

Potential regression since Feature #15022: config/chroot_local-packageslists/tails-common.list says we install python-qt4 to Enable Electrum's Qt GUI. I believe it’s wrong since electrum 3.0.6-1~bpo9+1 Recommends: python3-pyqt5 and has no dependency on Qt4.

We should check if we still need this package. If it’s pulled by something else that needs it anyway, let’s remove it from the list. If it’s not pulled automatically but we need it, let’s update the comment. If we don’t need it, let’s remove it from the list.

Subtasks

Related issues

Related to Tails - Feature #15022: Electrum package update to version 3.0.6 Resolved 2017-12-07
Blocks Tails - Feature #15139: Core work 2018Q2: Foundations Team Resolved 2018-01-01

History

#1 Updated by intrigeri 2018-03-09 08:34:44

  • related to Feature #15022: Electrum package update to version 3.0.6 added

#2 Updated by intrigeri 2018-03-27 12:11:59

  • Subject changed from Check what to do with python-qt4 to Stop shipping python-qt4 that is no longer required
  • Type of work changed from Research to Code

FYI on Tails 3.6.1 nothing depends on that package and removing it frees 14.0 MB disk space (uncompressed). Deinstalling its no longer required dependencies frees another 24.5 MB disk space (uncompressed). And then Electrum still starts.

#3 Updated by intrigeri 2018-04-13 11:34:17

#4 Updated by intrigeri 2018-04-13 11:34:27

  • Assignee changed from anonym to intrigeri

#5 Updated by intrigeri 2018-04-13 16:21:00

  • Status changed from Confirmed to In Progress
  • % Done changed from 0 to 10
  • Feature Branch set to bugfix/15391-stop-installing-useless-python-packages

#6 Updated by intrigeri 2018-04-14 06:33:01

  • Assignee changed from intrigeri to anonym
  • % Done changed from 10 to 50
  • QA Check set to Ready for QA

FWIW this saves 8MB on the resulting ISO size. I’ve run the full test suite and only failures are:

  • “features/additional_software_packages.feature:8 # Scenario: Additional software packages are installed even without network”: never seen it pass locally
  • “features/torified_gnupg.feature:31 # Scenario: Syncing OpenPGP keys using Seahorse should work and be done over Tor.”: I’ve heard on XMPP about the keyserver having issues
  • “features/torified_gnupg.feature:43 # Scenario: Syncing OpenPGP keys using Seahorse started from the OpenPGP Applet should work and be done over Tor.”: same as above

#7 Updated by bertagaz 2018-05-06 11:56:09

  • Assignee changed from anonym to bertagaz

Taking this over, for it to get in 3.7 in time.

#8 Updated by bertagaz 2018-05-07 09:55:05

  • Status changed from In Progress to Fix committed
  • Assignee deleted (bertagaz)
  • % Done changed from 50 to 100
  • QA Check changed from Ready for QA to Pass

bertagaz wrote:
> Taking this over, for it to get in 3.7 in time.

Full test suite run passes so it’s merged.

#9 Updated by bertagaz 2018-05-10 10:59:51

  • Status changed from Fix committed to Resolved