trusty/utopic pbuilder targets saucy by default

Bug #1316423 reported by Sergio Gelato
30
This bug affects 4 people
Affects Status Importance Assigned to Milestone
pbuilder (Ubuntu)
Undecided
Unassigned
Trusty
Undecided
B. Clausius

Bug Description

pbuilder version 0.215ubuntu7, currently in both trusty and utopic, still has
DISTRIBUTION=saucy
in its /usr/share/pbuilder/pbuilderrc . While easy to work around, this is inelegant.

Maybe the default should be something like $(lsb_release -s -c) ? I'm not sure it's safe to evaluate it at run time, but package build time should be OK.

SRU for 14.04
[Impact]
Developers setting up pbuilder on Ubuntu 14.04 will, unless specifying otherwise, get an older EOL-ed release selected as the default distribution.

[Test Case]
0. Verify ~/.pbuilderrc doesn't override the defaults by setting a value for DISTRIBUTION
1. Run `sudo pbuilder create`
The output should mention "Distribution is trusty.", not "Distribution is saucy."

[Regression potential]
I'm not familiar with this package, but I would assume very little. It only updates the default of one property to a valid distribution. It is presently fully possible to set the same value manually without any problems.

Related branches

Revision history for this message
Sergio Gelato (sergio-gelato) wrote :

… OK for Ubuntu; Debian would likely end up with DISTRIBUTION=sid no matter what, which is also inappropriate for testing/stable/oldstable.

Revision history for this message
B. Clausius (barcc) wrote :

Debdiff for trusty

Revision history for this message
B. Clausius (barcc) wrote :

Debdiff for utopic

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

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in pbuilder (Ubuntu):
status: New → Confirmed
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "trusty_pbuilder_targets_saucy_by_default.debdiff" seems to be a debdiff. The ubuntu-sponsors team has been subscribed to the bug report so that they can review and hopefully sponsor the debdiff. If the attachment isn't a patch, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are member of the ~ubuntu-sponsors, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issue please contact him.]

tags: added: patch
Martin Pitt (pitti)
Changed in pbuilder (Ubuntu):
status: Confirmed → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package pbuilder - 0.215ubuntu9

---------------
pbuilder (0.215ubuntu9) utopic; urgency=medium

  * pbuilderrc: Set default distribution to "utopic" (lp: #1316423)
 -- <email address hidden> (B. Clausius) Sun, 25 May 2014 09:14:44 +0200

Changed in pbuilder (Ubuntu):
status: Fix Committed → Fix Released
Revision history for this message
Ro (robert-markula) wrote :

Sorry guys, still not fixed in trusty. The latest package for trusty is pbuilder 0.215ubuntu7.

The DISTRIBUTION option is still set to 'saucy', which is a nasty and easily overlooked error.

Revision history for this message
Hans Joachim Desserud (hjd) wrote :

I ran into this issue on 14.04 a while back and found this issue when researching. Since I saw it had been fixed in utopic I created a similar patch for 14.04 and submitted that for review. Though, I now see that comment #2 had a debdiff for 14.04 all along. Could someone please take a look at reviewing this?

tags: added: trusty utopic
description: updated
Revision history for this message
Iain Lane (laney) wrote :

Uploaded to trusty, thanks.

Changed in pbuilder (Ubuntu Trusty):
status: New → Triaged
status: Triaged → In Progress
assignee: nobody → B Clausius (barcc)
Revision history for this message
Chris J Arges (arges) wrote : Please test proposed package

Hello Sergio, or anyone else affected,

Accepted pbuilder into trusty-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/pbuilder/0.215ubuntu7.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 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 to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

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

Changed in pbuilder (Ubuntu Trusty):
status: In Progress → Fix Committed
tags: added: verification-needed
Revision history for this message
Ro (robert-markula) wrote :

Confirmed as fixed in pbuilder 0.215ubuntu7.1 on trusty.

Thanks for fixing this bug!

tags: added: verification-done
removed: verification-needed
Revision history for this message
Chris J Arges (arges) wrote : Update Released

The verification of the Stable Release Update for pbuilder 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
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package pbuilder - 0.215ubuntu7.1

---------------
pbuilder (0.215ubuntu7.1) trusty; urgency=medium

  * pbuilderrc: Set default distribution to "trusty" (lp: #1316423)
 -- <email address hidden> (B. Clausius) Sun, 25 May 2014 09:14:44 +0200

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

Duplicates of this bug

Other bug subscribers