Age | Commit message (Collapse) | Author | Files | Lines |
|
|
|
Version the macros so that both can be used, and default the unversioned
one to the version 0 macro.
|
|
Add a missing comma, and $(shell) make function invocation.
Closes: #913816
Signed-off-by: Guillem Jover <guillem@debian.org>
|
|
When not using --multiversion, track any repeated instance that would
possibly benefit from running with --multiversion, and output it at the
end with a suggestion to use that option.
Prompted-by: Johannes Schauer <josch@debian.org>
|
|
When not in multi-version mode, we can only ever have one package in the
%packages hash, remove the confusing loop and replace with an explicit
assignment using the first array reference element.
|
|
When we specify a --hash argument we should not compute all checksums to
then discard them.
Closes: #916456
Based-on-patch-by: Chris Lamb <lamby@debian.org>
|
|
|
|
|
|
The .changes file describes an upload, and its Binary and Description
fields should contain (as documented) only references to the packages
being uploaded.
In case of a source-only upload, the Binary and Description fields
should be empty.
Closes: #818618
|
|
Filtering this meta-variable means that code parsing the field is unable
to know whether the source specified the build dependencies as relevant
for retriggering. We should also not expand the meta-variable as that
would lose information, so we let it through now.
Closes: #910734
|
|
This makes the creation of the object more compact and easier to read.
|
|
This move it at the beginning were it belongs, as the later code can
change behavior based on the format selected. This also will make it
possible to delegate the source format object selection to the
Dpkg::Source::Package class.
|
|
This move all ad-hoc code to use the new Dpkg::Source::Format module.
|
|
This new public module centralizes the handling of debian/source/format
so that other projects can reuse it instead of having to reimplement it.
Prompted-by: Mattia Rizzolo <mattia@debian.org>
Ref: https://salsa.debian.org/debian/devscripts/merge_requests/63
|
|
Make the regex more strict and decompose it right away instead of doing
a second pass over it. Only initialize minor when we need to update the
format. And fix the error for an invalid format to stop referencing the
Format field, which might not be involved during the parsing.
|
|
This makes it easier to construct a source package object with the
desired object, and initialize it as required.
Prompted-by: James McCoy <jamesan@debian.org>
Ref: https://salsa.debian.org/debian/devscripts/merge_requests/61
|
|
|
|
If the option has not been set, then the source format does not support
it and we should thus not recommend the --include-removal option in the
warning.
Closes: #913012
|
|
We initialized it already in the new constructor, no point in doing the
same twice.
|
|
Running code in the module scope means that anything importing the
module will execute that code, which is undesirable. Move the
initialization into a _select_objdump() sub, which gets assigned into
a state variable on demand.
|
|
The spawn() call was removed but not the matching import.
Fixes: commit 8ae966ae7d3635b8359829085db4262923ceae96
|
|
The default buildflags got moved from the Dpkg::BuildFlags module to
the Dpkg::Vendor::Debian, but this module was not adapted to match.
Instead of running the Debian hooks after the Ubuntu buildflags are set
up, run them first, and then modify/prepend the bits we need to change.
This fixes compiler optimization on ppc64el, and makes setting it more
future proof.
Fixes: commit d5374bc618310917557daa9c9ac2f4930515a0b2
Closes: #915881
Co-Author: Iain Lane <laney@ubuntu.com>
Co-Author: Adam Conrad <adconrad@ubuntu.com>
|
|
This makes several call sites more clear, as we move the logic inside
the function.
|
|
We should keep these and consider them as pseudo-EOF markers, and stop
processing once seen.
[lamby@debian.org:
- Initial test cases for dpkg-mergechangelogs functional tests. ]
Closes: #916056
|
|
|
|
Document inherited methods. Document the return values. Note which
arguments are optional. Reorder methods by their position in the stack.
|
|
The test suite expects to run under the C locale.
Fixes: https://rt.cpan.org/Ticket/Display.html?id=127314
|
|
All these environment variables are only relevant while running the
test suite. Do not pollute the rest of the environment.
|
|
Reported-by: Johannes Schauer <josch@debian.org>
Warned-by: shellcheck
|
|
Warned-by: shellcheck
|
|
Update to 603t.
|
|
|
|
This makes it possible to reuse the code by other modules.
|
|
|
|
The check is very simple, and can be done w/o requiring calling diff(1)
for each input file.
This makes the code shorter, more portable, and should be faster in the
non-binary cases.
|
|
|
|
|
|
This will contain different data depending on the program used. We do
not really care about that, later versions of GnuPG have stopped
emitting it, and it makes the test suite fail when using GnuPG 1.x.
Fixes: https://rt.cpan.org/Public/Bug/Display.html?id=127217
|
|
For the CPAN distribution or a non-dpkg-based distribution we cannot
assume that dpkg will be installed, or even available at build time.
Skip these tests if the tools is missing.
|
|
Split the test recommends from the requires and add develop recommends
into a prereqs metadata hierarchy.
|
|
|
|
This will enable the -TRIAL versioning that CPAN uses to avoid indexing
such releases.
|
|
Set the more generic name from within DPKG_DIST_IS_RELEASE, and use that
directly in the Build.PL.in file.
|
|
|
|
We do not distribute the PO files yet for CPAN, so the checks should not
fail when they are missing, even when testing with AUTHOR_TESTING=1.
|
|
When building the CPAN package, we need to also distribute the data
files, otherwise the modules will not work correctly once installed.
|
|
We had an empty do_perl_subst variable in the dist-cpan autotools
target that resulted in copying an empty file, but that was shadowed by
a subsequent copy of the original. So we were ending up with a pristine
non-substituted file.
We actually need to substitute the install paths during the CPAN module
building to preserve system and user settings at that point. Otherwise
the dpkg autotools configure settings might be completely out of touch
with the settings from the system where the CPAN module gets built.
|
|
Update to 602t.
|
|
The former calls /bin/pwd, while the latter uses the getcwd() syscall
directly.
Signed-off-by: Guillem Jover <guillem@debian.org>
|
|
This option will make the output be in reverse order.
|