]> git.saurik.com Git - apt.git/log
apt.git
8 years agoedsp: drop privileges before executing solvers
David Kalnischkies [Wed, 8 Jun 2016 11:44:29 +0000 (13:44 +0200)] 
edsp: drop privileges before executing solvers

Most (if not all) solvers should be able to run perfectly fine without
root privileges as they get the entire state they are supposed to work
on via stdin and do not perform any action directly, but just pass
suggestions on via stdout.

The new default is to run them all as _apt hence, but each solver can
configure another user if it chooses/must. The security benefits are
minimal at best, but it helps preventing silly mistakes (see
35f3ed061f10a25a3fb28bc988fddbb976344c4d) and that is always good.

Note that our 'apt' and 'dump' solver already dropped privileges if they
had them.

8 years agoedsp: if logging is requested, do it for internal, too
David Kalnischkies [Wed, 8 Jun 2016 09:34:53 +0000 (11:34 +0200)] 
edsp: if logging is requested, do it for internal, too

Create and log the EDSP(like) request even if we use the internal
resolver.

8 years agoedsp: if internal is used, keep this decision
David Kalnischkies [Wed, 8 Jun 2016 08:56:14 +0000 (10:56 +0200)] 
edsp: if internal is used, keep this decision

It wasn't noticeable before, but now with the (optional) logging it can
be observed that we decide in the internal path two times if an internal
or external solver should be used (and hence with logging, it is
attempted twice), so if we are in the internal path call the internal
resolver directly, which means those internal methods need to be public
– but we can hide them based on the symbol at least.

8 years agomove 'dump' solver from apt-utils to apt package
David Kalnischkies [Tue, 7 Jun 2016 18:08:27 +0000 (20:08 +0200)] 
move 'dump' solver from apt-utils to apt package

8 years agoedsp: optionally store a compressed copy of the last scenario
David Kalnischkies [Tue, 7 Jun 2016 15:01:33 +0000 (17:01 +0200)] 
edsp: optionally store a compressed copy of the last scenario

For bugreports and co it could be handy to have the scenario and all the
settings used in it around later for inspection for EDSP like protocols.
EDSP might not be the most interesting as the user can still interrupt
the process before the solution is applied and users tend to have an
opinion on the "rightness" of a solution, so it is disabled by default.

8 years agoedsp: prevent it from trying to write a cache
David Kalnischkies [Mon, 6 Jun 2016 13:54:06 +0000 (15:54 +0200)] 
edsp: prevent it from trying to write a cache

EDSP(-like) protocols are one-shot processes working on data which
exists only as long as they run (as they get feed via a pipe), so trying
to write a cache for it is pretty pointless, especially as it will
usually fail as the cache files tend to be owned by root, but the
process is run as a unpriviledged user (either _apt if called by root,
otherwise the user of the caller).

So this was in fact only observeable with our testcases which run as
non-root and the worst which happens is that a valid cache is overridden
with an invalid one which the next run will detect and not use.

Git-Dch: Ignore

8 years agoget right of the hardcoded string-length compares
David Kalnischkies [Mon, 6 Jun 2016 12:30:28 +0000 (14:30 +0200)] 
get right of the hardcoded string-length compares

With have better ways to compare, manipulate and work with strings, so
use it instead of counting string length by hand with is a wonder it
hasn't failed yet. Ignoreable from a changelog perspective as there is
no behaviour change.

Git-Dch: Ignore

8 years agorefactor EDSP classes for better internal reuse
David Kalnischkies [Sun, 5 Jun 2016 11:05:30 +0000 (13:05 +0200)] 
refactor EDSP classes for better internal reuse

The classes are all marked as hidden, so changing them is no problem ABI
wise and will help with introducing protocols similar to EDSP.
The change has no observeable behavior difference, its just code
juggling.

Git-Dch: Ignore

8 years agoprepare-release: ignore newer version symbols
David Kalnischkies [Sat, 4 Jun 2016 18:45:07 +0000 (20:45 +0200)] 
prepare-release: ignore newer version symbols

The script takes the version from the changelog, but if it lacks behind
and the symbols file already includes symbols tagged for the next
version the helper prints incorrect lines as NEW for these symbols, but
ideally it shouldn't print them at all as the symbol is already dealt
with.

Git-Dch: Ignore

8 years agoJapanese program translation update
Takuma Yamada [Sat, 4 Jun 2016 18:32:34 +0000 (20:32 +0200)] 
Japanese program translation update

Closes: 826291
8 years agoedsp: use a stanza based interface for solution writing
David Kalnischkies [Sat, 4 Jun 2016 17:53:54 +0000 (19:53 +0200)] 
edsp: use a stanza based interface for solution writing

EDSP had a WriteSolution method to write out the entire solution based
on the inspection of a given pkgDepCache, but that is rather inflexible
both for EDSP itself and for other EDSP like-protocols. It seems better
to use a smaller scope in printing just a single stanza based on a given
version as there is more reuse potential.

8 years agoedsp: use an ID mapping for the internal solver
David Kalnischkies [Sat, 4 Jun 2016 16:45:01 +0000 (18:45 +0200)] 
edsp: use an ID mapping for the internal solver

Currently an EDSP solver gets send basically all versions which means
the absolute count is the same, but that might not be true forever (and
with the skipping of rc-only versions it kinda is already) and even if
it were true, segfaulting on bad input seems wrong.

8 years agoapt-key: change to / before find to satisfy its CWD needs
David Kalnischkies [Thu, 2 Jun 2016 09:12:39 +0000 (11:12 +0200)] 
apt-key: change to / before find to satisfy its CWD needs

First seen on hurd, but easily reproducible on all systems by removing
the 'execution' bit from the current working directory and watching some
tests (mostly the no-output expecting tests) fail due to find printing:
"find: Failed to restore initial working directory: …"

Samuel Thibault says in the bugreport:
| To do its work, find first records the $PWD, then goes to
| /etc/apt/trusted.gpg.d/ to find the files, and then goes back to $PWD.
|
| On Linux, getting $PWD from the 700 directory happens to work by luck
| (POSIX says that getcwd can return [EACCES]: Search permission was denied
| for the current directory, or read or search permission was denied for a
| directory above the current directory in the file hierarchy). And going
| back to $PWD fails, and thus find returns 1, but at least it emitted its
| output.
|
| On Hurd, getting $PWD from the 700 directory fails, and find thus aborts
| immediately, without emitting any output, and thus no keyring is found.
|
| So, to summarize, the issue is that since apt-get update runs find as a
| non-root user, running it from a 700 directory breaks find.

Solved as suggested by changing to '/' before running find, with some
paranoia extra care taking to ensure the paths we give to find are really
absolute paths first (they really should, but TMPDIR=. or a similar
Dir::Etc::trustedparts setting could exist somewhere in the wild).

The commit takes also the opportunity to make these lines slightly less
error ignoring and the two find calls using (mostly) the same parameters.

Thanks: Samuel Thibault for 'finding' the culprit!
Closes: 826043
8 years agoignore std::locale exeception on non-existent "" locale
David Kalnischkies [Thu, 2 Jun 2016 10:44:58 +0000 (12:44 +0200)] 
ignore std::locale exeception on non-existent "" locale

In 8b79c94af7f7cf2e5e5342294bc6e5a908cacabf changing to usage of C++ way
of setting the locale causes us to be terminated in case of usage of an
ungenerated locale as LC_ALL (or similar) – but we don't want to fail
here, we just want to carry on as before with setlocale which we call in
that case just for good measure.

8 years agoPass -fvisibility-inlines-hidden to g++
Julian Andres Klode [Wed, 1 Jun 2016 12:54:32 +0000 (14:54 +0200)] 
Pass -fvisibility-inlines-hidden to g++

This reduces the number of symbols by about 10%. Unfortunately,
it does not seem to cover all the weird std::vector and friend
template expansions.

ABI should not brake due to that change: It was never specified
before whether an inline symbol was exported or not; so no library
could rely on its presence. Instead, the symbols were exported in
each library/program needing it and and then merged into a common
one by the dynamic linker.

Also update the symbol files to account for the removed symbols.

8 years agoAdd some more symbols to libapt-pkg5.0.symbols
Julian Andres Klode [Wed, 1 Jun 2016 12:54:32 +0000 (14:54 +0200)] 
Add some more symbols to libapt-pkg5.0.symbols

David added some more when he changed the output format for
numbers.

Gbp-Dch: ignore

8 years agoGet rid of .bzrignore
Julian Andres Klode [Wed, 1 Jun 2016 12:31:10 +0000 (14:31 +0200)] 
Get rid of .bzrignore

Gbp-Dch: ignore

8 years agotry to detect sudo spawned root-shell in prefixing
David Kalnischkies [Sun, 29 May 2016 20:09:51 +0000 (22:09 +0200)] 
try to detect sudo spawned root-shell in prefixing

It is a try as the we need to inspect SUDO_COMMAND which could be
anything – apt, apt-get, in /usr/bin, in a $DPKG_ROOT "chroot", build
from source, aliases, …

The best we can do is look if the SHELL variable is equal to the
SUDO_COMMAND which would mean a shell was invoked. That isn't fail-safe
if different shells are involved as sub-shells have the tendency of not
overriding the SHELL so a bash started from within zsh can happily
pretend to be still zsh, so we could have a look at /etc/shells for a
list, but oh well, we have to stop somewhere I guess.
This sudo-prefixing feature is a gimmick after all.

Closes: 825742
8 years agotravis: use gcc-5 instead of gcc(-4.9)
David Kalnischkies [Sat, 28 May 2016 15:02:26 +0000 (17:02 +0200)] 
travis: use gcc-5 instead of gcc(-4.9)

The std::put_time and std::get_time introduced in
9febc2b238e1e322dce1f94ecbed46d595893b52 are part of C++11, but not
implemented in GCC until version 5. std::put_time could actually be
worked around via using the facets put() directly, but get() isn't
implemented so that doesn't really help.

We require various tools from wily (which also means we can't build apt
on Debian stable) already, so requiring gcc-5 is just one more instead
of a big step [and an ignoreable change for changelog anyhow].

It also helps in testing what will actually be used (in terms of the
c++11 std ABI) instead of the old ABI.

Git-Dch: Ignore

8 years agolook into the right textdomain for apt-utils again
David Kalnischkies [Sat, 28 May 2016 11:53:09 +0000 (13:53 +0200)] 
look into the right textdomain for apt-utils again

Broken in e7e10e47476606e3b2274cf66b1e8ea74b236757 by looking always
into "apt" while we ship some tools in "apt-utils"…

8 years agouse std::locale::global instead of setlocale
David Kalnischkies [Sat, 28 May 2016 11:22:38 +0000 (13:22 +0200)] 
use std::locale::global instead of setlocale

We use a wild mixture of C and C++ ways of generating output, so having
a consistent world-view in both styles sounds like a good idea and
should help in preventing regressions.

8 years agouse de-localed std::put_time instead rolling our own
David Kalnischkies [Sat, 28 May 2016 10:55:21 +0000 (12:55 +0200)] 
use de-localed std::put_time instead rolling our own

8 years agoavoid changing the global LC_TIME for Release writing
David Kalnischkies [Sat, 28 May 2016 10:41:12 +0000 (12:41 +0200)] 
avoid changing the global LC_TIME for Release writing

Using C++ here avoids calling setlocale here which never really was that
ideal, but needed to avoid locale specific weekday/month names.

8 years agoaccept only the expected UTC timezones in date parsing
David Kalnischkies [Sat, 28 May 2016 09:03:35 +0000 (11:03 +0200)] 
accept only the expected UTC timezones in date parsing

HTTP/1.1 hardcodes GMT (RFC 7231 §7.1.1.1) and what is good enough for the
internet must be good enough for us™ as we reuse the implementation
internally to parse (most) dates we encounter in various places like the
Release files with their Date and Valid-Until header fields.

Implementing a fully timezone aware parser just feels too hard for no
effective benefit as it would take 5+ years (= until LTS's are out of
fashion) until a repository could use non-UTC dates and expect it to
work. Not counting non-apt implementations which might or might not
only want to encounter UTC here as well.

As a bonus, this eliminates the use of an instance of setlocale in
libapt.

Closes: 819697
8 years agoprevent C++ locale number formatting in text APIs
David Kalnischkies [Fri, 27 May 2016 16:10:39 +0000 (18:10 +0200)] 
prevent C++ locale number formatting in text APIs

Setting the C++ locale via std::locale::global(std::locale("")); which
would otherwise default to the default C locale (aka: unaffected by
setlocale) effects the formatting of numeric types in IO streams, which
for output for humans is perfectly sensible, but breaks our many text
interfaces used and parsed by us and others without expecting the
numbers to be formatted.

Closes: #825396
8 years agofix and document on the fly compressor config
David Kalnischkies [Fri, 27 May 2016 10:08:32 +0000 (12:08 +0200)] 
fix and document on the fly compressor config

libapt allows to configure compressors to be used by its system via
configuration implemented in 03bef78461c6f443187b60799402624326843396,
but that was never really documented and also only partly working, which
also explains why the tests weren't using it…

8 years agouse *.docs files instead of hardcoding in debian/rules
David Kalnischkies [Wed, 25 May 2016 06:32:01 +0000 (08:32 +0200)] 
use *.docs files instead of hardcoding in debian/rules

Git-Dch: Ignore

8 years agoremove semi-support for different build-dirs
David Kalnischkies [Wed, 25 May 2016 06:27:58 +0000 (08:27 +0200)] 
remove semi-support for different build-dirs

The debian/rules file tries to guess in which directory it is supposed
to be building, but that guess is always ./build – if it wasn't it
would fail later as not all rules take alternatives into acount.

So, as this is clearly not used lets remove this complexity instead of
fixing it up.

Git-Dch: Ignore

8 years agooverride lintian on doxygens embedded-javascript-library
David Kalnischkies [Tue, 24 May 2016 07:55:49 +0000 (09:55 +0200)] 
override lintian on doxygens embedded-javascript-library

The embedding is done completely automatic by doxygen and documented to
be that way for reasons: /usr/share/doc/doxygen/README.jquery

As we can't do anything about it, it is pointless to keep the warning.

8 years agouse https instead of git for Vcs-Git
David Kalnischkies [Tue, 24 May 2016 08:43:10 +0000 (10:43 +0200)] 
use https instead of git for Vcs-Git

Reported-By: lintian: vcs-field-uses-insecure-uri
Git-Dch: Ignore

8 years agofix two typos in untranslated errors of libapt-pkg
David Kalnischkies [Tue, 24 May 2016 08:40:22 +0000 (10:40 +0200)] 
fix two typos in untranslated errors of libapt-pkg

Reported-By: lintian: spelling-error-in-binary
Git-Dch: Ignore

8 years agofix two typos in apt.conf & apt-cache manpage
David Kalnischkies [Tue, 24 May 2016 08:37:57 +0000 (10:37 +0200)] 
fix two typos in apt.conf & apt-cache manpage

Reported-By: lintian: spelling-error-in-manpage
Git-Dch: Ignore

8 years agoremove duplicated 'the' from guides doc-base abstract
David Kalnischkies [Tue, 24 May 2016 07:24:28 +0000 (09:24 +0200)] 
remove duplicated 'the' from guides doc-base abstract

Reported-By: lintian: spelling-error-in-doc-base-abstract-field
Git-Dch: Ignore

8 years agono-change bump of Standards-Version to 3.9.8
David Kalnischkies [Tue, 24 May 2016 07:37:01 +0000 (09:37 +0200)] 
no-change bump of Standards-Version to 3.9.8

We don't have no menu file.

8 years agoupdate symbols file
David Kalnischkies [Fri, 20 May 2016 13:40:09 +0000 (15:40 +0200)] 
update symbols file

8 years agodon't try to get acquire lock in simulation mode
David Kalnischkies [Fri, 20 May 2016 12:42:50 +0000 (14:42 +0200)] 
don't try to get acquire lock in simulation mode

The code moving in eb1f04dda07c2b69549ad9fd793cca0e91841b3e
moved the acquire stuff above the simulation exit, so before getting
locks (and creating/chmod directories) we should be checking if we
should actually really do it…

[ignore as bugfix of an unreleased commit]

Git-Dch: Ignore

8 years agorefactor EDSP code into EDSP and EDSP-like parts
David Kalnischkies [Sat, 14 May 2016 16:07:12 +0000 (18:07 +0200)] 
refactor EDSP code into EDSP and EDSP-like parts

No real code change, just moving code around heavily to decouple the
EDSP specific parts from those we can reuse for EDSP-like protocols.

Git-Dch: Ignore

8 years agoedsp: use a limited scenario based on bool-array
David Kalnischkies [Fri, 6 May 2016 14:46:51 +0000 (16:46 +0200)] 
edsp: use a limited scenario based on bool-array

Its more space and runtime efficient to use a boolean set instead of a
CacheSet-based implementation.

Git-Dch: Ignore

8 years agoedsp: add Forbid-{New-Install,Remove} and Upgrade-All
David Kalnischkies [Fri, 6 May 2016 12:21:02 +0000 (14:21 +0200)] 
edsp: add Forbid-{New-Install,Remove} and Upgrade-All

This allows to differentiate properly between 'apt-get upgrade', 'apt
upgrade' and 'apt full-upgrade'.

8 years agoavoid triggering gcc's -Wunsafe-loop-optimizations in EDSP
David Kalnischkies [Mon, 2 May 2016 16:25:58 +0000 (18:25 +0200)] 
avoid triggering gcc's -Wunsafe-loop-optimizations in EDSP

apt/apt-pkg/edsp.cc: In function ‘bool EDSP::WriteLimitedScenario(pkgDepCache&, FILE*, const PackageSet&, OpProgress*)’:
apt/apt-pkg/edsp.cc:245:56: warning: cannot optimize loop, the loop counter may overflow [-Wunsafe-loop-optimizations]
    std::string dependencies[pkgCache::Dep::Enhances + 1];
                                                        ^
Using a std::array to silence gcc as well as as a code improvement feels right here.

Git-Dch: Ignore

8 years agoedsp: dump: support dumping into compressed file
David Kalnischkies [Wed, 27 Apr 2016 15:36:07 +0000 (17:36 +0200)] 
edsp: dump: support dumping into compressed file

8 years agoedsp: try harder to not generate unneeded error messages
David Kalnischkies [Wed, 27 Apr 2016 11:44:08 +0000 (13:44 +0200)] 
edsp: try harder to not generate unneeded error messages

The &= introduced in the EDSP-FileFd conversion isn't working to full
satisfaction for multiple && clauses as the && has a higher binding than
&= has, so that the methods were called even through they shouldn't
have because of previous errors. Using variadic functions we can solve
this in a slightly cleaner way bringing down the amount of 'broken pipe'
errors for the error case of the dump resolver substantially.

Git-Dch: Ignore

8 years agoconvert EDSP to be based on FileFd instead of FILE*
David Kalnischkies [Tue, 26 Apr 2016 10:26:12 +0000 (12:26 +0200)] 
convert EDSP to be based on FileFd instead of FILE*

I doubt there is any non-src:apt usage of these interfaces.

8 years agofail instead of segfault on unreadable config files
David Kalnischkies [Fri, 20 May 2016 07:37:24 +0000 (09:37 +0200)] 
fail instead of segfault on unreadable config files

The report mentions "apt list --upgradable", but there are others which
have inconsistent behavior ranging from segfaulting to doing something
with the partial (and hence incomplete) data. We had a recent report
about sources.list (#818628), this one mentions prefences, the obvious
next step is conf files… so the testcase is adapted to check for all
three in file and directory versions and run a bunch of commands each
time which should all have more or less the same behavior in such a case
(aka error out).

Closes: 824503
8 years agoRussian program translation update
Yuri Kozlov [Thu, 19 May 2016 18:54:32 +0000 (20:54 +0200)] 
Russian program translation update

Closes: 824702
8 years agodocument --no- as --show-upgraded is the default
David Kalnischkies [Mon, 16 May 2016 15:18:49 +0000 (17:18 +0200)] 
document --no- as --show-upgraded is the default

--show-upgraded is the default since
906fbf8886926eeb302332d997c9bd861291e155 so documenting it as if it
would be an option having an effect as is feels wrong and we do the same
for other options like install-recomends, download, …, too.

This commit also removes -u from the documentation, but still supports
it in the commandline parsing. Eventually we should deprecate the short
option, but for now lets just stop documenting it.

Closes: 824456
8 years agoshow final solution in --no-download --fix-missing mode
David Kalnischkies [Sat, 14 May 2016 08:43:03 +0000 (10:43 +0200)] 
show final solution in --no-download --fix-missing mode

This commit moves the creation of the fetcher and with it the
calculation of the filenames before the code generation the various
lists detailing the solution. This means that simulation comes even so
slightly closer to a real run as it will require and parse the package
indexes for filenames and queuing of URIs, so that a simulation "using"
an unavailable download method actually fails now.

The real benefit of this change is through that the rather special but
nontheless handy --no-download --fix-missing mode now actually shows
what the solution is it will apply to the system rather than the
solution it would if it could download all not-downloaded packages.

8 years agoshow globalerrors before asking for confirmation
David Kalnischkies [Fri, 13 May 2016 14:59:09 +0000 (16:59 +0200)] 
show globalerrors before asking for confirmation

Errors cause a kind of automatic no already, but warnings and notices
are only displayed at the end of the apt execution even through they
could effect the choice of saying yes/no to questions: E.g. if a
configuration (file) was ignored you wanted to have an effect or if an
external solver you used generated warnings suggesting that the solution
might be valid, but bogus non-the-less and similar things.

Note that this only moves those messages up to the question if the
answer is interactive – not if e.g. -y is used or no question is asked at
all so this has an effect only on interactive usage of apt(-get), not
script who might be parsing apt output.

8 years agoNormalize Signed-By values by removing trailing commas everywhere
Julian Andres Klode [Thu, 12 May 2016 08:04:19 +0000 (10:04 +0200)] 
Normalize Signed-By values by removing trailing commas everywhere

This fixes comparisons where either the stored or the input string
have a trailing comma.

8 years agoAdd conflicting Signed-By values to error message
Julian Andres Klode [Sun, 15 May 2016 17:16:54 +0000 (19:16 +0200)] 
Add conflicting Signed-By values to error message

This hopefully makes debugging things easier.

8 years agoapt.systemd.daily: Put opening brace of check_power on extra line
Julian Andres Klode [Sun, 15 May 2016 16:11:48 +0000 (18:11 +0200)] 
apt.systemd.daily: Put opening brace of check_power on extra line

The rest is also on the same line, so let's go consistent here
now that we have a bug report about it.

LP: #1581985

8 years agozh_CN.po: update simplified Chinese translation.
Zhou Mo [Sat, 14 May 2016 12:18:13 +0000 (12:18 +0000)] 
zh_CN.po: update simplified Chinese translation.

8 years agoProvide complete apt bash completion
Julian Andres Klode [Fri, 13 May 2016 08:06:31 +0000 (10:06 +0200)] 
Provide complete apt bash completion

This fixes Debian/apt#13 and the launchpad bug listed below,
but is far more advanced. I went through private-cmndline.cc
and looked at the supported options.

LP: #1573547
Thanks: Elias Fröhner and Svyatoslav Gryaznov for the initial work

8 years agoedsp: warn if unexpected stanzas appear in the solution
David Kalnischkies [Thu, 12 May 2016 14:21:10 +0000 (16:21 +0200)] 
edsp: warn if unexpected stanzas appear in the solution

Unexpected are for examples removal requests for versions which aren't
installed, installations of already installed versions & requests to
install and remove a package at the same time.

8 years agoedsp: document unique package identifiers
Johannes 'josch' Schauer [Thu, 12 May 2016 11:07:59 +0000 (13:07 +0200)] 
edsp: document unique package identifiers

Document that package identifiers must be unique (apt only uses the last
action for a given identifier) and that install requests do also imply
upgrades and downgrades (and thus removal of the old version). This is
to prevent that solvers express an upgrade or downgrade instruction as
two stanzas: a removal of the old version and an installation of the new
version. Instead, a single install stanza is sufficient to express
upgrade or downgrade requests.

8 years agoRelease 1.3~exp1 1.3_exp1
Julian Andres Klode [Wed, 11 May 2016 08:51:06 +0000 (10:51 +0200)] 
Release 1.3~exp1

8 years agoDutch program translation update
Frans Spiesschaert [Tue, 10 May 2016 21:21:46 +0000 (23:21 +0200)] 
Dutch program translation update

Closes: 823976
8 years agoedsp: support generic and solver-specific configs
David Kalnischkies [Tue, 10 May 2016 20:48:31 +0000 (22:48 +0200)] 
edsp: support generic and solver-specific configs

The spec was slightly inconsistent if the preferences setting is
available only as generic or specific setting & the code only supported
the specific one, while for the strict-pinning was only generic…

As the usual pattern for apt is to have both options we adapt the spec
and code to support both as well.

This also adds a purely informal "Solver" field so in case the request
is saved in a file, we know to which solver the sent preferences apply.

Closes: 823918
8 years agoupdate: Run Post-Invoke-Success if not all sources failed
Julian Andres Klode [Tue, 10 May 2016 17:15:17 +0000 (19:15 +0200)] 
update: Run Post-Invoke-Success if not all sources failed

Failures can happen and APT regardless will do a partial cache
update anyway. Because APT ensures that the list directory is
in a sane state, it makes sense to also call success hooks if
success was only partial - otherwise it loses sync with APT.

Most importantly, this causes the appstream cache to be empty,
see launchpad bug #1562733.

This is somewhat overly optimistic though: As soon as any repository
has nonexisting optional files, the missing optional files are also
treated as success, which means a single broken repository without an
InRelease file still runs Success hooks, even though it really should
not.

8 years agoStrip trailing commas for created signed-by fingerprint lists
Julian Andres Klode [Tue, 10 May 2016 17:09:19 +0000 (19:09 +0200)] 
Strip trailing commas for created signed-by fingerprint lists

This prevented some sources.list entries from working, an example
of which can be found in the test.

8 years agoMerge pull request julian-klode/apt#3 from adrian17/master
Julian Andres Klode [Tue, 10 May 2016 16:32:06 +0000 (18:32 +0200)] 
Merge pull request julian-klode/apt#3 from adrian17/master

Improve GetLocalitySortedVersionSet, speeds up apt search by 30%

8 years agodon't sent uninstallable rc-only versions via EDSP
David Kalnischkies [Tue, 10 May 2016 12:51:30 +0000 (14:51 +0200)] 
don't sent uninstallable rc-only versions via EDSP

Versions which are only available in dpkg/status aren't installable and
apt doesn't pick them as candidate for this reason – for the same reason
such packages shouldn't be sent to an external solver via EDSP. The
packages are pinned to -1, but if the solver has strict pinning disabled
it could end up picking this version anyhow – which is a request apt can
not satisfy.

Reported-By: Maximiliano Curia <maxy@debian.org> on IRC
8 years agoSpeed up GetLocalitySortedVersionSet.
Adrian Wielgosik [Sun, 8 May 2016 19:59:45 +0000 (21:59 +0200)] 
Speed up GetLocalitySortedVersionSet.

8 years agogpgv: show always webportal error on NODATA
David Kalnischkies [Sun, 8 May 2016 17:46:34 +0000 (19:46 +0200)] 
gpgv: show always webportal error on NODATA

gpg doesn't give use a UID on NODATA, which we were "expecting" (but not
using for anything), but just an error number. Instead of collecting
these as badsigners which will trigger a "invald signature" error with
remarks like "NODATA 1" we instead adapt a message similar to the NODATA
error of a clearsigned file (which is actually not reached anymore as we
split them up, which fails with a NOSPLIT error, which uses the same
general error message).

In other words: Not a security relevant change, just a user experience
improvement as we now point them to the most likely cause of the
problem instead of saying "invalid signature" which would point them in
the direction of the archive being broken (for everyone) instead.

Closes: 823746
8 years agoimplement Identifier field for IndexTargets
David Kalnischkies [Sun, 8 May 2016 16:03:48 +0000 (18:03 +0200)] 
implement Identifier field for IndexTargets

A frontend like apt-file is only interested in a specific set of files
and selects those easily via "Created-By". If it supports two locations
for those files through it would need to select both and a user would
need to know that implementation detail for sources.list configuration.

The "Identifier" field is hence introduced which by default has the same
value as "Created-By", but can be freely configured – especially it can
be used to give two indexes the same identifier.

8 years agoimplement Fallback-Of for IndexTargets
David Kalnischkies [Sun, 8 May 2016 09:58:36 +0000 (11:58 +0200)] 
implement Fallback-Of for IndexTargets

Sometimes index files are in different locations in a repository as it
is currently the case for Contents files which are per-component in
Debian, but aren't in Ubuntu. This has historic reasons and is perhaps
changed soon, but such cases of transitions can always happen in the
future again, so we should prepare:

Introduced is a new field declaring that the current item should only be
downloaded if the mentioned item wasn't allowing for transitions without
a flagday in clients and archives.

This isn't implemented 'simpler' with multiple MetaKeys as items (could)
change their descriptions and perhaps also other configuration bits with
their location.

8 years agodownload arch:all also for NATIVE_ARCHITECTURE indextargets
David Kalnischkies [Sat, 7 May 2016 14:45:35 +0000 (16:45 +0200)] 
download arch:all also for NATIVE_ARCHITECTURE indextargets

It looks a bit strange on the outside to have multiple "native
architecture", but all is considered an implementation detail and e.g.
packages of arch:all are in dependency resolution equal to native
packages.

8 years agodon't construct MetaIndex acquire items with IndexTargets
David Kalnischkies [Sat, 7 May 2016 12:52:08 +0000 (14:52 +0200)] 
don't construct MetaIndex acquire items with IndexTargets

We don't have to initialize the Release files with a set of IndexTargets
to acquire, but instead wait for the Release file to be acquired and
only then ask which IndexTargets to get.

Git-Dch: Ignore

8 years agodelay progress until Release files are downloaded
David Kalnischkies [Sat, 7 May 2016 12:44:53 +0000 (14:44 +0200)] 
delay progress until Release files are downloaded

Progress reporting used an "upper bound" on files we might get, expect
that this wasn't correct in case pdiff entered the picture. So instead
of calculating a value which is perhaps incorrect, we just accept that
we can't tell how many files we are going to download and just keep at
0% until we know. Additionally, if we have pdiffs we wait until we got
these (sub)index files, too.

That could all be done better by downloading all Release files first and
planing with them in hand accordingly, but one step at a time.

8 years agoTransactionManager can never be a nullptr
David Kalnischkies [Sat, 7 May 2016 12:08:35 +0000 (14:08 +0200)] 
TransactionManager can never be a nullptr

The code naturally evolved from a TransactionManager optional to a
required setup which resulted in various places doing unneeded checks
suggesting a more complicated setup than is actually needed.

Git-Dch: Ignore

8 years agofix same-mirror redirection for Release{,.gpg} pair
David Kalnischkies [Sat, 7 May 2016 11:45:25 +0000 (13:45 +0200)] 
fix same-mirror redirection for Release{,.gpg} pair

Commit 9b8034a9fd40b4d05075fda719e61f6eb4c45678 just deals with
InRelease properly and generates broken URIs in case the mirror (or the
achieve really) has no InRelease file.

[As this was in no released version no need to clutter changelog with a
fix notice.]

Git-Dch: Ignore

8 years agotests: disable generation of Release.gpg by default
David Kalnischkies [Wed, 4 May 2016 09:45:35 +0000 (11:45 +0200)] 
tests: disable generation of Release.gpg by default

Most tests just need a signed repository and don't care if it signed by
an InRelease file or a Release.gpg file, so we can save some time by
just generating one of them by default.

Sounds like not much, but quickly adds up to a few seconds with the
amount of tests we have accumulated by now.

Git-Dch: Ignore

8 years agotests: allow to disable generation of InRelease/Release.gpg file
David Kalnischkies [Wed, 4 May 2016 09:10:08 +0000 (11:10 +0200)] 
tests: allow to disable generation of InRelease/Release.gpg file

If the test just signs release files to throw away one of them to test
the other, we can just as well save the time and not create it.

Git-Dch: Ignore

8 years agotest: fix permission issue if run as root
David Kalnischkies [Wed, 4 May 2016 08:08:16 +0000 (10:08 +0200)] 
test: fix permission issue if run as root

Always those silly mistakes. Do what I mean, not what I said…

Reported-By: Travis
Git-Dch: Ignore

8 years agoallow redirection for items without a space in the desc again
David Kalnischkies [Tue, 3 May 2016 21:22:53 +0000 (23:22 +0200)] 
allow redirection for items without a space in the desc again

Broken in a4b8112b19763cbd2c12b81d55bc7d43a591d610.
If an item has a description which includes no space and is redirected
to another mirror the code which wants to rewrite the description
expects a space in there, but can't find it and the unguarded substr
command on the string will fail with an exception thrown…

Guarding it properly and everything is fine.

8 years agolet DPKG_COLORS default to our APT::Color setting
David Kalnischkies [Tue, 3 May 2016 12:11:30 +0000 (14:11 +0200)] 
let DPKG_COLORS default to our APT::Color setting

dpkg can optionally colorize its output since 1.18.5. Currently this
defaults to 'never', but it will eventually be 'auto'. It seems
reasonable to assume that a user who has enabled/disabled colors in apt
will want to have dpkg have the same state regarding color usage.

This isn't overriding explicit settings by the user, so in case a user
feels strongly about it one way or the other there are options.

8 years agoremove 100-levels config nesting limit
David Kalnischkies [Mon, 2 May 2016 16:18:37 +0000 (18:18 +0200)] 
remove 100-levels config nesting limit

The actual reason for this commit isn't the limit – there isn't much
point in using that much nesting – its in shutting up gcc mostly:

apt/apt-pkg/contrib/configuration.cc: In function ‘bool ReadConfigFile(Configuration&, const string&, const bool&, const unsigned int&)’:
apt/apt-pkg/contrib/configuration.cc:686:20: warning: cannot optimize loop, the loop counter may overflow [-Wunsafe-loop-optimizations]
    string Stack[100];
                    ^
by replacing this with C++s handy std::stack container (adapter).
Also cleans some whitespace noise from the file in the process.

8 years agoapt-key: add \n to dpkg-query --show --showformat
Carsten Hey [Sun, 1 May 2016 15:06:29 +0000 (17:06 +0200)] 
apt-key: add \n to dpkg-query --show --showformat

Guarding against 'broken' greps not dealing with non-text inputs
"just in case" by making the input text with a proper newline.

[commit message by David Kalnischkies]

Reported-On: IRC
Git-Dch: Ignore

8 years agowarn if apt-key is run unconditionally in maintainerscript
David Kalnischkies [Sun, 1 May 2016 12:43:23 +0000 (14:43 +0200)] 
warn if apt-key is run unconditionally in maintainerscript

We want to stop hard-depending on gnupg and for this it is essential
that apt-key isn't used in any critical execution path, which
maintainerscript are. Especially as it is likely that these script call
apt-key either only for (potentially now outdated cleanup) or still not
use the much simpler trusted.gpg.d infrastructure.

8 years agomove gnupg|gnupg2 from apt Depends to Recommends
David Kalnischkies [Sun, 1 May 2016 10:50:53 +0000 (12:50 +0200)] 
move gnupg|gnupg2 from apt Depends to Recommends

apt doesn't need gnupg in its main execution paths to function,
especially the Release file verification is done with gpgv only.
It is only used by apt-key for advanced key management functionality
most user will never use nor need.

The intend is to demote it eventually to Suggests, but we opt here for a
staged downgrade as there are still third-party repositories out there
which require apt-key functionality without depending on gnupg (or apt
for that matter).

8 years agoDon't copy strings in Startswith, Endswith
Adrian Wielgosik [Fri, 29 Apr 2016 08:38:02 +0000 (10:38 +0200)] 
Don't copy strings in Startswith, Endswith

8 years agoftparchive: Support writing Signed-By fields
Julian Andres Klode [Sun, 1 May 2016 09:38:44 +0000 (11:38 +0200)] 
ftparchive: Support writing Signed-By fields

8 years agobugscript: include all configuration fragment files
David Kalnischkies [Fri, 29 Apr 2016 16:40:08 +0000 (18:40 +0200)] 
bugscript: include all configuration fragment files

Closes: 820861
8 years agosupport Signed-By in Release files as a sort of HPKP
David Kalnischkies [Fri, 29 Apr 2016 14:48:16 +0000 (16:48 +0200)] 
support Signed-By in Release files as a sort of HPKP

Users have the option since apt >= 1.1 to enforce that a Release file is
signed with specific key(s) either via keyring filename or fingerprints.
This commit adds an entry with the same name and value (except that it
doesn't accept filenames for obvious reasons) to the Release file so
that the repository owner can set a default value for this setting
effecting the *next* Release file, not the current one, which provides a
functionality similar "HTTP Public Key Pinning". The pinning is in
effect as long as the (then old) Release file is considered valid, but
it is also ignored if the Release file has no Valid-Until at all.

8 years agosupport multiple fingerprints in signed-by
David Kalnischkies [Fri, 29 Apr 2016 08:16:42 +0000 (10:16 +0200)] 
support multiple fingerprints in signed-by

A keyring file can include multiple keys, so its only fair for
transitions and such to support multiple fingerprints as well.

8 years agogpgv: cleanup statusfd parsing a bit
David Kalnischkies [Fri, 29 Apr 2016 08:08:13 +0000 (10:08 +0200)] 
gpgv: cleanup statusfd parsing a bit

We parse the messages we receive into two big categories: Most of the
messages have a keyid as well as a userid and as they are errors we want
to show the userids as well. The other category is also errors, but have
no userid (like NO_PUBKEY). Explicitly expressing this in code should
make it a bit easier to look at and it also help in dropping additional
fields or just the newline at the end consistently.

Git-Dch: Ignore

8 years agodon't show NO_PUBKEY warning if repo is signed by another key
David Kalnischkies [Thu, 28 Apr 2016 22:31:49 +0000 (00:31 +0200)] 
don't show NO_PUBKEY warning if repo is signed by another key

Daniel Kahn Gillmor highlights in the bugreport that security isn't
improving by having the user import additional keys – especially as
importing keys securely is hard.

The bugreport was initially about dropping the warning to a notice, but
in given the previously mentioned observation and the fact that we
weren't printing a warning (or a notice) for expired or revoked keys
providing a signature we drop it completely as the code to display a
message if this was the only key is in another path – and is considered
critical.

Closes: 618445
8 years agogpgv: handle expired sig as worthless
David Kalnischkies [Thu, 28 Apr 2016 20:02:50 +0000 (22:02 +0200)] 
gpgv: handle expired sig as worthless

Signatures on data can have an expiration date, too, which we hadn't
handled previously explicitly (no problem – gpg still has a non-zero
exit code so apt notices the invalid signature) so the error message
wasn't as helpful as it could be (aka mentioning the key signing it).

8 years agogpgv: use EXPKEYSIG instead of KEYEXPIRED
David Kalnischkies [Thu, 28 Apr 2016 17:05:06 +0000 (19:05 +0200)] 
gpgv: use EXPKEYSIG instead of KEYEXPIRED

The upstream documentation says about KEYEXPIRED:
"This status line is not very useful". Indeed, it doesn't mention which
key is expired, and suggests to use the other message which does.

8 years agoshow StateCache flags in Pkg debug prettyprint
David Kalnischkies [Thu, 28 Apr 2016 13:03:55 +0000 (15:03 +0200)] 
show StateCache flags in Pkg debug prettyprint

This basically introduces ~33 flags in the output, but a package can
have only ~11 of them displayed at the same time. There is quiet a bit
of duplication also (an uninstalled package is by definition a
newinstall if its getting installed), but as this is debug output we are
better of showing them all in case one of them isn't set in a way it is
supposed to be set.

Git-Dch: Ignore

8 years agozh_TW.po: remove several fuzzy tags after review
Zhou Mo [Sat, 30 Apr 2016 13:24:42 +0000 (13:24 +0000)] 
zh_TW.po: remove several fuzzy tags after review

8 years agodeb822: Restore support for <multivalue>-{Add,Remove}
James McCoy [Wed, 20 Apr 2016 02:27:21 +0000 (22:27 -0400)] 
deb822: Restore support for <multivalue>-{Add,Remove}

Redesign of multivalue options in 463c8d801595ce5ac94d7c032264820be7434232
caused the parser to look for <multivalue>{Add,Remove} (no hyphen)
instead of the expected <multivalue>-{Add,Remove}.

8 years agofactor out Pkg/DepIterator prettyprinters into own header
David Kalnischkies [Thu, 28 Apr 2016 07:22:55 +0000 (09:22 +0200)] 
factor out Pkg/DepIterator prettyprinters into own header

The old prettyprinters have only access to the struct they pretty print,
which isn't enough usually as we want to know for a package also a bit
of state information like which version is the candidate.

We therefore need to pull the DepCache into context and hence use a
temporary struct which is printed instead of the iterator itself.

8 years agodeprecate confusing Pkg.CandVersion() method
David Kalnischkies [Thu, 28 Apr 2016 07:06:39 +0000 (09:06 +0200)] 
deprecate confusing Pkg.CandVersion() method

This method does not return the 'current' candidate of the DepCache
which would be most expected, but instead returns the version which
would be candidate in a default-only policy setting – aka ignoring
apt_preferences settings and co.

8 years agorespect user pinning in M-A:same version (un)screwing
David Kalnischkies [Wed, 27 Apr 2016 17:39:02 +0000 (19:39 +0200)] 
respect user pinning in M-A:same version (un)screwing

Using Pkg.CandVersion() here is wrong as its implementation will return
a candidate based just on the default policy settings ignoring user
preferences and otherwise set candidates (aka: it sidesteps the
pkgDepCache).

This causes M-A:same libraries to be detected as screwed even through
they aren't, so that they end up being kept back.

Reported-By: Felipe Sateler on IRC
8 years agoFileFd: avoid further writing if file failed
David Kalnischkies [Wed, 27 Apr 2016 11:41:31 +0000 (13:41 +0200)] 
FileFd: avoid further writing if file failed

If the file is in a failed state there is no point in trying to flush
out the buffers as the file is to be discarded anyhow & its likely all
this flushing is producing is additional error messages.

Git-Dch: Ignore

8 years agoMerge branch 'fix-https-noproxy' of github.com:patcable/apt
Julian Andres Klode [Wed, 27 Apr 2016 21:24:51 +0000 (23:24 +0200)] 
Merge branch 'fix-https-noproxy' of github.com:patcable/apt

8 years agorefactored no_proxy code to work regardless of where https proxy is set
Patrick Cable [Wed, 27 Apr 2016 20:55:55 +0000 (16:55 -0400)] 
refactored no_proxy code to work regardless of where https proxy is set

when using the https transport mechanism, $no_proxy is ignored if apt is
getting it's proxy information from $https_proxy (as opposed to
Acquire::https::Proxy somewhere in apt config). if the source of proxy
information is Acquire::https::Proxy set in apt.conf (or apt.conf.d),
then $no_proxy is honored.

8 years agoprivate-show: Get rid of old policy support code
Julian Andres Klode [Mon, 25 Apr 2016 19:58:10 +0000 (21:58 +0200)] 
private-show: Get rid of old policy support code

This does not make much sense anymore, now that we dropped the
old candidate ver algorithm.

8 years agopolicy: Remove TODO for replacing old GetCandidateVer()
Julian Andres Klode [Mon, 25 Apr 2016 19:50:20 +0000 (21:50 +0200)] 
policy: Remove TODO for replacing old GetCandidateVer()

Gbp-Dch: ignore