aaa

Dpkg::Source::Package: Honor require_valid_signature option

We need to pass this option forward to the verify_signature() calls,
otherwise we use the default.

Fixes: commit 139dfc4c78593d995610c0aa180300a9a7dd94ac
Fixes: commit 3821f024d92aabf24a333025c1c1956d8a45e718

po: Remove previous msgids for up-to-date translations

This was causing unit test failures.

Update German translation of manual pages

Update to 3016t.

Update German scripts translation

Update to 609t.

Dpkg::Source::Package::V3::Native: Do not say v1.0 for 3.0 formats

Dpkg::Changelog: Clarify inheritance and implementation references

Make it clear that these classes inherit from some other base class,
which will contain the missing documentation.

Prompted-by: intrigeri <intrigeri@debian.org>

Dpkg: Say class instead of object when appropriate

These were referring to the type and not the instance, which makes
using the incorrect nomenclature confusing.

Dpkg::Exit: Register exit handlers also for __DIE__

This makes sure we execute the handlers on normal errors, and can thus
remove the traps from several functions to run the handlers and to then
rethrow the exception.

Dpkg::Exit: Unregister all signal handlers once we have executed them

We should cleanup the handlers to the previous state otherwise we
end up changing the behavior globally, when this should be a local
cleanup behavior.

Closes: #932841

dpkg-source: Remove backwards compatibility code for legacy build-profiles

Dpkg::Vendor: Remove obsolete 'keyrings' hook

This hook has been obsoleted for a while and been emitting deprecation
warnings, so it's safe to remove now.

Dpkg::Conf: Remove obsolete methods and obsolete croak for method option

We do not bump the module version to 2.00, as these functions and option
have croak'ed almost since their introduction.

Dpkg::Checksums: Remove obsolete 'program' property warning

Dpkg::Version: Remove deprecation warning from semantic change in bool overload

Dpkg::Index: Change default value for unique_tuple_key to 1

Bump module version to 2.00.

Dpkg::Substvars: Remove obsolete no_warn() method

Bump module version to 2.00.

Dpkg::Source::Package: Remove and hide internal lowercase variables

Bump module version to 2.00.

Dpkg::Gettext: Remove obsolete _g() function

Bump module version to 2.00.

Dpkg::Exit: Hide internal lowercase @handlers variable

Bump module version to 2.00.

Dpkg::Deps::KnownFacts: Remove obsolete check_package() method

Bump module version to 2.00.

Dpkg::Compression: Hide internal lowercase variables

Bump module version to 2.00.

Dpkg::Changelog::Parse: Remove warnings for obsolete options and functions

Remove warnings for obsolete options forceplugin and libdir. Remove
obsolete functions changelog_parse_debian() and changelog_parse_plugin().

Bump module version to 2.00.

Dpkg::Changelog::Parse: Remove $ sigil from option names in POD

These are not actual variables, but option passed in a hash, so using
the $ sigil here is very confusing.

Dpkg::Changelog::Entry::Debian: Remove obsolete methods and hide variables

Remove obsolete methods check_header() and check_trailer(). Hide internal
lowercase variables $regex_header and $regex_trailer.

Bump module version to 2.00.

Dpkg::Changelog: Remove obsolete methods dpkg() and rfc822()

Bump module version to 2.00.

Dpkg: Remove internal lowercase variables

Bump module version to 2.00.

Dpkg::Deps: Check for valid virtual package version relations

Do not allow non-equal version relations in virtual provides.

Closes: #930317

Dpkg::Vendor::Debian: Do not set -Werror=implicit-function-declaration for C++

Closes: #939969

Dpkg::Vendor::Debian: Only scan /usr/local/ directories that exist

Closes: #932967

man: Move template symbol documentation into new deb-src-symbols(5)

Having the template symbols file format described in the dpkg-gensymbols
has been a continuous source of confusion for users, and makes
discovering it more difficult. Move it to its own man page, and link to
it from the various relevant man pages.

man: Describe the SONAME formats supported in deb-shlibs

dselect: Remove methods state files on purge

Ref: #930788
Spotted-by: Sven Joachim <svenjoac@gmx.de>

dpkg: Use DPKG_ADMINDIR to set the admindir

This was done in the name of security, but it does not make much sense
as root should make sure to be running in a clean environment. This is
problematic as it makes the interface hard to understand, and produces
unexpected behavior as dpkg can be called from within itself, or as a
wrapper for dpkg-query, which will then not inherit the admindir value.

Closes: #900071

dpkg: Fix and clarify behavior for packages marked to be on “hold”

A package marked to be on hold, denotes that the package should be kept
in its current version and should not be automatically installed, upgraded
or removed. The same applies to frontends. This behavior does not apply
when the user explicitly requested one of these actions, or the
--force-hold option is used.

It should never prevent dpkg from processing these for configuration (if
they were already in unpacked state) or from trigger processing, otherwise
we might get into trigger loops. Some of the code was already doing this,
but this was not consistently done. We update also the help output and
documentation to clarify the fixed behavior.

Closes: #926472

dpkg-gencontrol: Take hardlinks into account when computing Installed-Size

We should not add up the size of each hardlink into the final sum,
otherwise we will get a size in excess.

Closes: #923475
Co-Authored-by: Guillem Jover <guillem@debian.org>
Signed-off-by: Guillem Jover <guillem@debian.org>

dpkg-query: Try to use the description from the available file if not installed

When we have been requested to load the available file, and that
contains a description for a package that is not installed, we should
use that instead of printing the confusing string stating that the
description is not available.

Closes: #43573

libdpkg: Add missing symbols to the version map

Several of these are used by the test suite.

debian: Replace custom rule for 'configure' with call to dh_autoreconf

Having this custom rule to create the «configure» file means that it
might end up not being rebuilt if the timestamps of some other files
are off compared to the debian/changelog date. Using dh_autoreconf
guarantees that even if these timestamps got updated incorrectly,
the file will always be rebuilt as expected.

The historic rationale for not doing this before, even though this has
been considered a best packaging practice, was that because dpkg needs
explicit support for new architectures, which is the main reason to
get autotools updated to pull in new config.* files, we would get
reports so that we could add such support and release a new upstream
version. But this has never happened, and it's causing other problems.

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

po: Fix typo in Simplified Chinese programs translation

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

po: Update Portuguese programs translation

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

German dpkg translation update

Update to 1135t.

Fix a typo

Update German translation of manual pages

Update to 3000t.

Update German scripts translation

Update to 608t.

dpkg, u-a: Make variables static

Warned-by: sparse

test: Suppress new bogus cppcheck 1.89 false positives

Warned-by: cppcheck

dpkg: Remove redudant condition for sourcefile in updateavailable()

The preceding “if” already checks exclusively whether this variable
is NULL, so in this branch it will always be non-NULL.

Warned-by: cppcheck
Fixes: knownConditionTrueFalse

libdpkg: Use p instead of name in dpkg_arch_name_is_illegal()

We assign name to p, which we operate on everywhere in this function
except for the first check which we still perform on name. This confuses
at least cppcheck, and it's in any case inconsistent.

Warned-by: cppcheck
Fixes: nullPointerRedundantCheck

dselect: Make baselist::draw_column_*() col arguments const

Warned-by: cppcheck
Fixes: constParameter

man: Split dselect(1) --color from --colour option items

These become too long to fit in a 80-char terminal for many
translations, so let's split them, as we do anyway for the preferred
formatting to avoid the mandb warning.

man: Globally adjust left and disable hyphenation

These were being applied in the SEE ALSO section, but not consistently.
The output is more clear this way.

build: Bump minimal gettext version to 0.19.8

The GNU gettext macros used to check for symbols that were only provided
by the GNU gettext implementation, but not by other implementations such
as the one in musl.

Starting with GNU gettext 0.19.8 these internal symbols are not required
anymore.

test: Update stopwords for codespell 1.16.0

test: Skip build directories from codespell

These contain files that are not currently skept, and might contain
spelling errors from external sources.

Warned-by: codespell

test: Fix typos

Warned-by: codespell

libdpkg, dpkg: Use new versiondescribe_c() for non-localizable call sites

The output on log files must not be localized.

Reported-by: Julien Cristau <jcristau@debian.org>

libdpkg: Add new versiondescribe_c() to force a C locale

In some contexts we need output in the C locale, such as log files.
We add a new versiondescribe_c() that wraps the locale-dependant
versiondescribe() function and switches to the C locale and back
to the previous one before returning.

libdpkg: Add new C locale switch over support

This will make it possible to select either the current locale or a
specified one, currently only C is selectable.

We use this because the uselocale(3) POSIX API is recent, and the system
might not have it available, so we might need to fallback to use
setlocale(3) instead, which even though is not thread-safe, it's better
than nothing.

debian: Remove obsolete Breaks satisfied since oldstable

scripts: Match ~deb also as a backport marker in versions

Closes: #934980

scripts: Remove support for ~vola versioning

This was part of the volatile.debian.org archive, which was
decommissioned some time ago.

Ref: https://lists.debian.org/debian-volatile-announce/2012/msg00000.html

dpkg: Initialize flagdeppossi in check_conflict()

We only use this variable as a sentinel pointer so that we can make
sure the pdep pointer is not NULL in subsequent checks. But cppcheck
gets confused, so let's initialize it to make the code more
future-proof.

Warned-by: cppcheck

dpkg-deb: Fold two adjacent if conditionals into a single one

Warned-by: cppcheck

dselect: Reduce scope of variable

The variable was being assigned in one side of an if branch, but the
result never used. Let's reduce the scope to the if branch that actually
assigns and uses it.

Warned-by: cppcheck

libdpkg: Use the totalwritten variable for a consistency check

The variable was written to but never used.

Warned-by: cppcheck

libdpkg: Use the variable instead of a type as sizeof() argument

This is more future proof, and in this particular case it was tripping
over cppcheck and emitting a false positive for constArgument.

Warned-by: cppcheck

test: Clarify cppcheck va_list_usedBeforeStarted suppression

This is actually due to our local va_copy() compatibility code, which
cppcheck does not understand. Using the same code w/o the local macro
does not emit the false positive.

test: Remove perlcritic Documentation::RequirePodLinksIncludeText suppression

This bogus test has been removed from upstream perlcritic, and now the
code complains about an uninstalled check.

Warned-by: perlcritic

Dpkg::Changelog: Document methods provided by subclasses

Reported-by: Felix Lechner <felix.lechner@lease-up.com>

Dpkg::Source::Package::V2: Emit a special patch header on single-debian-patch

The previous default patch header does not make sense for a conglomerate
patch. Instead provide a more suitable default, that can always be
overridden with either the local-patch-header or patch-header files.

Closes: #933152

Dpkg::BuildFlags: Add new unset() method

This makes it possible to unset variables, so that they do not get
output by the various tools.

Requested-by: Daniel Schepler <dschepler@gmail.com>

Dpkg::BuildFlags: Remove unused hash keys

dpkg-shlibdeps: Document split_soname() function

Explain what each different pattern means, and what it maps to.

Prompted-by: Christopher Crim <christopher.crim@quoininc.com>

man: Clarify that the pager is called via «$SHELL -c»

man: Use a minus sign for a literal string

libdpkg: Clarify lock contender error message

Print the PID of the lock contender, switch the tense to past as the lock
might not be locked anymore by the time we finish, and add a warning
explaining that removing the lock file is never the correct solution.

build: Install m4 files into system aclocal directory

This makes it possible to publish these files so that other dpkg-related
projects can use them, instead of either embedding copies, or
duplicating the macros.

build: Add a serial versioning to the m4 files

aclocal uses this to determine whether it needs to update m4 files in
local trees.

Dpkg::Source::Package: Verify original tarball signatures at build time

When we are building the source package, while detecting whether there
is any original upstream tarball signature to be included in the .dsc,
it's the best time to verify them, so that when the .dsc gets eventually
signed there's a certification path for the maintainer that they are
including what they expected to be there.

Dpkg::OpenPGP: Add support for importing an OpenPGP key into a keyring

This is needed, for example, to verify original tarball signatures.

Dpkg::OpenPGP: Make it possible to verify detached signatures

This is required to be able to verify original upstream tarball
signatures, as they are expected to be ASCII armored detached
signatures for the upstream tarballs.

Dpkg::OpenPGP: Refactor signature verification into a new function

perl: Use File::Copy instead of spawning mv/cp commands

We use cp() in Dpkg::Source::Package to preserve the file attributes,
but just move() in dpkg-buildpackage where it does not matter.

Dpkg::Source::Package: Refactor original tarball handling

scripts: Remove support for versioned GnuPG 2 program and packages

The current default in Debian and anywhere else is to use unversioned
GnuPG 2.x binaries, so there's no need anymore to try these first or at
all.

dpkg-shlibdeps: Add support for new Build-Depends-Packages

This makes it possible to specify more than one package, which might be
needed during transition periods for example.

Closes: #926669
Based-on-patch-by: Frank Schaefer <kelledin@gmail.com>

dpkg: Do not clear selections for unknown packages

We should not clear the selections for packages that are not already
installed in the system, as they will end up disappearing on the next
operation anyway.

Closes: #927752

dpkg-genbuildinfo: Do not include irrelevant packages in the Binary field

This change is the companion to the one in dpkg-genchanges that lists
only relevant packages, instead of all the ones listed in debian/control,
but for the .buildinfo file instead of the .changes file.

scripts/mk: Add support for nostrip DEB_BUILD_OPTIONS when setting STRIP

man: Fix uncommon wording constructs

Warned-by: codespell

dpkg-buildpackage: Remove transient backwards R³ compat code

This code got deprecated before the spec had been finalized, but left in
just to make sure no remaining implementations were still using it.

scripts: Switch Getopt::Long from bundling to bundling_values

This got introduced after a request so that we could switch our tools
to a safe subset of the bundling logic, so let's enable it now that it
is available in the minimum supported perl version.

Bundling is in general unsafe, and not future-proof, as new options
might gain arguments which would then get misinterpreted, we should
not really be using it. But bundling the value into the option is a
common pattern and should be allowed.

build: Bump minimal Perl version to 5.24.1

The Perl version in Debian stretch is 5.24.1, which is the release that
will be oldstable once 1.20.x gets uploaded to Debian unstable.

libdpkg: Do not generate a backup file for the available database

This file contains information only valid as long as the downloaded
sources are also available. So it is closer to a cache than a database,
and as such, there is no need for an uncompressed backup, which can
take quite some space.

Closes: #343578

debian: Fold man page translation fix into translation section

Further fix as discussed in #931135

And in follow up on debian-l10n-german

Update German translation of manual pages

Update to 2998t.

Improvements from debian-l10n-german

Fix German man page translation. Closes: #931135

Bump version to 1.20.0