Age | Commit message (Collapse) | Author | Files | Lines |
|
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
our life much easier in bug #913946 & #916975
|
|
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
|
|
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
The problem occurred because we overrode the list of recorded
build-ids after each package. This works perfectly fine for dbgsym
packages as they always have a 1:1 with the original package.
However, a manual --dbg-package can host debug symbols for multiple
binary packages.
There are two paths to triggering this issue:
dh_strip -p pkg1 -p pkg2 --dbg-package=foo-dbg
and:
dh_strip -p pkg1 --dbg-package=foo-dbg
dh_strip -p pkg2 --dbg-package=foo-dbg
(This case was observed in krb5/1.15.2-2 where it loops over most of
the packages and calls dh_strip on each of them with the same
--dbg-package parameter.)
They fundamentally have the same issue and fix but the second case is
slightly more involved (as we have to merge with the previous run).
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
|
|
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
In practise, compat would never issue a stat call itself, but in the
future, we may remove the compat call that happens before this code
path is run.
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Ideally, people would migrate to the automatic dbgsym packages, which
do not seem trigger this problem.
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Use $main::VERSION to determine the version of the tool and embed that
into auto-generated snippets (e.g. via autoscript). This enables
lintian to extract the tool + version and display it on:
https://lintian.debian.org/tags/debhelper-autoscript-in-maintainer-scripts.html
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Set the local variables for Emacs in the central placed intended for
it, rather than in (currently) 95 individual files.
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
|
|
Re-indent of code pending next commit...
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
The pural version ($dh{DEBUGPACKAGES}) should no longer be used.
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
They were not needed in the first place (as "install -d" DTRT).
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Reported-by: Julien Cristau <jcristau@debian.org>
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
It breaks in fun and exciting ways for hardlinked ELF binaries if we
are not using build-ids. For now just disable the file(1) caching
in that case as it is not critical and an uncommon case.
The build-id case works (thanks to Sven Joachim) since the deattached
debug symbol is in a predictable location we can test for.
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Improve handling of hardlinked ELF files when extracting dbgsyms using
build-ids. This patches does not cover manual dbg packages under
compat 8 or older.
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Might as well, since it can save a fork+exec for readelf.
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Since binutils in stable supports "--enable-deterministic-archives"
just pass it unconditionally.
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
All uses are now singular, so change the code to only accept one
package (the latest passed) and only store one value.
The original $dh{DEBUGPACKAGES} is preserved for backwards
compatibility (e.g. dh_clistrip uses it), although it will now always
have at most one value in the array.
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
This ensures we do not collect build-ids for the udeb at all (unless
it actually contains debug symbols).
Signed-off-by: Niels Thykier <niels@thykier.net>
|
|
Signed-off-by: Niels Thykier <niels@thykier.net>
|