Age | Commit message (Collapse) | Author | Files | Lines |
|
|
|
When new slave links are added or the targets for existing ones are
changed, we should not give bogus and confusing warnings as if the link
group was broken on disk and needs to be updated. Instead detect these
cases and give an informative message noting that the link group needs
to be updated due to the slave links changes. But still warn in case any
of the slaves is not being updated and is trully broken.
|
|
The new function adds a struct slave to a struct alternative, taking
care of allocating the slave strings as copies.
|
|
|
|
|
|
Closes: #719746
Spotted-by: "Brian S. Julin" <bri@abrij.org>
|
|
|
|
Reported-by: Pedro Ribeiro <pedrib@gmail.com>
|
|
|
|
Conflicts:
debian/changelog
dpkg-deb/main.c
scripts/Dpkg/Shlibs/SymbolFile.pm
|
|
Just print a reference to use --help for further details. Which stops
annoying experienced users, or users that made a typo on the
command-line, by not overwhelming them with screenfuls of help output.
Closes: #681371
|
|
The dpkg implementation was replaced by the GNU texinfo implementation
some time ago, through a wrapper used to ease the transition. Now long
complete, it's time to clean up.
|
|
Fixes ControlStructures::ProhibitCStyleForLoops.
Warned-by: perlcritic
|
|
Using double-quotes imposes a small performance penalty as the perl
parser needs to check if any interpolation is needed. Use double-quotes
only when the string contains single-quotes. Ideally we'd use
double-quotes too for escaped meta-characters that might otherwise be
confusing to immediately see if they need interpolation or not, but the
policy does not (currently) allow to ignore these.
Fixes ValuesAndExpressions::ProhibitInterpolationOfLiterals.
Warned-by: perlcritic
|
|
Addresses ValuesAndExpressions::ProhibitInterpolationOfLiterals.
Warned-by: perlcritic
|
|
Do not quote the keys for any hash which will always use simple
identifiers, as that is taken care of by the => operator. So this does
not apply to overload pragmas, version operators, field names, substvars,
command line options or filenames.
Addresses ValuesAndExpressions::ProhibitInterpolationOfLiterals.
Warned-by: perlcritic
|
|
Fixes Subroutines::ProhibitReturnSort.
Warned-by: perlcritic
|
|
Fixes InputOutput::ProhibitBarewordFileHandles.
Warned-by: perlcritic
|
|
This marks all these files as script files, so that tools like
perlcritic do not misrecognize them as modules.
|
|
|
|
These just clutter the code base, as adding modelines for each possible
editor out there does not scale, and they are currently not exhaustive
anyway.
|
|
|
|
This makes sure that if there are several alternatives with the same
priority, the current one will always be preferred, regardless of how
they sort alphabetically.
Closes: #699807
|
|
We'll make alternative_get_current() use alternative_get_best(),
so to avoid a forward declaration let's move it afterwards.
|
|
This will allow to call alternative_get_current() multiple times w/o
needing to perform useless work. It also gets rid of some annoying
resource handling.
|
|
This turns spawn() into a simpler wrapper around execvp(), and gets rid
of the code needed to generate the new argv.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
This reverts 18d9373b27effd533d386fdf7bfde6e6b3782101.
|
|
Conflicts:
debian/changelog
|
|
The case of --slave with a non-install action was already handled, but
not the case of missing action.
LP: #1037431
|
|
|
|
The test suite got fixed to not hardcode binary paths, so that it
can work on other systems than Debian, where those binaries might
be located in a different place. Dynamically generate the expected
database file to match the file system layout.
Known to be affecting Gentoo and Mac OS X.
|
|
Conflicts:
debian/changelog
|
|
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>
|
|
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
|
|
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
|
|
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
|
|
The function contains an infinite loop that does not break out.
|
|
[guillem@debian.org:
- Change mode to 0666. ]
Closes: #675918
Signed-off-by: Guillem Jover <guillem@debian.org>
|
|
The only fundamental difference between these two functions is the base,
one being decimal and the other being zero.
Change the code to call parse_unsigned() with an explicit base.
|
|
Verify that the numbers are not out of the range; i.e. that no negative
values are allowed if not appropriate, and that no overflows occur.
Closes: #580038
|
|
Verify that the string is not empty or does not contain trailing junk.
|
|
|
|
Autoconf provides an AC_PROG_MKDIR_P macro defining MKDIR_P which is
called by AM_INIT_AUTOMAKE; the obsolete mkdir_p, currently aliased to
MKDIR_P will disappear with automake 1.13.
|
|
|