Ubuntu
ubuntu-dev-tools package

pbuilder-dist: Local archives othermirror overridden by later othermirror

Bug #1314076 reported by Per Carlson
24
This bug affects 5 people
Affects Status Importance Assigned to Milestone
ubuntu-dev-tools (Ubuntu)
Fix Released
Undecided
Felipe Reyes
Nominated for Artful by Felipe Reyes
Nominated for Bionic by Felipe Reyes
Declined for Cosmic by Mattia Rizzolo
Nominated for Trusty by Felipe Reyes
Nominated for Xenial by Felipe Reyes

Bug Description

When having a local archive at "/var/cache/archive", it's automatically detected and used by pbuild-dist, but will be overidden by a later othermirror in the argument list to pbuilder.

pelle@milliways:~$ ls -ld /var/cache/archive/trusty/
drwxr-xr-x 2 root root 4096 Apr 29 10:03 /var/cache/archive/trusty/

pelle@milliways:~$ pbuilder-dist trusty create --debug-echo
['sudo', 'HOME=/home/pelle', 'ARCHITECTURE=amd64', 'DISTRIBUTION=trusty', 'ARCH=amd64', 'DIST=trusty', 'DEB_BUILD_OPTIONS=', 'pbuilder', '--create', '--distribution', 'trusty', '--buildresult', '/var/cache/pbuilder/trusty_result', '--basetgz', '/var/cache/pbuilder/trusty-base.tgz', '--logfile', '/var/cache/pbuilder/trusty_result/last_operation.log', '--bindmounts', '/var/cache/archive/', '--othermirror', 'deb file:///var/cache/archive/ trusty/', '--mirror', 'http://archive.ubuntu.com/ubuntu', '--othermirror', 'deb http://archive.ubuntu.com/ubuntu trusty-security main restricted universe multiverse|deb http://archive.ubuntu.com/ubuntu trusty-updates main restricted universe multiverse|deb http://archive.ubuntu.com/ubuntu trusty-proposed main restricted universe multiverse', '--aptcache', '/var/cache/pbuilder/aptcache/ubuntu', '--components', 'main restricted universe multiverse']

As pbuilder only use the _last_ --othermirror argument, '--othermirror deb file:///var/cache/archive/ trusty' is lost.

This is gathered on:

pelle@milliways:~$ lsb_release -rd
Description: Ubuntu 14.04 LTS
Release: 14.04

pelle@milliways:~$ apt-cache policy ubuntu-dev-tools
ubuntu-dev-tools:
  Installed: 0.154~daily+bzr1420~ubuntu14.04.1
  Candidate: 0.154~daily+bzr1420~ubuntu14.04.1
  Version table:
 *** 0.154~daily+bzr1420~ubuntu14.04.1 0
        500 http://ppa.launchpad.net/udt-developers/daily/ubuntu/ trusty/main amd64 Packages
        100 /var/lib/dpkg/status
     0.153 0
        500 http://no.archive.ubuntu.com/ubuntu/ trusty/universe amd64 Packages
pelle@milliways:~$

Revision history for this message
Per Carlson (p8n) wrote :

Added to commits to the bug.

The first one (rev 1421) collapses all --othermirror configuration in one place.

The second one adds support for a -backport archive in the local archive. This is hand when backporting packages with the 'backportpackage' tool.

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

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

Changed in ubuntu-dev-tools (Ubuntu):
status: New → Confirmed
Revision history for this message
Florian W. (florian-will) wrote :

I was about to go insane re-creating and updating chroots and using all kinds of pbuilderrc options. Nothing seemed to help. Then I found this bug report and applied this patch to my local pbuilder-dist copy:
http://bazaar.launchpad.net/~p8n/ubuntu-dev-tools/othermirror_fixes/revision/1421

Sanity restored. Thanks, Per Carlson! Maybe we can get at least the bug fix commit into ubuntu-dev-tools?

Felipe Reyes (freyes)
Changed in ubuntu-dev-tools (Ubuntu):
assignee: nobody → Felipe Reyes (freyes)
Mattia Rizzolo (mapreri)
Changed in ubuntu-dev-tools (Ubuntu):
status: Confirmed → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-dev-tools - 0.166

---------------
ubuntu-dev-tools (0.166) unstable; urgency=medium

  * Team upload.

  [ Felipe Reyes ]
  * pbuilder-dist:
    + Fix handling of --othermirror when a local archive found. LP: #1314076

  [ Jelmer Vernooij ]
  * Recommend Bazaar (brz) or Breezy (brz); the latter provides a
    command-line compatible interface.

  [ Mathieu Trudel-Lapierre ]
  * Add a new tool "ubuntu-archive-assistant" tool for
    proposed-migration / mir review. More information on:
    https://lists.ubuntu.com/archives/ubuntu-devel/2018-September/040492.html
    The tool is not currently installed, see the launchpad bug #1799568.

  [ Benjamin Drung ]
  * ubuntutools/test: Introduce get_source_files helper function.
  * Update pylint and flake8 unittests.
  * Fix invalid escape sequences '\(' or '\)'. Closes: #911689

  [ Mattia Rizzolo ]
  * Add missing dependencies on sensible-utils (thanks lintian!).
  * wrap-and-sort -ast.
  * Bump Standards-Version to 4.2.1, no changes needed.
  * Use the new debhelper-compat(=11) notation and drop d/compat.
  * Clarify package descriptions for Python libraries.
    Thanks to Ben Finney for the patch. Closes: #804198, #804199
  * Add a recommends on ubuntu-keyring | ubuntu-archive-keyring.
    Closes: #838254
  * mk-sbuild: disable recommends also within the chroot.
    Thanks to Steve Beattie for the patch. LP: #1268684

 -- Mattia Rizzolo <email address hidden> Tue, 23 Oct 2018 22:08:04 +0200

Changed in ubuntu-dev-tools (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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