u-boot-tools missing binutils dependency

Bug #1814930 reported by Dave Jones
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
u-boot (Ubuntu)
Fix Released
Medium
Unassigned
Bionic
Fix Released
Low
Unassigned
Cosmic
Fix Released
Low
Unassigned

Bug Description

[Impact]

The u-boot-tools package, specifically the mkknlimg script within it, relies upon several external utilities (specifically perl, grep, tr, and strings). Whilst most are provided by "essential" packages, strings is provided by binutils which is not. On a fresh install of the RPi3 beta image, an upgrade involved u-boot succeeds, but nonetheless prints a warning that strings cannot be found.

All package upgrades from the beta state should upgrade without any errors or warnings.

[Test Case]

On a pi3 image (or a chroot without binutils installed) with the old u-boot-tools version, run mkknlimg:

`touch a; mkknlimg --dtok a b`

Confirm that the command succeeds but prints out: "sh: 1: strings: not found". Upgrade the package and re-run the same command:

`mkknlimg --dtok a b`

Confirm that no "not found" message is visible anymore. Check that binutils has been installed during the update process.

[Regression Potential]

None. This only adds a new dependency. The dependency is in main and seeded in almost all daily and daily-live images.

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

+1 on that, let me sponsor it with a few tweaks to the changelog.

I guess we should try getting this change into Debian? I'm pretty sure they might be having the same problem. They might have binutils installed everywhere but since it's not essential there as well, I suppose they should have the dep explicitly stated.

I'll also backport it to bionic once it migrates.

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

Sponsored for disco, will sponsor it for bionic (and possibly cosmic?) in a bit.

Changed in u-boot (Ubuntu):
status: New → Fix Committed
description: updated
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

Sponsored for bionic and cosmic (waiting in the unapproved queue).

Changed in u-boot (Ubuntu):
importance: Undecided → Medium
Changed in u-boot (Ubuntu Bionic):
importance: Undecided → Low
Changed in u-boot (Ubuntu Cosmic):
importance: Undecided → Low
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package u-boot - 2018.07~rc3+dfsg1-0ubuntu2

---------------
u-boot (2018.07~rc3+dfsg1-0ubuntu2) disco; urgency=medium

  * d/control: Add missing dependency on binutils (for strings) (LP: #1814930)

 -- Dave Jones <email address hidden> Wed, 06 Feb 2019 16:40:22 +0000

Changed in u-boot (Ubuntu):
status: Fix Committed → Fix Released
Revision history for this message
Steve Langasek (vorlon) wrote : Please test proposed package

Hello Dave, or anyone else affected,

Accepted u-boot into cosmic-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/u-boot/2018.07~rc3+dfsg1-0ubuntu2~18.10.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 and change the tag from verification-needed-cosmic to verification-done-cosmic. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-cosmic. 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 u-boot (Ubuntu Cosmic):
status: New → Fix Committed
tags: added: verification-needed verification-needed-cosmic
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

Any reason the bionic upload has not been reviewed/accepted along with the bionic one? It's much more important there.

Revision history for this message
Brian Murray (brian-murray) wrote :

Hello Dave, or anyone else affected,

Accepted u-boot into bionic-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/u-boot/2018.07~rc3+dfsg1-0ubuntu2~18.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 and change the tag from verification-needed-bionic to verification-done-bionic. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-bionic. 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 u-boot (Ubuntu Bionic):
status: New → Fix Committed
tags: added: verification-needed-bionic
Revision history for this message
Dave Jones (waveform) wrote :

Verification done for bionic on a Pi 3B with both arm64 and armhf images; no issues encountered.

tags: added: verification-done-bionic
removed: verification-needed-bionic
Revision history for this message
Łukasz Zemczak (sil2100) wrote :

For cosmic I have performed the same validation of u-boot-tools 2018.07~rc3+dfsg1-0ubuntu2~18.10.1 on my very old alpha cosmic pi3 image. Looked good.

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

This bug was fixed in the package u-boot - 2018.07~rc3+dfsg1-0ubuntu2~18.10.1

---------------
u-boot (2018.07~rc3+dfsg1-0ubuntu2~18.10.1) cosmic; urgency=medium

  * d/control: Add missing dependency on binutils (for strings) (LP: #1814930)

 -- Dave Jones <email address hidden> Wed, 06 Feb 2019 16:40:22 +0000

Changed in u-boot (Ubuntu Cosmic):
status: Fix Committed → Fix Released
Revision history for this message
Brian Murray (brian-murray) wrote : Update Released

The verification of the Stable Release Update for u-boot has completed successfully and the package has now been 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.

Revision history for this message
Dave Jones (waveform) wrote :

Bionic verification done with u-boot-tools 2018.07~rc3+dfsg1-0ubuntu2~18.04.1

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

This bug was fixed in the package u-boot - 2018.07~rc3+dfsg1-0ubuntu2~18.04.1

---------------
u-boot (2018.07~rc3+dfsg1-0ubuntu2~18.04.1) bionic; urgency=medium

  * d/control: Add missing dependency on binutils (for strings) (LP: #1814930)

 -- Dave Jones <email address hidden> Wed, 06 Feb 2019 16:40:22 +0000

Changed in u-boot (Ubuntu Bionic):
status: Fix Committed → 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.