From b6f1b480164b454661ddd4fdd3968302c6a3ebf6 Mon Sep 17 00:00:00 2001 From: David Kalnischkies Date: Wed, 10 Feb 2016 12:26:49 +0100 Subject: get dpkg lock in build-dep if cache was invalid again MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Regression introduced in a249b3e6fd798935a02b769149c9791a6fa6ef16, which in the case of an invalid cache would build the first part unlocked and later pick up the (still unlocked) cache for further processing, so the system got never locked and apt would end up complaining about being unable to release the lock at shutdown. The far more common case of having a valid cache worked as expected and hence covered up the problem – especially as tests who would have noticed it are simulations only, which do not lock. Closes: 814139 Reported-By: Balint Reczey Reported-By: Helmut Grohne on IRC --- apt-private/private-source.cc | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) (limited to 'apt-private/private-source.cc') diff --git a/apt-private/private-source.cc b/apt-private/private-source.cc index 7126feb78..5aaf6f48b 100644 --- a/apt-private/private-source.cc +++ b/apt-private/private-source.cc @@ -73,8 +73,6 @@ static pkgSrcRecords::Parser *FindSrc(const char *Name, pkgSrcRecords &SrcRecs,std::string &Src, CacheFile &Cache) { - if (Cache.BuildCaches(false) == false) - return nullptr; std::string VerTag, UserRequestedVerTag; std::string ArchTag = ""; std::string RelTag = _config->Find("APT::Default-Release"); @@ -321,12 +319,11 @@ bool DoSource(CommandLine &CmdL) return _error->Error(_("Must specify at least one package to fetch source for")); CacheFile Cache; - // Read the source list - if (Cache.BuildSourceList() == false) + if (Cache.BuildCaches(false) == false) return false; - pkgSourceList *List = Cache.GetSourceList(); // Create the text record parsers + pkgSourceList * const List = Cache.GetSourceList(); pkgSrcRecords SrcRecs(*List); if (_error->PendingError() == true) return false; @@ -725,8 +722,11 @@ bool DoBuildDep(CommandLine &CmdL) VolatileCmdL.size()); } + bool const WantLock = _config->FindB("APT::Get::Print-URIs", false) == false; if (CmdL.FileList[1] != 0) { + if (Cache.BuildCaches(WantLock) == false) + return false; // Create the text record parsers pkgSrcRecords SrcRecs(*List); if (_error->PendingError() == true) @@ -747,7 +747,6 @@ bool DoBuildDep(CommandLine &CmdL) Cache.AddIndexFile(new debStringPackageIndex(buildDepsPkgFile.str())); - bool WantLock = _config->FindB("APT::Get::Print-URIs", false) == false; if (Cache.Open(WantLock) == false) return false; pkgProblemResolver Fix(Cache.GetDepCache()); -- cgit v1.2.3