ubuntu-drivers-common now pulls in build tools on end-user systems

Bug #1880564 reported by Julian Andres Klode
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ubuntu-drivers-common (Ubuntu)
Fix Released
High
Alberto Milone
Focal
Fix Released
High
Alberto Milone

Bug Description

SRU Request:

[Test Case]

1) Uninstall the dpkg-dev package.

2) Enable the -proposed repository, and install the new ubuntu-drivers-common (0.8.4~0.20.04.1).

3) Check that the dpkg-dev package was not installed.

3) Restart your computer, and see if everything works correctly when
accessing the desktop.

[Regression Potential]
Low. The new get_system_architecture() function is very small, and replaces dpkg-architecture for our use-case.
_______
In bug 1875339, a depends on dpkg-dev was added. This is pulling in all of build-essential on upgrades (or just binutils and make and patch if you pass --no-install-recommends).

This needs to be reverted and fixed differently.

Revision history for this message
Julian Andres Klode (juliank) wrote :

This should be fixed before 20.04.1, so we don't install compilers on user system by default.

Changed in ubuntu-drivers-common (Ubuntu):
status: New → Triaged
importance: Undecided → High
tags: added: rls-ff-incoming
Changed in ubuntu-drivers-common (Ubuntu):
milestone: none → ubuntu-20.04.1
tags: added: rls-gg-incoming
Changed in ubuntu-drivers-common (Ubuntu):
assignee: nobody → Alberto Milone (albertomilone)
tags: removed: rls-ff-incoming rls-gg-incoming
Changed in ubuntu-drivers-common (Ubuntu Focal):
assignee: nobody → Alberto Milone (albertomilone)
importance: Undecided → High
status: New → Triaged
Changed in ubuntu-drivers-common (Ubuntu):
status: Triaged → In Progress
Revision history for this message
Steve Langasek (vorlon) wrote :

you may have already done this analysis, Alberto, but fwiw what I see is that dpkg-architecture is used at runtime to get the multiarch path - however, this is always called in the context of the native architecture, so this can instead be calculated at build-time and embedded in the binary since this is in an arch-dependent package. That would solve the problem of needing dpkg-dev at runtime.

Revision history for this message
Alberto Milone (albertomilone) wrote :

Steve: I already have a commit that gets rid of dpkg-architecture, and removes the dependency.

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

This bug was fixed in the package ubuntu-drivers-common - 1:0.8.4

---------------
ubuntu-drivers-common (1:0.8.4) groovy; urgency=medium

  * ubuntu-drivers:
    - Rework the argument parser.
  * debian/control:
    - Depend on python3-click.
    - Drop dependency on dpkg-dev (LP: #1880564).
  * Add support for the "Support" field in packages.
  * Add support for passing in driver and vendor for
    the "install" case (LP: #1881586).
  * gpu-manager.c:
    - Add get_system_architecture().

 -- Alberto Milone <email address hidden> Mon, 01 Jun 2020 12:08:39 +0200

Changed in ubuntu-drivers-common (Ubuntu):
status: In Progress → Fix Released
Revision history for this message
Robie Basak (racb) wrote : Proposed package upload rejected

An upload of ubuntu-drivers-common to focal-proposed has been rejected from the upload queue for the following reason: "Cannot review: missing SRU information. Please see https://wiki.ubuntu.com/StableReleaseUpdates#Procedure. Not all changes appear to have bug references either.".

description: updated
Revision history for this message
Robie Basak (racb) wrote :

An upload of ubuntu-drivers-common to focal-proposed has been rejected from the upload queue for the following reason: "SRU information is still missing for multiple changes claimed in your changelog entries. Regression discussion is still missing in the bugs in which you do have some SRU information. Please follow ALL of https://wiki.ubuntu.com/StableReleaseUpdates#Procedure, or if that is not possible, please explain in the relevant bugs why an exception is warranted.".

Revision history for this message
Robie Basak (racb) wrote :

For example: where is the SRU bug and documentation related to this change?

   * ubuntu-drivers:
     - Rework the argument parser.
     - Do not print an error when linux-modules cannot
       be found.
     - Add support for the new -server NVIDIA drivers.

Revision history for this message
Alberto Milone (albertomilone) wrote :

@Robie:

1) "Do not print an error when linux-modules cannot be found"
This prevents the ubuntu-drivers tools from being misleading, hoping that people won't file bug reports such as LP: #1882402

2) "Rework the argument parser." - This is a feature that was requested by the Foundations team. It makes the "install" behave consistently with or without "--gpgpu" parameter is used (i.e. you can pass in the driver that you need, as an option). For example "ubuntu-drivers install" did not accept any extra arguments, whereas "ubuntu-drivers install --gpgpu" (e.g. ubuntu-drivers install --gpgpu nvidia) did. Now both do.

I am fine having one more bug report about it, but I'd rather not have to follow many bug reports for the same SRU, if possible.

3) "Add support for the new -server NVIDIA drivers" - it's LP: #1881586

Revision history for this message
Łukasz Zemczak (sil2100) wrote : Please test proposed package

Hello Julian, or anyone else affected,

Accepted ubuntu-drivers-common into focal-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/ubuntu-drivers-common/1:0.8.4~0.20.04.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-needed-focal to verification-done-focal. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-focal. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Changed in ubuntu-drivers-common (Ubuntu Focal):
status: Triaged → Fix Committed
tags: added: verification-needed verification-needed-focal
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

When verifying this bug, can we make sure https://bugs.launchpad.net/oem-priority/+bug/1875339 is still fixed?

Revision history for this message
Łukasz Zemczak (sil2100) wrote :

Hello Julian, or anyone else affected,

Accepted ubuntu-drivers-common into focal-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/ubuntu-drivers-common/1:0.8.4~0.20.04.2 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-needed-focal to verification-done-focal. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-focal. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Revision history for this message
Alberto Milone (albertomilone) wrote :

1:0.8.4~0.20.04.2 solves the problem, and LP: #1875339 is definitely fixed too:

:~$ apt-cache policy ubuntu-drivers-common
ubuntu-drivers-common:
  Installed: 1:0.8.4~0.20.04.2
  Candidate: 1:0.8.4~0.20.04.2
  Version table:
 *** 1:0.8.4~0.20.04.2 400
        400 http://archive.ubuntu.com/ubuntu focal-proposed/main amd64 Packages
        100 /var/lib/dpkg/status
     1:0.8.1.1 500
        500 http://it.archive.ubuntu.com/ubuntu focal-updates/main amd64 Packages
     1:0.8.1 500
        500 http://it.archive.ubuntu.com/ubuntu focal/main amd64 Packages
     1:0.8.1~ppa3 500
        500 http://ppa.launchpad.net/albertomilone/u-d-c-release/ubuntu focal/main amd64 Packages

:~$ apt-cache policy dpkg-dev
dpkg-dev:
  Installed: (none)
  Candidate: 1.19.7ubuntu3
  Version table:
     1.19.7ubuntu3 500
        500 http://it.archive.ubuntu.com/ubuntu focal/main amd64 Packages
        500 http://it.archive.ubuntu.com/ubuntu focal/main i386 Packages
        100 /var/lib/dpkg/status

:~$ prime-select query
nvidia

:~$ nvidia-smi
Mon Jul 27 18:16:39 2020
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 440.95.01 Driver Version: 440.95.01 CUDA Version: 10.2 |
|-------------------------------+----------------------+----------------------+
| GPU Name Persistence-M| Bus-Id Disp.A | Volatile Uncorr. ECC |
| Fan Temp Perf Pwr:Usage/Cap| Memory-Usage | GPU-Util Compute M. |
|===============================+======================+======================|
| 0 GeForce RTX 2060 Off | 00000000:01:00.0 Off | N/A |
| N/A 44C P8 3W / N/A | 452MiB / 5934MiB | 0% Default |
+-------------------------------+----------------------+----------------------+

+-----------------------------------------------------------------------------+
| Processes: GPU Memory |
| GPU PID Type Process name Usage |
|=============================================================================|
| 0 1107 G /usr/lib/xorg/Xorg 134MiB |
| 0 1656 G /usr/lib/xorg/Xorg 135MiB |
| 0 1906 G /usr/bin/gnome-shell 165MiB |
+-----------------------------------------------------------------------------+

tags: added: verification-done verification-done-focal
removed: verification-needed verification-needed-focal
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-drivers-common - 1:0.8.4~0.20.04.2

---------------
ubuntu-drivers-common (1:0.8.4~0.20.04.2) focal; urgency=medium

  * gpu-manager.c:
    - Use the full multiarch path for the X11 libraries.

ubuntu-drivers-common (1:0.8.4~0.20.04.1) focal; urgency=medium

  * UbuntuDrivers/detect.py,
    detect-plugins/sl-modem.py:
    - Avoid triggering test_static.py
      with "E741 ambiguous variable name 'l'" error.
  * ubuntu-drivers:
    - Rework the argument parser to allow passing in
      driver and vendor for the "install" case
      (LP: #1881586).
    - Do not print an error when linux-modules cannot
      be found. This prevents the ubuntu-drivers tools
      from printing misleading warnings, as seen in
      LP:#1882402.
    - Add support for the new -server NVIDIA drivers
      (LP: #1881586).
  * debian/control:
    - Depend on python3-click (for the argument
      parsing code).
    - Drop dependency on dpkg-dev (LP: #1880564).
  * gpu-manager.c:
    - Add get_system_architecture() (to drop the
      dpkg-dev dependency).

 -- Alberto Milone <email address hidden> Mon, 27 Jul 2020 16:10:25 +0200

Changed in ubuntu-drivers-common (Ubuntu Focal):
status: Fix Committed → Fix Released
Revision history for this message
Łukasz Zemczak (sil2100) wrote : Update Released

The verification of the Stable Release Update for ubuntu-drivers-common has completed successfully and the package is now being released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

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.