Age | Commit message (Collapse) | Author | Files | Lines |
|
Addresses Subroutines::RequireArgUnpacking.
Warned-by: perlcritic
|
|
|
|
Fixes InputOutput::RequireBracedFileHandleWithPrint.
Warned-by: perlcritic
|
|
This will make it safe to remove parenthesis around builtins.
|
|
Fixes CodeLayout::ProhibitHardTabs.
Warned-by: perlcritic
|
|
|
|
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
|
|
This gives a nicer error message, and makes sure we always exit on
option parse errors.
|
|
|
|
Rename, uppercase and do not export new variables by default, to avoid
cluttering the caller namespace, and to give them consistent names. We'll
keep the old variables for a while, to not break uses from external
modules.
Addresses Variables::ProhibitPackageVars.
|
|
Fixes Variables::ProhibitUnusedVariables.
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
|
|
|
|
Fixes InputOutput::ProhibitBarewordFileHandles.
Warned-by: perlcritic
|
|
Print short options before long ones.
|
|
|
|
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.
|
|
|
|
|
|
Having to keep this information twice is error-prone as it easily gets
out of sync. Having to translate it is bothersome. It's not consistent
across dpkg tools, some do print it some don't. It's currently not
accurate, as the output would need to include the holders for all files
that end up being part of the binary. And listing it in the --versions
output is visually annoying and the wrong place.
Just keep this where it belongs, at the file comment headers, above the
license information.
|
|
The usage of version_compare() has the side effect of erroring out as
soon as you encounter an invalid version. Replacing this call
with a comparison operator invoked on Dpkg::Version objects fixes
the issue.
Closes: #651993
Reported-by: Martin Packman <martin.packman@canonical.com>
|
|
Reported-by: Andrew Bennetts <andrew.bennetts@canonical.com>
|
|
Made dpkg-mergechangelogs check the return value of close and fail
if it was not successful. Previously dpkg-mergechangelogs would
silently fail to write a file if (e.g.) the device was full.
Signed-off-by: Raphaël Hertzog <hertzog@debian.org>
|
|
Remove trailing spaces. Remove blank lines not separating different code
blocks. Remove blank lines at the end of the file.
|
|
|
|
|
|
|