]> git.saurik.com Git - apt.git/log
apt.git
8 years agoChange anonscm.d.o links to /git/apt/apt.git and https
Julian Andres Klode [Sat, 13 Aug 2016 21:54:26 +0000 (23:54 +0200)] 
Change anonscm.d.o links to /git/apt/apt.git and https

This also fixes Debian/apt#20, but is slightly more complete. I
think /git also looks better than /cgit, so let's switch the Vcs
entry in control over too.

8 years agohttp(s): allow empty values for header fields
David Kalnischkies [Fri, 12 Aug 2016 20:13:09 +0000 (22:13 +0200)] 
http(s): allow empty values for header fields

It seems completely pointless from a server-POV to sent empty header
fields, so most of them don't do it (simply proven by this limitation
existing since day one) – but it is technically allowed by the RFC as
the surounding whitespaces are optional and Github seems to like sending
"X-Geo-Block-List:\r\n" since recently (bug reports in other http
clients indicate July) at least sometimes as the reporter claims to have
seen it on https only even through it can happen with both.

Closes: 834048
8 years agodrop incorrect const attribute from DirectoryExists
David Kalnischkies [Fri, 12 Aug 2016 17:11:01 +0000 (19:11 +0200)] 
drop incorrect const attribute from DirectoryExists

Since its existence in 2010 DirectoryExists was always marked with this
attribute, but for no real reason. Arguably a check for the existence of
the file is not modifying global state, so theoretically this shouldn't
be a problem. It is wrong from a logical point of view through as
between two calls the directory could be created so the promise we made
to the compiler that it could remove the second call would be wrong, so
API wise it is wrong.

It's a bit mysterious that this is only observeable on ppc64el and can be
fixed by reordering code ever so slightly, but in the end its more our
fault for adding this attribute than the compilers fault for doing
something silly based on the attribute.

LP: 1473674

8 years agofileutl: empty file support: Avoid fstat() on -1 fd and check result
Julian Andres Klode [Fri, 12 Aug 2016 11:55:09 +0000 (13:55 +0200)] 
fileutl: empty file support: Avoid fstat() on -1 fd and check result

When checking if a file is empty, we forget to check that
fstat() actually worked.

8 years agotests: don't do boundless string compares with data()
David Kalnischkies [Fri, 12 Aug 2016 09:27:19 +0000 (11:27 +0200)] 
tests: don't do boundless string compares with data()

Git-Dch: Ignore

8 years agoensure a good clock() value for usage and tests
David Kalnischkies [Fri, 12 Aug 2016 08:02:28 +0000 (10:02 +0200)] 
ensure a good clock() value for usage and tests

We use clock() as a very cheap way of getting a "random" value, but the
manpage warns that this could return -1, so we should be dealing with
this. Additionally, e.g. on hurd-i386 the value increases only slowly –
to slow for our fast running tests for randomness hence producing the
same range in both samples, so we introduce a simple busy-wait loop (as
clock is counting processor time used by the program) in the test which
delays the second sample just enough making our randomness a bit more
predictable.

8 years agodon't perform int<float in progress bar drawing
David Kalnischkies [Fri, 12 Aug 2016 07:07:59 +0000 (09:07 +0200)] 
don't perform int<float in progress bar drawing

Comparing floating numbers is always fun and in this instance a 9 < 9.0
is "somehow" true on hurd-i386 letting the tests fail by reporting that
too much progress achieved. A bit mysterious, but with some rework we
can use code which avoids dealing with the floats in this way entirely
and make our testcases happy.

8 years agoctest: show test output in case of failures
David Kalnischkies [Fri, 12 Aug 2016 08:45:45 +0000 (10:45 +0200)] 
ctest: show test output in case of failures

ctest as run by cmake by default does not show the output of the tests
even if the tests failed. In terms of our tests it could be handy to set
it always, but unfortunately it seems like cmake doesn't allow it if the
internet is to be believed, so lets enable it at least while building
packages and on travis.

Gbp-Dch: Ignore

8 years agoCMake: Exclude .md5 and .map doxygen files from install
Julian Andres Klode [Thu, 11 Aug 2016 16:09:39 +0000 (18:09 +0200)] 
CMake: Exclude .md5 and .map doxygen files from install

This is much better than removing them in debian/rules.

Gbp-Dch: ignore

8 years agoCMake: Use COPYONLY instead of @ONLY
Julian Andres Klode [Thu, 11 Aug 2016 15:54:19 +0000 (17:54 +0200)] 
CMake: Use COPYONLY instead of @ONLY

I don't know what happened back in 2009 when I wrote this,
but it seems I used the wrong option. These files should
not have any variable substitution done to them.

8 years agoCMake: Mark Doxygen as required and use DOXYGEN_EXECUTABLE
Julian Andres Klode [Thu, 11 Aug 2016 15:34:12 +0000 (17:34 +0200)] 
CMake: Mark Doxygen as required and use DOXYGEN_EXECUTABLE

Seems like I missed that when adding doxygen support.

Gbp-Dch: ignore

8 years agodebian/NEWS: Get rid of 1.3~pre3+cmake1 entry
Julian Andres Klode [Thu, 11 Aug 2016 17:48:54 +0000 (19:48 +0200)] 
debian/NEWS: Get rid of 1.3~pre3+cmake1 entry

This was only needed temporarily

Thanks: Axel Beckert for reporting

8 years agoRelease 1.3~rc1 1.3_rc1
Julian Andres Klode [Thu, 11 Aug 2016 12:05:04 +0000 (14:05 +0200)] 
Release 1.3~rc1

This commit looks heavy. Most of that comes from the fact that the
ordering of files in the translations changed with the switch to
CMake. I could have gone the extra mile to figure out the original
ordering and replicate it, but I have chosen to re-order everything
by file and line number, as that's easier.

8 years agodoc/po: Adjust translations for programmatic typo fix
Julian Andres Klode [Thu, 11 Aug 2016 12:00:20 +0000 (14:00 +0200)] 
doc/po: Adjust translations for programmatic typo fix

This enhances commit b9e6db821a6ddbc5bf6a90c80c296d4e91283a63.

Gbp-Dch: ignore

8 years agotests: copy 01autoremove from the right place
David Kalnischkies [Thu, 11 Aug 2016 00:19:31 +0000 (02:19 +0200)] 
tests: copy 01autoremove from the right place

With cmake using BUILDDIRECTORY at this place is not only as wrong as it
was before, but it might not even work always copying the system
provided one which might or might not be current and hence fails tests
needing it to be current like ./test-apt-move-and-forget-manual-sections

We don't want to always use the one from the source directory through
either like in autopkgtests.

Gbp-Dch: Ignore

8 years agoMerge branch 'feature/apt-dpkg-comm'
David Kalnischkies [Wed, 10 Aug 2016 23:36:23 +0000 (01:36 +0200)] 
Merge branch 'feature/apt-dpkg-comm'

8 years agoMerge branch 'feature/methods'
David Kalnischkies [Wed, 10 Aug 2016 23:36:18 +0000 (01:36 +0200)] 
Merge branch 'feature/methods'

8 years agohttp: auto-configure for local Tor proxy if called as 'tor'
David Kalnischkies [Sat, 6 Aug 2016 20:54:31 +0000 (22:54 +0200)] 
http: auto-configure for local Tor proxy if called as 'tor'

With apts http transport supporting socks5h proxies and all the work
in terms of configuration of methods based on the name it is called with
it becomes surprisingly easy to implement Tor support equally (and
perhaps even a bit exceeding) what is available currently in
apt-transport-tor.

How this will turn out to be handled packaging wise we will see in
https://lists.debian.org/deity/2016/08/msg00012.html , but until this is
resolved we can add the needed support without actively enabling it for
now, so that this can be tested better.

8 years agoblock direct connections to .onion domains (RFC7687)
David Kalnischkies [Sat, 6 Aug 2016 11:53:05 +0000 (13:53 +0200)] 
block direct connections to .onion domains (RFC7687)

Doing a direct connect to an .onion address (if you don't happen to use
it as a local domain, which you shouldn't) is bound to fail and does
leak the information that you do use Tor and which hidden service you
wanted to connect to to a DNS server. Worse, if the DNS is poisoned and
actually resolves tricking a user into believing the setup would work
correctly…

This does block also the usage of wrappers like torsocks with apt, but
with native support available and advertised in the error message this
shouldn't really be an issue.

Inspired-by: https://bugzilla.mozilla.org/show_bug.cgi?id=1228457
8 years agodisable explicit configuration of all packages at the end
David Kalnischkies [Thu, 28 Jul 2016 07:13:24 +0000 (09:13 +0200)] 
disable explicit configuration of all packages at the end

With b4450f1dd6bca537e60406b2383ab154a3e1485f we dropped what we
calculated here later on and now that we don't need it in the meantime
either we can just skip the busy work by default and expect dpkg to do
the right thing dropping also our little "last explicit configures"
removal trick introduced in b4450f1dd6bca537e60406b2383ab154a3e1485f.

This enables the last of a bunch of previously experimental options,
some of them existing still, but are very special and hence not really
worth documenting anymore (especially as it would need to be rewritten
now entirely) which is why the documentation is nearly completely
dropped.

The order of configuration stanzas in the simulation code changes
slightly as it isn't concerning itself with finding the 'right' order,
but any order is valid anyhow as long as the entire set happens in the
same call.

8 years agosimulate all package manager actions explicitly
David Kalnischkies [Thu, 28 Jul 2016 09:43:36 +0000 (11:43 +0200)] 
simulate all package manager actions explicitly

If a planner lets actions to be figured out by dpkg in pending calls
these actions aren't mentioned in a simulation. While that might be
a good thing for debugging, it would be a change in behavior and
especially if a planner avoids explicit removals could be confusing for
users. As such we perform the same 'trick' as in the dpkg implementation
by performing explicitly what would be done by the pending calls.

To save us some work and avoid desyncs we perform a layer violation by
using deb/ code in the generic simulation – and further we perform ugly
dynamic_cast to avoid breaking the ABI for nothing; aptitude is the only
other user of the simulation class according to codesearch.d.n and for
that our little trick works. It just isn't working if you happen to
extend pkgSimulate or otherwise manage to call the protected Go methods
directly – which isn't very realistic/practical.

8 years agotry to avoid removal of crossgraded packages
David Kalnischkies [Mon, 25 Jul 2016 14:36:53 +0000 (16:36 +0200)] 
try to avoid removal of crossgraded packages

The user has to approve the removal of a crossgraded package as it might
be needed to remove it (temporarily) in the process, but in most cases
we can happily avoid it and let dpkg unpack over it skipping the
remove. This has some effects on progress reporting and how deal with
selections through which makes this a tiny bit complicated.

8 years agoallow user@host (aka: no password) in URI parsing
David Kalnischkies [Mon, 1 Aug 2016 19:45:29 +0000 (21:45 +0200)] 
allow user@host (aka: no password) in URI parsing

If the URI had no password the username was ignored

8 years agofix programmatic typo in apt-key manpage
David Kalnischkies [Mon, 1 Aug 2016 13:13:14 +0000 (15:13 +0200)] 
fix programmatic typo in apt-key manpage

Reported-By: Johannes 'josch' Schauer on IRC
Gbp-Dch: Ignore

8 years agoallow methods to be disabled and redirected via config
David Kalnischkies [Sat, 6 Aug 2016 17:59:57 +0000 (19:59 +0200)] 
allow methods to be disabled and redirected via config

To prevent accidents like adding http-sources while using tor+http it
can make sense to allow disabling methods. It might even make sense to
allow "redirections" and adding "symlinked" methods via configuration.
This could e.g. allow using different options for certain sources by
adding and configuring a "virtual" new method which picks up the config
based on the name it was called with like e.g. http does if called as
tor+http.

8 years agoimplement socks5h proxy support for http method
David Kalnischkies [Thu, 4 Aug 2016 06:45:38 +0000 (08:45 +0200)] 
implement socks5h proxy support for http method

Socks support is a requested feature in sofar that the internet is
actually believing Acquire::socks::Proxy would exist. It doesn't and
this commit isn't adding it as that isn't how our configuration works,
but it allows Acquire::http::Proxy="socks5h://…". The HTTPS method was
changed already to support socks proxies (all versions) via curl. This
commit implements only SOCKS5 (RFC1928) with no auth or pass&user auth
(RFC1929), but not GSSAPI which is required by the RFC. The 'h' in the
protocol name further indicates that DNS resolution is delegated to the
socks proxy rather than performed locally.

The implementation works and was tested with Tor as socks proxy for
which implementing socks5h only can actually be considered a feature.

Closes: 744934
8 years agoimplement generic config fallback for methods
David Kalnischkies [Sun, 31 Jul 2016 16:05:56 +0000 (18:05 +0200)] 
implement generic config fallback for methods

The https method implemented for a long while now a hardcoded fallback
to the same options in http, which, while it works, is rather inflexible
if we want to allow the methods to use another name to change their
behavior slightly, like apt-transport-tor does to https – most of the
diff being s#https#tor#g which then fails to do the full circle
fallthrough tor -> https -> http for https sources. With this config
infrastructure this could be implemented now.

8 years agouse the same redirection handling for http and https
David Kalnischkies [Tue, 2 Aug 2016 12:49:58 +0000 (14:49 +0200)] 
use the same redirection handling for http and https

cURL which backs our https implementation can handle redirects on its
own, but by dealing with them on our own we gain finer control over which
redirections will be performed (we don't like https → http) and by whom
so that redirections to other hosts correctly spawn a new https method
dealing with these instead of letting the current one deal with it.

8 years agodetect redirection loops in acquire instead of workers
David Kalnischkies [Tue, 2 Aug 2016 20:44:50 +0000 (22:44 +0200)] 
detect redirection loops in acquire instead of workers

Having the detection handled in specific (http) workers means that a
redirection loop over different hostnames isn't detected. Its also not a
good idea have this implement in each method independently even if it
would work

8 years agofail on unsupported http/https proxy settings
David Kalnischkies [Wed, 3 Aug 2016 19:17:26 +0000 (21:17 +0200)] 
fail on unsupported http/https proxy settings

Closes: #623443
8 years agosuggest transport-packages based on established namescheme
David Kalnischkies [Mon, 1 Aug 2016 15:52:55 +0000 (17:52 +0200)] 
suggest transport-packages based on established namescheme

apt-transports not shipped in apt directly are usually named
apt-transport-% with % being what is in the name of the transport.
tor additional introduced aliases via %+something, which isn't a bad
idea, so be strip the +something part from the method name before
suggesting the installation of an apt-transport-% package.

This avoids us the maintainance of a list of existing transports
creating a two class system of known and unknown transports which would
be quite arbitrary and is unfriendly to backports.

8 years agosupport all socks-proxy known to curl in https method
David Kalnischkies [Sun, 31 Jul 2016 16:46:34 +0000 (18:46 +0200)] 
support all socks-proxy known to curl in https method

8 years agoensure all removes are reported to hook scripts
David Kalnischkies [Sun, 24 Jul 2016 17:00:08 +0000 (19:00 +0200)] 
ensure all removes are reported to hook scripts

Same reason and implementation as for configure.

8 years agoensure all configures are reported to hook scripts
David Kalnischkies [Sun, 24 Jul 2016 09:52:04 +0000 (11:52 +0200)] 
ensure all configures are reported to hook scripts

A planner might not explicitly configure all packages, but we need to
know all packages which will be configured for progress reporting and to
tell the hook scripts about them as they rely on this for their own
functionality.

8 years agodon't purge directly, but remove and do purge at the end
David Kalnischkies [Thu, 21 Jul 2016 16:46:34 +0000 (18:46 +0200)] 
don't purge directly, but remove and do purge at the end

If we want a package to be purged from the system tell dpkg in the
ordering (if it has to touch it explicitly) to remove it and cover the
purging of the config files at the end with a --purge --pending call.

That should help packages move conffiles around between packages
correctly even if the user is purging packages directly in big actions
like dist-upgrades involving many packages.

8 years agocall dpkg with --no-triggers by default
David Kalnischkies [Thu, 21 Jul 2016 14:33:01 +0000 (16:33 +0200)] 
call dpkg with --no-triggers by default

Implemented a long while ago now with relatively good progress reporting
involving triggers is a good time to try delaying the execution of
triggers across dpkg invocations finally by default.

Note: The bugreport talks also about 'smarter' configuration which is a
much bigger part and approached from multiple directions, but doesn't
really involve triggers per-se so considering it decoupled should help
in getting it done…

Closes: #626599
8 years agoselect remove/purge packages early on for dpkg
David Kalnischkies [Fri, 8 Jul 2016 07:40:46 +0000 (09:40 +0200)] 
select remove/purge packages early on for dpkg

Telling dpkg early on that we are going to remove these packages later
helps it with auto-deconfiguration decisions and its another area where
a planner can ignore the nitty gritty details and let dpkg decide the
course of action if there are no special requirements.

8 years agosave and restore selection states before/after calling dpkg
David Kalnischkies [Sun, 3 Jul 2016 12:39:16 +0000 (14:39 +0200)] 
save and restore selection states before/after calling dpkg

dpkg decides certain things on its own based on selections and
especially if we want to call --pending on purge/remove actions, we need
to ensure a clean slate or otherwise we surprise the user by removing
packages we weren't allowed to remove by the user in this run (the
selection might be an overarching plan for the not-yet "future").

Ideally dpkg would have some kind of temporal selection interface for
this case, but it hasn't, so we make it temporal with the risk of
loosing state if we don't manage to restore them.

8 years agouse dpkg --unpack --recursive to avoid long cmdlines
David Kalnischkies [Thu, 9 Jun 2016 09:48:16 +0000 (11:48 +0200)] 
use dpkg --unpack --recursive to avoid long cmdlines

Having long commandlines split into two is a huge problem if it happens
and additionally if we want to introduce planners which perform less
micromanagment its a good idea to leave the details for dpkg to decide.

In practice this doesn't work yet unconditionally as a bug is hiding in
the ordering code of dpkg, but it works if apt imposes its ordering so
this commit allows for now at least to solve the first problem.

8 years agopass --force-remove-essential to dpkg only if needed
David Kalnischkies [Sun, 3 Jul 2016 11:57:25 +0000 (13:57 +0200)] 
pass --force-remove-essential to dpkg only if needed

APT (usually) knows which package is essential or not, so we can avoid
passing this force flag to dpkg unconditionally  if the user hasn't
chosen a non-default essential handling obscuring the information.

8 years agoDo not set the binary dir in run-tests, it breaks stuff
Julian Andres Klode [Wed, 10 Aug 2016 14:42:23 +0000 (16:42 +0200)] 
Do not set the binary dir in run-tests, it breaks stuff

This breaks -j and does all sort of other weird stuff I did not
notice in the previous (non-parallel) runs.

Gbp-Dch: ignore

8 years agoFix some indentation issues in README.md
Julian Andres Klode [Wed, 10 Aug 2016 14:26:23 +0000 (16:26 +0200)] 
Fix some indentation issues in README.md

8 years agoMerge branch 'cmake'
Julian Andres Klode [Wed, 10 Aug 2016 14:19:00 +0000 (16:19 +0200)] 
Merge branch 'cmake'

8 years agotest: Automatically discover CMake build directory
Julian Andres Klode [Wed, 10 Aug 2016 12:16:52 +0000 (14:16 +0200)] 
test: Automatically discover CMake build directory

Look at the project root, and all directories directly below it and
pick the directory with the newest CMakeCache.txt file.

Gbp-Dch: ignore

8 years agoUpdate gitignore for cmake
Julian Andres Klode [Tue, 9 Aug 2016 18:17:21 +0000 (20:17 +0200)] 
Update gitignore for cmake

We cannot be sure if you are not running the build tree, so this
is a bit ugly.

Gbp-Dch: ignore

8 years agoGet rid of the old buildsystem
Julian Andres Klode [Tue, 9 Aug 2016 15:40:01 +0000 (17:40 +0200)] 
Get rid of the old buildsystem

Bye, bye, old friend.

8 years agoAdd statvfs.h.in to CMake directory
Julian Andres Klode [Tue, 9 Aug 2016 15:39:25 +0000 (17:39 +0200)] 
Add statvfs.h.in to CMake directory

Gbp-Dch: ignore

8 years agoMerge README.cmake into README.md
Julian Andres Klode [Tue, 9 Aug 2016 15:49:46 +0000 (17:49 +0200)] 
Merge README.cmake into README.md

Gbp-Dch: ignore

8 years agoapt-private: Do not include apti18n.h in headers
Julian Andres Klode [Mon, 8 Aug 2016 21:52:14 +0000 (23:52 +0200)] 
apt-private: Do not include apti18n.h in headers

This causes build failures when turning translations off.

8 years agoprepare-release: Switch over to CMake, set version in CMakeLists.txt
Julian Andres Klode [Mon, 8 Aug 2016 20:12:16 +0000 (22:12 +0200)] 
prepare-release: Switch over to CMake, set version in CMakeLists.txt

Teach the prepare-release script about the version new locations
and also set the version in CMakeLists, as that is better than
reading it from the changelog: CMake would not rerun automatically
otherwise if the version changed.

Gbp-Dch: ignore

8 years agoCMake: Rewrite existing Documentation support and add doxygen
Julian Andres Klode [Mon, 8 Aug 2016 19:53:46 +0000 (21:53 +0200)] 
CMake: Rewrite existing Documentation support and add doxygen

This can now build all documentation. It should also be fairly
reusable for other projects, as long as they follow the same
naming scheme for the po4a output files and set the PACKAGE_*
variables used here.

We could have done all translations in a single call to po4a
like the makefile based buildsystem does. While that would
have made the output slightly nicer, this solution offers a
huge performance gain because it can translate the documents
in parallel, which also means that the xsltproc stage does not
have to wait for all translations to be done first.

You might think that the add_custom_command() should list the
actual output files as BYPRODUCTS. This is not true however:
Because the files are not always generated, Ninja will think
missing byproducts mean that the target is out of date - which
is not what we want.

Finally, also add the missing doxygen support. Note that the
packaging script cleans up some md5 and map files created by
doxygen, otherwise it is fairly boring.

8 years agoCMake: Misc: Extract command path_join() from Translations
Julian Andres Klode [Mon, 8 Aug 2016 19:53:05 +0000 (21:53 +0200)] 
CMake: Misc: Extract command path_join() from Translations

This abstracts the joining of paths a bit better than having
basically the same code twice in the Translations module.

Gbp-Dch: ignore

8 years agoCMake: Check for ptsname_r() again
Julian Andres Klode [Mon, 8 Aug 2016 12:56:53 +0000 (14:56 +0200)] 
CMake: Check for ptsname_r() again

This was dropped in autotools as I found no use of the HAVE_PTSNAME_R
macro. Turns out it was typoed as HAVE_PTS_NAME_R. Fix the #ifdef and
add checks to CMake for it.

Closes: #833674
8 years agoCMake: Allow building without dpkg
Julian Andres Klode [Mon, 8 Aug 2016 12:35:09 +0000 (14:35 +0200)] 
CMake: Allow building without dpkg

It's not entirely clear if that is useful, but it cannot hurt. We
need to check what is missing to build on foreign systems without
dpkg.

Gbp-Dch: ignore

8 years agoCMake: Bump minimum required version to 3.4.0
Julian Andres Klode [Mon, 8 Aug 2016 12:16:36 +0000 (14:16 +0200)] 
CMake: Bump minimum required version to 3.4.0

The SOURCE_DIR property is used for the translation building and
was introduced in cmake 3.4

8 years agodebian: make autopkgtest run with CMake build dir
Julian Andres Klode [Mon, 8 Aug 2016 09:31:06 +0000 (11:31 +0200)] 
debian: make autopkgtest run with CMake build dir

8 years agoCMake: Add unit tests
Julian Andres Klode [Sun, 7 Aug 2016 16:22:14 +0000 (18:22 +0200)] 
CMake: Add unit tests

Add support for our GTest based unit tests. By default, CMake will
look in /usr/src/gtest for the external GTest project, but this can
be overriden by defining GTEST_ROOT when invoking cmake.

Gbp-Dch: ignore

8 years agoCMake: Translations: Build .pot in po binary dir, not project one
Julian Andres Klode [Tue, 9 Aug 2016 19:52:08 +0000 (21:52 +0200)] 
CMake: Translations: Build .pot in po binary dir, not project one

This was cluttering up things all over the place.

Gbp-Dch: ignore

8 years agoCMake: Translations: Allow excluding languages from translation
Julian Andres Klode [Tue, 9 Aug 2016 16:14:41 +0000 (18:14 +0200)] 
CMake: Translations: Allow excluding languages from translation

This seems to be needed for the hebrew translations.

Gbp-Dch: ignore

8 years agoCMake: Translations: Avoid rebuilding .mo if .pot did not change
Julian Andres Klode [Tue, 9 Aug 2016 11:10:28 +0000 (13:10 +0200)] 
CMake: Translations: Avoid rebuilding .mo if .pot did not change

Use the witness/byproducts approach to build the translations. A
byproduct of a command is like an output, but may be older than the
input.

Here, we generate a normal template with headers in the normal way
as a witness (and for Launchpad translations), but we also generate
a .pot-tmp0 template file without a header that gets copied to a
.pot-tmp byproduct only if it changed. This way, the .pot-tmp is
only updated if an actual string translation changed. We also
create a custom target for the .pot file that we'll depend on
later in the overall target creating the mo files to ensure that
the template is build before we try to build mo files.

Then we make the msgmerge depend on the .pot-tmp instead of the .pot
file, which means that msgmerge and msgfmt only get re-run if a string
change occured.

Gbp-Dch: ignore

8 years agoCMake: Translations: Add a statistics target
Julian Andres Klode [Tue, 9 Aug 2016 09:58:28 +0000 (11:58 +0200)] 
CMake: Translations: Add a statistics target

This is really useful stuff to have.

Gbp-Dch: ignore

8 years agoCMake: Translations: Build apt-all.pot and update .po files
Julian Andres Klode [Sun, 7 Aug 2016 14:01:18 +0000 (16:01 +0200)] 
CMake: Translations: Build apt-all.pot and update .po files

Merge all the per-domain templates into one template file using
msgcomm, stripping any line numbers in the input files, and sorting
the output per file.

This should create reasonably stable .pot and .po files that do not
change just because files move around. It should also be resilient
against some line changes, as long as one translated line is not
moved before/after another translated line.

Gbp-Dch: ignore

8 years agoCMake: Translations: Add support for shell scripts
Julian Andres Klode [Sun, 7 Aug 2016 13:52:59 +0000 (15:52 +0200)] 
CMake: Translations: Add support for shell scripts

Rework the arguments to apt_add_translation_domain so a user
can specify TARGETS and SCRIPTS, the latter being Shell scripts.

For each language (TARGETS being C++, SCRIPTS being Shell), a separate
template is generated via xgettext. Those templates are then merged
together by using msgcomm. In case there are no Shell scripts in
the translation domain, msgcomm will receive /dev/null instead of
a shell translation template.

This also reintroduces line numbers, as msgcomm would otherwise
re-order the merged files not only by filename, but also by message
string. It's unclear why it does that, it could just leave strings
within a file alone.

In contrast to the old build system, we use xgettext for shell scripts
instead of bash --dump-strings, as it's just easier to use the same
tool for everything. We also create valid headers.

8 years agoCMake: Translations: Write a .po file before writing a .mo file
Julian Andres Klode [Sun, 7 Aug 2016 13:50:04 +0000 (15:50 +0200)] 
CMake: Translations: Write a .po file before writing a .mo file

This makes debugging things easier.

Gbp-Dch: ignore

8 years agoCMake: Translations: Pass some additional arguments to commands
Julian Andres Klode [Sun, 7 Aug 2016 11:37:15 +0000 (13:37 +0200)] 
CMake: Translations: Pass some additional arguments to commands

This gets rid of the line numbers, adds the plural keyword,
and makes msgfmt print statistics, so we know how well translated
we are.

Gbp-Dch: ignore

8 years agoCMake: Translations: Make po templates depend on input files
Julian Andres Klode [Sun, 7 Aug 2016 11:36:11 +0000 (13:36 +0200)] 
CMake: Translations: Make po templates depend on input files

I wondered why the template was not rebuilt after I changed a file,
now I have the answer.

Gbp-Dch: ignore

8 years agoCMake: Set PACKAGE_MAIL variable
Julian Andres Klode [Sun, 7 Aug 2016 13:45:36 +0000 (15:45 +0200)] 
CMake: Set PACKAGE_MAIL variable

This is needed in a lot of places. Also adjust config.h.in to use
it instead of the bare email address.

Gbp-Dch: ignore

8 years agoCMake: po: Add mirror method
Julian Andres Klode [Sun, 7 Aug 2016 12:24:14 +0000 (14:24 +0200)] 
CMake: po: Add mirror method

I forgot this one, sorry

8 years agoCMake: Improve handling of vendor files
Julian Andres Klode [Sun, 7 Aug 2016 01:34:18 +0000 (03:34 +0200)] 
CMake: Improve handling of vendor files

First of all, instead of creating the files at configure time,
generate the files using normal target. This has the huge advantage
that they are rebuilt if their input changes. While we are at it,
also add dependencies on the vendor entity files.

This also fixes the path to the vendor script, which was given
relatively before, which obviously won't work when running from
inside a deeper subdirectory.

To speed things up, pass the --vendor option to getinfo, so we
do not have to find out the current vendor in getinfo all over
again.

Gbp-Dch: ignore

8 years agoCMake: Cache CURRENT_VENDOR and make it configurable
Julian Andres Klode [Sun, 7 Aug 2016 01:33:11 +0000 (03:33 +0200)] 
CMake: Cache CURRENT_VENDOR and make it configurable

Cache the current vendor, so we do not have to rerun getinfo when
reconfiguring stuff. This also has the nice effect of making the
vendor configurable, so you can manually specify it on a platform
that might not have dpkg (not that building without dpkg works
yet).

Gbp-Dch: ignore

8 years agovendor/getinfo: Accept --vendor VENDOR option
Julian Andres Klode [Sun, 7 Aug 2016 01:32:12 +0000 (03:32 +0200)] 
vendor/getinfo: Accept --vendor VENDOR option

This can be used to query a field for a specific vendor. It
also speeds up things a lot if we can cache the current vendor
in cmake and pass it to further getinfo invocations.

Gbp-Dch: ignore

8 years agoCMake: Use find_package() for curl instead of pkg_check_modules()
Julian Andres Klode [Sat, 6 Aug 2016 22:22:44 +0000 (00:22 +0200)] 
CMake: Use find_package() for curl instead of pkg_check_modules()

No need to involve pkg-config when CMake has builtin support

Gbp-Dch: ignore

8 years agoHandle interrupt when running Pre-Install hooks
Julian Andres Klode [Sat, 6 Aug 2016 22:45:25 +0000 (00:45 +0200)] 
Handle interrupt when running Pre-Install hooks

If we receive an interrupt, set a flag and do not abort
immediately without waiting for the child. Once the child
exited, exit with an error if the interrupted flag is set.

Closes: #832593
8 years agoreleasing package apt version 1.3~pre3+cmake2 1.3_pre3+cmake2
Julian Andres Klode [Sat, 6 Aug 2016 21:18:46 +0000 (23:18 +0200)] 
releasing package apt version 1.3~pre3+cmake2

8 years agoBuild-depend on pkg-config
Julian Andres Klode [Sat, 6 Aug 2016 21:18:17 +0000 (23:18 +0200)] 
Build-depend on pkg-config

8 years agoreleasing package apt version 1.3~pre3+cmake1 1.3_pre3+cmake1
Julian Andres Klode [Sat, 6 Aug 2016 20:49:28 +0000 (22:49 +0200)] 
releasing package apt version 1.3~pre3+cmake1

8 years agoCMake: Adjust abicheck locations
Julian Andres Klode [Sat, 6 Aug 2016 20:48:08 +0000 (22:48 +0200)] 
CMake: Adjust abicheck locations

Look in build/apt-pkg and build/apt-inst instead of build/bin.

Gbp-Dch: ignore

8 years agoCMake: debian: Switch packaging over to CMake and dh 9
Julian Andres Klode [Sat, 6 Aug 2016 19:40:01 +0000 (21:40 +0200)] 
CMake: debian: Switch packaging over to CMake and dh 9

This new packaging is much easier to read, although the duplication
in the install files is a bit annoying. We should probably also get
rid of the movefiles for solvers, planners, and https method; but
then we have to keep track of which methods exist in the apt package.

Another disadvantage is that building only the documentation packages
also requires building the code, as there's no way to turn off code
building in the project.

8 years agoCMake: Switch integration tests and travis over
Julian Andres Klode [Sat, 6 Aug 2016 19:32:36 +0000 (21:32 +0200)] 
CMake: Switch integration tests and travis over

This early support seems a bit hacky, but it's a hard switch: The
integration tests do not understand the old build system anymore
afterwards. I don't really like that.

8 years agoCMake: Add initial support for documentation building
Julian Andres Klode [Sat, 6 Aug 2016 19:18:39 +0000 (21:18 +0200)] 
CMake: Add initial support for documentation building

Build HTML docbook guides (untranslated) and manual pages
(including translations). Also install the examples in the
example subdirectory.

Translation of docbook guides has not been implemented yet,
but should be easy to do. The code also needs some cleanup
to automatically detect the available translations.

8 years agoCMake: Add support for building and installing .mo files
Julian Andres Klode [Sat, 6 Aug 2016 19:11:06 +0000 (21:11 +0200)] 
CMake: Add support for building and installing .mo files

Introduce support for building translation domain-specific
templates, merging them with the translations, and building
a language-specific .mo file.

The invocation of xgettext is done in the project source
directory, not in the current source directory, and all paths
are made relative to the project root, in order to have clean
templates.

This only supports the C++ source code for now, it unfortunately
does not handle the shell scripts of deselect yet.

8 years agoCMake: Add basic CMake build system
Julian Andres Klode [Sat, 6 Aug 2016 19:03:43 +0000 (21:03 +0200)] 
CMake: Add basic CMake build system

Introduce an initial CMake buildsystem. This build system can build
a fully working apt system without translation or documentation.

The FindBerkelyDB module is from kdelibs, with some small adjustements
to also look in db5 directories.

Initial work on this CMake build system started in 2009, and was
resumed in August 2016.

8 years agoprepare-release: Also search for libraries in CMake locations
Julian Andres Klode [Sat, 6 Aug 2016 17:07:22 +0000 (19:07 +0200)] 
prepare-release: Also search for libraries in CMake locations

With this change, the 'library' command looks for a library libX
in the directories: build/bin, */X, X.

This allows it to find the library when building with the
upcoming CMake backend, which places the libraries in a sub
directory of the build tree with the same name as the source
tree.

For example, if building in 'build/', the apt-pkg library
will be available at 'build/apt-pkg/libapt-pkg.so.5.0'.

In case there are multiple instances of a library,
the newest one will be used.

Gbp-Dch: ignore

8 years agodebian/control: Use versioned provides for same-ver library deps
Julian Andres Klode [Sat, 6 Aug 2016 03:10:36 +0000 (05:10 +0200)] 
debian/control: Use versioned provides for same-ver library deps

This simplifies the design a bit, as we do not need to read the
major ABI version number from some file / command.

Gbp-Dch: ignore

8 years agovendor/getinfo: Teach it about sourceslist-list-format
Julian Andres Klode [Sat, 6 Aug 2016 18:16:38 +0000 (20:16 +0200)] 
vendor/getinfo: Teach it about sourceslist-list-format

This makes it easier to write a generic subsitution tool for
handling substitutions in apt-key.in and sources.list.in

Gbp-Dch: ignore

8 years agovendor/getinfo: Provide command to determine vendor to use
Julian Andres Klode [Sat, 6 Aug 2016 01:50:54 +0000 (03:50 +0200)] 
vendor/getinfo: Provide command to determine vendor to use

Introduce the 'current' command to eventually replace the current
symbolic link. The current command does roughly the same as the
makefile, the code has just been cleaned up a bit to work better
as a shell function.

Gbp-Dch: ignore

8 years agodoc: Fix fuzzy string regression
Julian Andres Klode [Sat, 6 Aug 2016 20:20:56 +0000 (22:20 +0200)] 
doc: Fix fuzzy string regression

Commit b559d4846018c3adac362c6f1d0d697956586208 updated the
documentation to refer to apt.systemd.daily instead of the
apt cron job, introducing fuzzy strings in all the translations.

Gbp-Dch: ignore

8 years agotest: Pass -maxdepth 1 when running find in methods dir
Julian Andres Klode [Fri, 5 Aug 2016 12:32:56 +0000 (14:32 +0200)] 
test: Pass -maxdepth 1 when running find in methods dir

This fixes a test failures in the cmake branch which contains
sub directories in the methods output dir.

8 years agoconfigure.ac: Drop broken unused check for ptsname_r()
Julian Andres Klode [Sat, 6 Aug 2016 00:03:09 +0000 (02:03 +0200)] 
configure.ac: Drop broken unused check for ptsname_r()

AC_CHECK_FUNCS() defines HAVE_* variables, but AC_CHECK_FUNC()
does not.

Anyway: We do not have any code using HAVE_PTSNAME_R, so
just remove it.

8 years agoconfigure.ac: Re-enable pthread support for thread-local _error
Julian Andres Klode [Fri, 5 Aug 2016 23:51:41 +0000 (01:51 +0200)] 
configure.ac: Re-enable pthread support for thread-local _error

This was disabled in 1999 by jgg due to "glibc bugs". Let's hope
those are fixed now, 17 years later.

8 years agoMerge pull request Debian/apt#18 from corburn/doc
Julian Andres Klode [Fri, 5 Aug 2016 16:08:05 +0000 (18:08 +0200)] 
Merge pull request Debian/apt#18 from corburn/doc

doc: update path to periodic options script

8 years agodoc: update path to periodic options script
Jason Travis [Fri, 5 Aug 2016 14:59:42 +0000 (07:59 -0700)] 
doc: update path to periodic options script

8 years agoapt-private: #ifdef HAVE_STRUCT_STATFS_F_TYPE, not #if
Julian Andres Klode [Thu, 4 Aug 2016 18:49:10 +0000 (20:49 +0200)] 
apt-private: #ifdef HAVE_STRUCT_STATFS_F_TYPE, not #if

While autotools defines all macros to 1 explicitly, CMake only
defines them without a value. In such a case, #if fails with an
error and #ifdef works.

In preparation for a possible switch to CMake and to clean up
the code (rest uses #ifdef), use #ifdef here

8 years agoRelease 1.3~pre3 1.3_pre3
Julian Andres Klode [Thu, 4 Aug 2016 08:25:36 +0000 (10:25 +0200)] 
Release 1.3~pre3

8 years agoExecGPGV: Pass current config state to apt-key via temp file
Julian Andres Klode [Wed, 3 Aug 2016 16:50:37 +0000 (18:50 +0200)] 
ExecGPGV: Pass current config state to apt-key via temp file

Create a temporary configuration file with a dump of our
configuration and pass that to apt-key.

LP: #1607283

8 years agoExecGPGV: Fork in all cases
Julian Andres Klode [Wed, 3 Aug 2016 16:45:51 +0000 (18:45 +0200)] 
ExecGPGV: Fork in all cases

8 years agoExecGPGV: Rework file removal on exit()
Julian Andres Klode [Wed, 3 Aug 2016 16:44:27 +0000 (18:44 +0200)] 
ExecGPGV: Rework file removal on exit()

Create a local exiter object which cleans up files on exit.

8 years agogpgv: Unlink the correct temp file in error case
Julian Andres Klode [Thu, 28 Jul 2016 10:41:27 +0000 (12:41 +0200)] 
gpgv: Unlink the correct temp file in error case

Previously, when data could be created and sig not, we would unlink
sig, not data (and vice versa).

8 years agoapt-key: ignore any error produced by gpgconf --kill
David Kalnischkies [Sun, 31 Jul 2016 08:29:25 +0000 (10:29 +0200)] 
apt-key: ignore any error produced by gpgconf --kill

gpgconf wasn't always equipped with a --kill option as highlighted by
our testcases failing on Travis and co as these use a much older version
of gpg2. As this is just for cleaning up slightly faster we ignore any
error a call might produce and carry on. Use a recent enough gpg2
version if you need the immediate killing…

Gbp-Dch: Ignore
Reported-By: Travis CI
8 years agoapt-key: kill gpg-agent explicitly in cleanup
David Kalnischkies [Sun, 31 Jul 2016 07:27:19 +0000 (09:27 +0200)] 
apt-key: kill gpg-agent explicitly in cleanup

apt-key has (usually) no secret key material so it doesn't really need
the agent at all, but newer gpgs insist on starting it anyhow. The
agents die off rather quickly after the underlying home-directory is
cleaned up, but that is still not fast enough for tools like sbuild
which want to unmount but can't as the agent is still hanging onto a
non-existent homedir.

Reported-By: Johannes 'josch' Schauer on IRC