Release 1.16.5

Regenerate .pot files and merge .po files with them

Switch some words to their american spelling

dpkg-query: Add an Architecture column to --list output

Even if the output format is designed to be human readable, and for
machine parseable output «dpkg-query -W» should be used instead, there's
probably scripts in the wild parsing «dpkg-query -l». Thus, the new
column is inserted before the Description column which is unreliable
to parse anyway as its value will always contain spaces, which are the
column separators, and as such should be pretty safe.

The arch-qualified package names will still be printed, as that
allows to copy-and-paste package names as input to dpkg commands w/o
any possibility of ambiguity.

Closes: #673190

Suggested-by: Jonathan Nieder <jnieder@gmail.com>

dpkg-shlibdeps: Do not use undefined values returned from deps_parse()

When there's a bogus Build-Depends field on the source control file,
and the binary package being built is using a shared library package
that has a Build-Depends-Package field in the symbols file, the
dpkg-shlibdeps code will try to use an undefined value as deps_parse()
failed but the error was not handled before. Instead, error out early.

Closes: #640676

Based-on-patch-by: Raphaël Hertzog <hertzog@debian.org>
Signed-off-by: Guillem Jover <guillem@debian.org>

scripts: Accept “:native” arch-qualified Build-Dependencies

Closes: #558095

Signed-off-by: Guillem Jover <guillem@debian.org>

Add support for specific arch-qualified dependencies

This allows to use dependencies with arch-qualified package names. For
example:

  Depends: libc6:amd64 (>= 2.14), libgcc1

Closes: #676232

Signed-off-by: Guillem Jover <guillem@debian.org>

Recognize -? as an alias for --help in all commands

dpkg-mergechangelogs: Unify --help output with other commands

Print short options before long ones.

Align --version with --help on help output

Change all programs to accept -? instead of -h for help output

This switch frees the -h option to be used in the distant future for
other purposes, it also uses a character that does not have any other
obvious meaning for help output, and which is pretty safe to be used
blindly by the user in the same way as --help.

Avoid full stop and double newline at the end of errors and warnings

Error messages like "couldn't parse control information from foo.deb"
are not full sentences, so don't punctuate them like one. The main
purpose of this patch is stylistic consistency, but perhaps it can
also make copy-and-paste from messages like

dpkg-query: no path found matching pattern /usr/bin/agrep.

a little easier.

Most actual full sentences should remain untouched. A few full sentences
are error messages at heart, so this patch converts those to lower-case
sentence fragment form, too.

[guillem@debian.org:
 - Add missed strings.
 - Minor tweaks to strings. ]

Closes: #624000

Requested-by: Jari Aalto <jari.aalto@cante.net>
Signed-off-by: Guillem Jover <guillem@debian.org>

Switch to new notice() output function

This switches all ad-hoc stderr printing for error notices to the
notice() function.

libdpkg: Switch buffer I/O code to use struct dpkg_error

As a side effect this mkes the messages more clear as the caller has
more context to describe the error conditions.

Closes: #621763

libdpkg: Change pkg_format_parse() to take a dpkg_error argument

This fixes two issues at once: the more detailed inner message is
relied back to the more general outter printer which allows a clearer
error message, and it makes the pkg-format module stop printing things
at all which is bad for a library,

dpkg-deb: Rename string variable from err to errstr

libdpkg: Add new notice() output function

libdpkg: Move warning functions to a new report module

u-a: Print master and slave link names

Change --query output to print the master and slave link names. This
implies switching the Link field to print the alternative link instead
of the alternative name which was passed already as an argument. Add a
new Name field to print the alternative name.

Although, strictly speaking, this is an interface change, the previous
Link field value was wrong and not really useful, because it's already
known by the caller. A check of the whole archive does not reveal any
user of that field, so it seems pretty safe to do this change.

This also modifies the --config output when thre's a single alternative
to print the alternative link.

Closes: #679010

dpkg-query: Add new --control-list and --control-show commands

These replace the now deprecated --control-path command, as these do
not rely on any specific database layout.

dpkg-query: Refactor pkg_infodb_check_filetype() from control_path()

dpkg-query: Refactor pkg_infodb_is_internal() from pkg_infodb_print_filename()

libdpkg: Add new file_show() function

dpkg: Switch showdiff() to command_get_pager()

libdpkg: Add new command_get_pager() function

libdpkg: Detect ar header fields truncation

Do not allow building archives with truncated fields which would not
correspond to the packed member information, either the name being too
long or the size being too large.

Closes: #678933

build: Switch source compression to xz

s-s-d: Add new --no-close option to disable closing fds on --background

This enabled the caller to see process messages for debugging purposes,
or to be able to redirect file descriptors to log files, syslog or
similar.

Closes: #627333, #646425

deb-src-control(5): Document Private- field prefix

This prefix avoids getting warnins from dpkg-deb for unknown fields, and
also avoids stomping over the global namespace so that no future
official field will be reused.

deb(5): Document in more detail the supported ar archive format

Reported-by: Niels Thykier <niels@thykier.net>

libcompat: Add a dummy symbol to the empty module

This avoids warnings on systems that do not like a completely empty
object file, or an empty output after the preprocessor has run.

Dpkg::Source::Package: Do not assume $ENV{'HOME'} is defined

Closes: #677631

Reviewed-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Guillem Jover <guillem@debian.org>

dpkg-source: Minimize source architecture list

Reduce the source architecture list by removing architectures covered
already by architecture wildcards.

Closes: #675333

Dpkg::Arch: Add new debarch_is_wildcard function

Add support for binary-only key-value option in changelogs

This allows to mark changelog entries as being for a binary-only upload,
which implies there cannot be any source produced, and the binary
version is going to be different to the source version.

As such the version substvars will get different values, w/o needing to
hardcode a specific syntax. Although we fallback to the old syntax just
in case, for example to handle possible multiple consecutive binary-only
entries w/o needing to complicate the code too much.

Closes: #440094, #672723

Dpkg::Changelog::Entry::Debian: Correctly capitalize field names

Dpkg::Control::Fields: Remove obsolete changelog fields

These were made not visible as fields in commit
825656afe3e9856dc846c225df8af73d2376fa3e.

dpkg-buildflags: Add --status action to describe the flag settings

It's hard to see from a build log file what values should have been
used and why. The new --status action added by this patch tries to
output all meaningful information in a way useful for human consumption
and for automatic log parsers.

[guillem@debian.org:
 - Mark dpkg-buildflags as bold in man page and escape dash.
 - Use report("status", string). ]

Closes: #664058

Signed-off-by: Bernhard R. Link <brlink@debian.org>
Signed-off-by: Guillem Jover <guillem@debian.org>

dpkg-buildflags: Coalesce --list and --dump option parsing

Signed-off-by: Bernhard R. Link <brlink@debian.org>
Signed-off-by: Guillem Jover <guillem@debian.org>

Dpkg::BuildFlags: Record environment variables modified by maintainer

As flags modified by DEB_*_MAINT_* are not reflected by its origin, add
a new flag to describe flags modified that way.

Signed-off-by: Bernhard R. Link <brlink@debian.org>
Signed-off-by: Guillem Jover <guillem@debian.org>

Dpkg::BuildFlags: Record environment variables accessed or modified

Record environment variables looked at by Dpkg::BuildFlags and the
vendor hooks and make them available via the new Dpkg::BuildEnv module.

Based-on-patch-by: Bernhard R. Link <brlink@debian.org>
Signed-off-by: Guillem Jover <guillem@debian.org>

Dpkg::BuildEnv: New module to track build environment variables

Dpkg::BuildFlags: Add get_feature_areas()

Add a way for a caller to enumerate all possible values of get_features().

Signed-off-by: Bernhard R. Link <brlink@debian.org>
Signed-off-by: Guillem Jover <guillem@debian.org>

dpkg-source: support new option --no-unapply-patches

This option is supported by formats "2.0" and "3.0 (quilt)". It disables
the patch unapplication that can happen in the --after-build hook.

Extract Dpkg::Source::Quilt out of Dpkg::Source::Package::V3::quilt

This new Dpkg::Source::Quilt object handles all the traditional quilt
operations of applying/unapplying patches and the associated handling of
the quilt database (the .pc directory).

This commit should not introduce any user-visible changes.

Update German translation of manual pages

Update to 2097t.

Update German translation of manual pages

Update to 2055t37f4u.

Update German scripts translation

Update to 533t.

dpkg-source: enhance --commit to auto-whitelist modified binary files

That way the same command can be used whatever kind of upstream files has
been modified.

Reported-by: Cyril Brulebois <kibi@debian.org>

Dpkg::Source::Package::V2: refactor the handling of binary files

Move the code handling debian/source/include-binaries in a dedicated
object.

Dpkg::Source::Functions::is_binary(): localize $_ to avoid side-effects

Without this local($_) the function could modify the $_ variable as used
by the caller.

Dpkg::Source::Package: enable exit handlers for the commit operation

The commit function already uses various exit handlers to clean up in case
of failures. Unfortunately those handlers were never triggered because
there was no global eval intercepting the failures. This commit adds the
required logic.

Merge branch 'sid' (through tag '1.16.4.3')

Conflicts:
debian/changelog

Release 1.16.4.3

u-a: Only warn for now on out of range priorities on --install

There seems to be packages using priorities > INT_MAX, which although
bogus as they were previously overflowing the int used to store them,
that would cause installation failures when upgrading from squeeze.

Turn this into a warning for now and clamp the values, which will be
switched back to an error after wheezy, in dpkg 1.17.x.

Closes: #676874

libdpkg: Add new clamp macro

dpkg-source: keep the current patch header when regenerating the automatic patch

When dpkg-source regenerates the automatic patch (with formats "2.0" or
"3.0 (quilt)") it will keep the current patch header to avoid losing
changes made by the maintainer.

Reported-by: Cyril Brulebois <kibi@debian.org>

Dpkg::Source::Patch: let analyze() extract the patch header too

dpkg-source: 3.0 (quilt): restore quilt backup files after a failed patch

dpkg-source will now clean up after a failed application of a quilt patch.

This required passing “--reject-file=-” to patch to avoid the creation
of *.rej files that would clutter the source package and intercepting
the failure to restore the files from .pc/$patch/.

dpkg-source will also display a message explaining the most likely cause
of failure (patch applying with fuzz).

Closes: #652970

Update Swedish translation.

po/sv.po: 1055t0f0u.
dselect/po/sv.po: 270t0f0u.
man/po/sv.po: 2076t0f0u.

Merge branch 'sid' (through tag '1.16.4.2')

Conflicts:
debian/changelog

Release 1.16.4.2

libdpkg: Check correctly for out of range negative field width values

Use INT_MIN instead of INT_MAX when checking for negative out of range
values.

Regression introduced in commit 2bf4b48a9a6f7ddf854179b4b74013534e4594b9.

Closes: #676796

Merge branch 'sid' (through tag '1.16.4.1')

Conflicts:
debian/changelog

Release 1.16.4.1

libdpkg: Fix explicit file trigger activation

The lex parser passes a pointer to a buffer with the text, and as such
if we just keep a copy of the pointer the contents will change over the
parsing process. Instead copy the contents on trk_file_activate_start(),
and use that copy later on.

Regression introduced in commit 4ce011f68f8e499882db638c065008ce4015cd38.

Closes: #676684

Fix typos in changelogs

Signed-off-by: Niels Thykier <niels@thykier.net>
Signed-off-by: Guillem Jover <guillem@debian.org>

Bump version to 1.16.5

Release 1.16.4

Changelog police

Spotted-by: Helge Kreutzmann <debian@helgefjell.de>

dpkg-split: Allow overridding admindir from the environment

This allows to use the correct depotdir whenever dpkg is being called
with --admindir.

libdpkg: Defer Multi-Arch field reset to pkg_blank()

When removing a package pkgbin_blank() is called and an entry written
to the update log, which might make it unparseable if there's other
Multi-Arch instances present for that package.

Closes: #676496

dpkg: Activate all path components for file triggers

File triggers have been activated up to now explicitly whenever seen,
and only the requested pathname. While unpacking or removing, this is
not an issue as dpkg will traverse the hierarchy and trigger parent
directories during the process.

Because conffiles get a two staged installation, first unpacked into
<conffile>.dpkg-new and then installed in place on configure, a trigger
activated only after unpack will not see the <conffile> at its final
location or see it at all if it correctly ignores those file extensions.

This is an issue for conffiles or explicit dpkg-trigger file triggers,
as an interest on parent paths will not activate the trigger as those
parents are not traversed. With the subsequent cause of missed updates
because code has not been run on the actual installation of conffiles.

Closes: #675613, #676061, #676062, #676107, #676118, #676122

libdpkg: Use trig_file_activate_byname() instead of ad-hoc code

The trk_file functions were open coding this function, use it instead.

libdpkg: Add liblzma compression support

Use liblzma if available instead of external commands to handle .xz
and .lzma compressed files.

Among other things, this means the xz-utils package no longer has to
be part of the base system.

Based-on-patch-by: Jonathan Nieder <jrnieder@gmail.com>
Signed-off-by: Guillem Jover <guillem@debian.org>

scripts: Slice --help output strings

Use to opportunity that the strings have been modified in previous
commits to slice them, which should make translation a slightly more
manageable work in the future. It should also get identical strings
coalesced so that there's less to translate.

dpkg-gensymbols: Pack --help output

Reduce the amount of vertical lines produced by the --help output.

dpkg-buildpackage: Improve, split and group --help output

Improve some option descriptions and add missing ones, mark some
options as the default behaviour, group related options together,
and split pass-through options into separate sections.

dselect: On --help print action names from the menuentries table

This unmarks those strings for translation, as they should not get
translated anyway, makes sure all actions will always be listed and
those will not get out of sync.

scripts: Unify “<option>...” --help output

scripts: Hyphenate option arguments

Missed in commit 36a5005fcaa9b69efb2e9b08c0efe0f88042c5b3.

Update mailmap

build: Add new run-script to run local scripts

This sets up the environment to be able to run the local scripts using
local modules and data files.

u-a: Remove unreachable code at the end of alternative_select_choice()

The function contains an infinite loop that does not break out.

dpkg: Add missing space before && operand

dpkg: Remove unneeded showdiff() forward declaration

dpkg-divert: Do not warn on missing db .list file for never installed packages

When the package has never been installed before, and the unpack has not
yet finished, the package will be present on the database but the files
list file will not, which would produce a bogus warning. Check if the
package has ever been configured, before printing the warning.

Closes: #673518

Dpkg::Control::Fields: Put an & before field_capitalize() calls

The definition of the function appears later on, and perl 5.16 gives
warnings about usage before declaration.

Closes: #676262

Dpkg::File: Demote libfile-fcntllock-perl Depends to a Recommends

Fallback to use flock based locking, because libfile-fcntllock-perl
being an XS module makes building a new perl package bumping the perl
ABI impossible, as both packages become uninstallable.

Closes: #675947

Based-on-patch-by: Dominic Hargreaves <dom@earth.li>

Dpkg::File: Refactor file locking logic into this new module

Move libfile-fcntllock-perl dependency from dpkg-dev to libdpkg-perl.

dpkg: Use enum dep_check values instead of literal strings

s-s-d: Do not follow symlinks when creating pidfiles

[guillem@debian.org:
 - Change mode to 0666. ]

Closes: #675918

Signed-off-by: Guillem Jover <guillem@debian.org>

libcompat: Add compat symbol for O_NOFOLLOW

This symbol was only standardized in POSIX.1-2008, which is too recent.
Although it has been available on at least FreeBSD and Linux systems
before that, but dpkg supports other systems which might not have it.

dpkg(1): Clarify valid filenames for dpkg.cfg.d/ fragment files

This information is already present in dpkg.cfg(5), but it might not be
obvious one has to look there, so add it too to the main dpkg(1) where
that directory is referenced and a new entry in the FILES section.

Closes: #674674

dpkg(1): Clarify that the «dpkg -l» example only lists installed packages

Also add a reference to «dpkg-query --load-avail» as a way to list
available packages.

Closes: #673305

dpkg-deb: Add support for “none” as a valid compression strategy value

This is needed to be able to reset the compression strategy if one had
been specified before, otherwise a different compressor would error out
on compressor specific invalid values.

Closes: #674711

libdpkg: Switch compression strategy code from strings to enums

Make those values global, because they could be shared by different
compressors, because for validation purposes there's no point in making
them compressor specific as that's too late in case the compressor has
not been specified yet. And finally using enums instead of strings
allows to more easily handle the different strategy values.

libdpkg: Add more test cases for version module

This covers dpkg_version_blank(), dpkg_version_is_informative(),
branches for different numerical and alphabetical version and revision
part in dpkg_version_compare() comparison, and dpkg_version_relate().

libdpkg: Add test cases for eflags manipulators

libdpkg: Add more test cases for path module