Louis-Philippe Véronneau pushed to branch bts_1110969 at lintian / lintian
Commits:
a88b2fea by Louis-Philippe Véronneau at 2025-09-08T13:16:25+00:00
Bump the uses-debhelper-compat-file tag from pedantic to warning
The debhelper-compat virtual package has been available for a long time
now and is the recommended way to set the debhelper compat level.
Having multiple concurrent ways to set this value is confusing and makes
it way harder to do QA consistently.
- - - - -
74c4efd5 by Louis-Philippe Véronneau at 2025-09-08T13:16:25+00:00
rework the uses-debhelper-compat-file tag message and add mention of breakage
in dh14
- - - - -
2673c1de by Louis-Philippe Véronneau at 2025-09-08T13:18:35+00:00
Try to decode font file using utf8 if it fails with cp1252.
Some font files are not cp1252 (for example latex-cjk-chinese-arphic-gbsn00lp),
which crashed the test.
Closes: #1110969
- - - - -
fada7a62 by Louis-Philippe Véronneau at 2025-09-08T13:18:35+00:00
add missing space
- - - - -
b59ab77a by Louis-Philippe Véronneau at 2025-09-08T13:18:35+00:00
use try-catch again instead of eval, not to fail silently
- - - - -
3 changed files:
- lib/Lintian/Check/Fonts/Postscript/Type1.pm
- t/recipes/runner-features/runtests-options/eval/literal
- tags/u/uses-debhelper-compat-file.tag
Changes:
=====================================
lib/Lintian/Check/Fonts/Postscript/Type1.pm
=====================================
@@ -50,14 +50,24 @@ sub visit_installed_files {
my @command = ('t1disasm', $item->unpacked_path);
my $bytes = safe_qx(@command);
+ my $enc_warning = 'In file ' . $item->name . $COLON . $SPACE;
my $output;
try {
# iso-8859-1 works too, but the Font 1 standard could be older
$output = decode('cp1252', $bytes, Encode::FB_CROAK);
- } catch {
- die 'In file ' . $item->name . $COLON . $SPACE . $@;
+ } catch ($e) {
+ if ($e =~ m{^cp1252 "\\x81" does not map to Unicode at .+}) {
+ try {
+ # sometimes, the file is utf8
+ $output = decode('utf8', $bytes, Encode::FB_CROAK);
+ } catch {
+ die $enc_warning . $@;
+ }
+ } else {
+ die $enc_warning . $@;
+ }
}
my @lines = split(/\n/, $output);
=====================================
t/recipes/runner-features/runtests-options/eval/literal
=====================================
@@ -2,4 +2,4 @@ E: runtests-options source: invalid-standards-version 1.2.3
E: runtests-options: new-essential-package
W: runtests-options: empty-binary-package
W: runtests-options source: no-versioned-debhelper-prerequisite 13
-P: runtests-options source: uses-debhelper-compat-file [debian/compat]
+W: runtests-options source: uses-debhelper-compat-file [debian/compat]
=====================================
tags/u/uses-debhelper-compat-file.tag
=====================================
@@ -1,14 +1,18 @@
Tag: uses-debhelper-compat-file
-Severity: pedantic
+Severity: warning
Check: debhelper
-Explanation: This package uses a <code>debian/compat</code> file to denote the
- required debhelper compatibility number.
+Explanation: This package declares its debhelper compatibility level with the
+ <code>debian/compat</code> file.
.
- However, debhelper has replaced <code>debian/compat</code> with the
- <code>debhelper-compat</code> virtual package for most circumstances.
+ The recommended way to do so is to use the virtual package
+ <code>debhelper-compat</code> instead.
.
- Packages not using an experimental or beta compatibility level should
- Build-Depend on the <code>debhelper-compat</code> virtual package, for
+ From debhelper 14 onward, using <code>debian/compat</code> will not work
+ anymore.
+ .
+ As such, unless you need an experimental or beta compatibility level, please
+ remove the <code>debian/compat</code> file and add the
+ <code>debhelper-compat</code> virtual package to your Build-Depends, for
example:
.
Build-Depends: debhelper-compat (= 13)
View it on GitLab:
https://salsa.debian.org/lintian/lintian/-/compare/7b5c6f47f8bdf09fa5731543fbaad61c670a139b...b59ab77a9056eed907043b6b661d808adef36299
--
View it on GitLab:
https://salsa.debian.org/lintian/lintian/-/compare/7b5c6f47f8bdf09fa5731543fbaad61c670a139b...b59ab77a9056eed907043b6b661d808adef36299
You're receiving this email because of your account on salsa.debian.org.
