Ubuntu
apt package

'apt-get source' fails when any "deb-src" repository is unreachable

Bug #85590 reported by kko
14
This bug affects 1 person
Affects Status Importance Assigned to Milestone
apt (Ubuntu)
Fix Released
Medium
Unassigned

Bug Description

Binary package hint: apt

Hello.

When any "deb-src" repository listed in '/etc/apt/sources.list' can't be reached, 'apt-get source' fails.

To reproduce (on Kubuntu Dapper, with 'build-essential' installed):
1) add the following (nonsensical by content, just useful to demonstrate) line to your 'sources.list': "deb-src http://archive.canonical.com/ubuntu dapper-commercial main", and save it
2) do 'LC_ALL=C apt-get source libxine-main1'

The command dies with the following response:
"Reading package lists... Done
Building dependency tree... Done
E: Could not open file /var/lib/apt/lists/archive.canonical.com_ubuntu_dists_dapper-commercial_main_source_Sources - open (2 No such file or directory)"

This is somewhat silly, because the repository in question has nothing to do with the package we are requesting, and the sources are available from the main repository.

Tags: patch

Related branches

lp:~mvo/apt/mvo
Ubuntu Core Development Team: Pending requested
Diff: 443 lines (+76/-48)
14 files modified
apt-pkg/contrib/fileutl.cc (+10/-5)
doc/po/fr.po (+54/-32)
test/integration/framework (+1/-0)
test/integration/run-tests (+1/-1)
test/integration/test-autoremove (+1/-1)
test/integration/test-bug-590438-broken-provides-thanks-to-remove-order (+1/-1)
test/integration/test-bug-591882-conkeror (+1/-1)
test/integration/test-bug-595691-empty-and-broken-archive-files (+1/-1)
test/integration/test-bug-598669-install-postfix-gets-exim-heavy (+1/-1)
test/integration/test-compressed-indexes (+1/-1)
test/integration/test-disappearing-packages (+1/-1)
test/integration/test-pdiff-usage (+1/-1)
test/integration/test-policy-pinning (+1/-1)
test/integration/test-ubuntu-bug-614993 (+1/-1)
Ubuntu Core Development Team: Pending requested
Diff: 73066 lines (+13639/-12297) (has conflicts)
76 files modified
apt-pkg/contrib/fileutl.cc (+26/-2)
apt-pkg/contrib/strutl.cc (+2/-2)
apt-pkg/deb/debindexfile.cc (+13/-9)
apt-pkg/deb/deblistparser.cc (+6/-0)
apt-pkg/deb/debsystem.cc (+2/-2)
apt-pkg/depcache.cc (+15/-1)
apt-pkg/packagemanager.cc (+3/-3)
apt-pkg/pkgcache.cc (+1/-0)
apt-pkg/pkgcache.h (+1/-1)
apt-pkg/policy.cc (+5/-4)
buildlib/debiandoc.mak (+2/-2)
buildlib/po4a_manpage.mak (+1/-1)
cmdline/apt-get.cc (+12/-0)
debian/apt.cron.daily (+15/-28)
debian/changelog (+158/-0)
debian/control (+1/-1)
doc/examples/configure-index (+1/-0)
doc/po/de.po (+292/-590)
doc/po/fr.po (+8/-4)
po/apt-all.pot (+278/-266)
po/ar.po (+278/-266)
po/ast.po (+299/-287)
po/bg.po (+278/-266)
po/bs.po (+278/-266)
po/ca.po (+375/-241)
po/cs.po (+278/-266)
po/cy.po (+278/-266)
po/da.po (+405/-226)
po/de.po (+350/-247)
po/dz.po (+278/-266)
po/el.po (+278/-266)
po/es.po (+278/-266)
po/eu.po (+278/-266)
po/fi.po (+278/-266)
po/fr.po (+278/-268)
po/gl.po (+278/-266)
po/he.po (+2/-2)
po/hu.po (+278/-266)
po/it.po (+284/-272)
po/ja.po (+278/-266)
po/km.po (+278/-266)
po/ko.po (+278/-266)
po/ku.po (+278/-266)
po/lt.po (+278/-266)
po/mr.po (+278/-266)
po/nb.po (+377/-236)
po/ne.po (+278/-266)
po/nl.po (+280/-268)
po/nn.po (+278/-266)
po/pl.po (+278/-266)
po/pt.po (+279/-267)
po/pt_BR.po (+278/-266)
po/ro.po (+278/-266)
po/ru.po (+279/-267)
po/sk.po (+278/-266)
po/sl.po (+278/-266)
po/sv.po (+278/-266)
po/th.po (+278/-266)
po/tl.po (+278/-266)
po/uk.po (+278/-266)
po/vi.po (+543/-535)
po/zh_CN.po (+278/-273)
po/zh_TW.po (+278/-266)
test/integration/Packages-policy-pinning (+12/-0)
test/integration/framework (+2/-1)
test/integration/run-tests (+1/-1)
test/integration/test-autoremove (+1/-1)
test/integration/test-bug-590438-broken-provides-thanks-to-remove-order (+1/-1)
test/integration/test-bug-591882-conkeror (+5/-5)
test/integration/test-bug-595691-empty-and-broken-archive-files (+91/-0)
test/integration/test-bug-598669-install-postfix-gets-exim-heavy (+22/-0)
test/integration/test-compressed-indexes (+1/-1)
test/integration/test-disappearing-packages (+1/-1)
test/integration/test-pdiff-usage (+1/-1)
test/integration/test-policy-pinning (+228/-0)
test/integration/test-ubuntu-bug-614993 (+62/-0)
lp:ubuntu/natty/apt
lp:debian/apt
Michael Vogt (mvo)
Changed in apt:
importance: Undecided → Medium
status: Unconfirmed → Confirmed
Revision history for this message
Harrison Conlin (harrisony) wrote :

Problem still exits on Hardy

Revision history for this message
Robert Roth (evfool) wrote :

Still exists on Natty.

Revision history for this message
Torsten Spindler (tspindler) wrote :

It seems that an error is thrown when a file is not found when the sources.list file and directory is parsed. However, I have not found the place where this error happens. Currently I'm looking at apt-pkg/deb/debsrcrecords.

Revision history for this message
Torsten Spindler (tspindler) wrote :

The attached patch checks if the source file in the lists directory exists via the stat function. If it doesn't exist it fails silently by returning null, instead of throwing an error.

Revision history for this message
Torsten Spindler (tspindler) wrote :

The last patch only applies against Lucid's version of apt, this one against Natty.

Revision history for this message
Torsten Spindler (tspindler) wrote :

I've placed upgraded packages for Lucid and Natty in a PPA: https://launchpad.net/~tspindler/+archive/apt .

Changed in apt (Ubuntu):
status: Confirmed → In Progress
Brian Murray (brian-murray)
tags: added: patch
Revision history for this message
Michael Vogt (mvo) wrote :

Thanks Thorsten! Slightly modfied version is in my branch now.

Changed in apt (Ubuntu):
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package apt - 0.8.11.5ubuntu2

---------------
apt (0.8.11.5ubuntu2) natty; urgency=low

  [ Michael Vogt ]
  * apt-pkg/deb/debindexfile.cc:
    - ignore missing deb-src files in /var/lib/apt/lists, thanks
      to Thorsten Spindler (LP: #85590)
  * apt-pkg/contrib/fileutl.cc, apt-pkg/deb/dpkgpm.cc:
    - honor Dpkg::Chroot-Directory in the RunScripts*() methods
 -- Michael Vogt <email address hidden> Thu, 03 Mar 2011 17:39:30 +0100

Changed in apt (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.