don't explicitly init ExtractTar InFd with invalid fd

The default constructor of the FileFd will kick in anyway,
which will know that the Fd is invalid while with this explicit
call it must be assumed that the fd is in fact valid, which
might generate errors in the future

set Fail flag in FileFd on all errors consistently

Previously some errors would set the Fail flag while some didn't
without a clear reason as all errors leave a bad FileFd behind,
so we use a helper now to ensure that all errors set the flag.

make the vprintf like _error->Insert public

Git-Dch: Ignore

fix support for multiple patterns in apt-cache search

Patterns can appear in the name as well as in the description,
they don't have to match all in the name/description only.

Closes: 691453

initialize the whole search array with zero

Git-Dch: Ignore

deprecate InstallProtect as a cpu-eating no-op

In the past packages were flagged "Protected" so that install/
remove markings where issued before the ProblemResolver.
Nowadays, the marking methods check if they are allowed to modify
the marking of a package instead, so that markings set by FromUser
calls are not overwritten anymore by automatic calls which elimates
the need for InstallProtect which just eats CPU now.

The method itself is left untouched for now in case frontend needs it
still for some wierd usecase, but they should be eliminated.

do unpacks before configures in SmartConfigure

Splits the big loop over dependencies in SmartConfigure which unpacks and
configures dependencies into two loops and reverse their order, so that all
dependencies which need to be unpacked are handled first and only after that
configures are issued for dependencies.

This is needed as otherwise the unpack of a (new) dependency will be issued
in between a configure call for two (or more) packages which form a loop,
which means the configure calls aren't part of the same dpkg call and
therefore dpkg bails out.

Such tight loops should really be avoided as they are usually wrong – and in
reality the dependencies in libreoffice were greatly simplified thanks to
Rene Engelhard so the problem is gone for the benefit of all.

Closes: 707578

try all providers in order if uninstallable in MarkInstall

fix priority sorting by prefering higher in MarkInstall

Used to work until a certain (here unnamed) person came along and used
the wrong operator causing low-priority packages to be sorted above
high-priority packages while choosing a provider in commit
 2b5c35c7bb915dbd46fefd7c79f05364ba22f93b from Nov 2011

prefer Essentials over Removals in ordering score

Doing Removes early is good to have them out of the way, so they
don't break 'Inst' or 'Conf' chains, but scoring them above Essentials
means that we end up upgrading (many) less important packages before
we handle big stuff like libc6 or debconf which not only fails if those
less important packages have unannounced (strict) dependencies, but also
leads to having these packages unconfigured for a long time triggering
bugs in maintainer scripts for no good reason (#708831).

So this commits sets the default value for remove scores to 100, which
is below the one for essentials (200) and a lot lower than the previous
default value (500).

rewrite pkgOrderList::DepRemove to stop incorrect immediate setting

Some squeeze → wheezy upgrades indicate that DepRemove runs amok
in complicated setups as it wasn't correctly working with or-groups.
Completely rewritten the check is now moving from or-group to or-group
instead.

The behavior should be the same as the code before, but
(hopefully) with less bugs and more comments.

Closes: 645713

remove -ldl from cdrom and -lutil from apt-get linkage

Building src:apt shows:
dpkg-shlibdeps: warning: package could avoid a useless dependency if
debian/apt/usr/lib/apt/methods/cdrom was not linked against libdl.so.2
(it uses none of the library's symbols)
dpkg-shlibdeps: warning: package could avoid a useless dependency if
debian/apt/usr/bin/apt-get was not linked against libutil.so.1 (it uses
none of the library's symbols)

build the en manpages in subdirectory doc/en

Building manpages becames more consistent this way and
it is simpler to ignore build artefacts, too.

streamline .gitignore further

adding some comments, regrouping files and use some patterns to avoid
updating the ignore file for new translations or abi changes

Git-Dch: Ignore

merged trunk

Merge branch 'debian/sid' of https://github.com/ion1/apt into debian/sid

prepare release

Fix double free (closes: #711045)

* Fix double free (closes: #711045)

* Fix crash when the "mirror" method does not find any entry
  (closes: #699303)

fix double free (closes: #711045)

cmdline/apt-key: Accept nonexistent --keyring file with adv as well

cmdline/apt-key: Create new keyrings with mode 0644 instead of 0600

* buildlib/apti18n.h.in:
  - fix build failure when building without NLS (closes: #671587)

debian/gbp.conf: add debian-tag key to ensure git-dch works

Update changelog to include Vcs-* changes

French translation : typo fix. Closes: #677272

Update Vcs fields

Use the canonical URLs. Rename Vcs-Bzr to Vcs-Git and use the more
efficient git protocol instead of http. Point Vcs-Browser to the
gitweb interface.

add export-dir and debian-branch

update .gitignore

prepare upload

merged previous bzr commit

moved to git, updated gbp.conf to match the bzr version

non-inline RunGPGV methods to restore ABI compatibility with previous versions to fix partial upgrades (Closes: #707771)

The rename in 0.9.7.9~exp2 moved the method body to the class definition
which means it became inline, which isn't ABI compatibile. The reverse of
moving inline to non-inline is safe though.

rebuild po

add .gitignore and gbp.conf

releasing version 0.9.8

fix uri test

refresh translations

merged patch from Daniel Hartwig to fix URI and proxy releated issues

merged from the debian-wheezy branch

* apt-pkg/algorithms.cc:
  - Do not propagate negative scores from rdepends. Propagating the absolute
    value of a negative score may boost obsolete packages and keep them
    installed instead of installing their successors.  (Closes: #699759)

properly handle if-modfied-since with libcurl/https
(closes: #705648)

Fix English spelling error in a message ('A error'). Unfuzzy
translations. Closes: #705087

Changelog entry for translation fixes

Complete French translation

Complete translation

Reformat PO file

Fix typo

releasing version 0.9.7.9~exp3

po: refresh

merged bundle from david

merged lp:~mvo/apt/add-missing-dsc-hashes

* debian/control:
  - replace manpages-it (closes: #704723)

* apt-pkg/sourcelist.cc:
  - fix segfault when a hostname contains a [, thanks to
    Tzafrir Cohen (closes: #704653)

share version strings between same versions (of different architectures)
to save some space and allow quick comparisions later on

* apt-pkg/cacheiterators.h:
  - provide DepIterator::IsSatisfied as a nicer shorthand for DepCheck

releasing version 0.9.7.9~exp2

po/*: refreshed

apt-pkg/contrib/gpgv.cc: fix InRelease check

  - sort group and package names in the hashtable on insert
* apt-pkg/pkgcache.cc:
  - assume sorted hashtable entries for groups/packages

merged lp:~mvo/apt/fix-inrelease5

po/{ca,cs,ru}.po: fix merge artifacts harder

micro-optimize and enhance readability of ListParser::VersionHash

equal comparisions are used mostly in same-source relations,
so use this to try to reuse some version strings

* apt-pkg/deb/debversion.cc:
  - add a string-equal shortcut for equal version comparisions

merge stuff from donkult

* doc/apt.conf.5.xml:
  - document Acquire::ForceIPv{4,6}

* methods/connect.cc:
  - use Errno() instead of strerror(), thanks to David Kalnischk

merge patches from daniel to fix merge artifact and test failure

merge patch from Daniel Hartwig

* Update all PO files and apt-all.pot
* French translation completed (Christian Perrier)

Update POT and PO files

releasing version 0.9.7.9~exp1

prepare upload

merged lp:~mvo/apt/fix-tagfile-hash

merged the debian-wheezy branch

add new config options "Acquire::ForceIPv4" and
"Acquire::ForceIPv6" to allow focing one or the other
(closes: #611891)

Turkish (Mert Dirik). Closes: #703526

merge patch from Colin to fix error message from getaddrinfo() (#703603)

apt-pkg/edsp.cc: do not use stderr in WriteSolution at all

apt-pkg/edsp.cc: do not spam stderr in WriteSolution

support dash-escaped text in clearsigned files as implementations are
free to escape all lines (we have no lines in our files which need
 to be escaped as these would be invalid fieldnames) and while ExecGPGV
would detect dash-escaped text as invalid (as its not expected in
 messages with detached signatures) it would be possible to "comment"
lines in (signed) dsc files which are only parsed but not verified

use FileFd instead of int fds to tidy up the interface a bit

* apt-pkg/deb/debindexfile.cc,
  apt-pkg/deb/deblistparser.cc:
  - use OpenMaybeClearSignedFile to be free from detecting and
    skipping clearsigning metadata in dsc and Release files

We can't write a "clean" file to disk as not all acquire methods copy
Release files before checking them (e.g. cdrom), so this reverts recombining,
but uses the method we use for dsc files also in the two places we
deal with Release files

  - add method to open (maybe) clearsigned files transparently
* ftparchive/writer.cc:
  - use OpenMaybeClearSignedFile to be free from detecting and
    skipping clearsigning metadata in dsc files

rename testcase to mention CVE number, make the code more consistent with
the rest and add some more tests (by fixing commented ones)

test/libapt/tagfile_test.cc: add missing "unistd.h" (thanks to Niels Thykier)

fix pkgTagSection::Exists() and add test

test/integration/framework: use EXIT_CODE to be consistent with the run-tests code

report failures via exit and ensure we don't overflow

* test/integration/framework:
  - continue after test failure but preserve exit status

* test/libapt/assert.h, test/libapt/run-tests:
  - exit with status 1 on test failure

ensure that we fclose/unlink/free in the new gpg-code as soon as possible

add testcase and update changelog

merged from davids bundle

  - if ExecGPGV deals with a clear-signed file it will split this file
    into data and signatures, pass it to gpgv for verification and
    recombines it after that in a known-good way without unsigned blocks
    and whitespaces resulting usually in more or less the same file as
    before, but later code can be sure about the format
* apt-pkg/deb/debmetaindex.cc:
  - reenable InRelease by default

split out a method to strip whitespaces only on the right side

* apt-pkg/acquire-item.cc:
  - keep the last good InRelease file around just as we do it with
    Release.gpg in case the new one we download isn't good for us

don't close stdout/stderr if it is also the statusfd

* apt-pkg/contrib/gpgv.cc:
  - ExecGPGV is a method which should never return, so mark it as such
    and fix the inconsistency of returning in error cases

* apt-pkg/indexcopy.cc:
  - rename RunGPGV to ExecGPGV and move it to apt-pkg/contrib/gpgv.cc