Package: devscripts
Version: 2.25.29
Severity: normal
File: /usr/bin/sadt
Tags: patch

src:libsdl3 contains a debian/tests/control that has had its format 
canonicalized with `debputy reformat` (similar to `wrap-and-sort 
-abst`), with stanzas that look like this:

Tests:
 build,
Depends:
 build-essential,
 clang:native,
 libsdl3-dev,
 pkgconf,
Restrictions:
 allow-stderr,
 superficial,
(etc.)

Trying to run this with sadt produces messages indicating that it has 
been mis-parsed, with the trailing commas treated as part of the token 
rather than part of the separator:

cannot parse package relationship "", returning it raw
cannot parse package relationship "", returning it raw
: SKIP (unknown restriction: superficial,)
build,: SKIP (unknown restriction: superficial,)
(etc.)

The autopkgtest specification 
https://salsa.debian.org/ci-team/autopkgtest/-/blob/master/doc/README.package-tests.rst?ref_type=heads
 
says that Tests, Features and Restrictions are "separated by commas 
and/or whitespace". The implementation used in autopkgtest is to replace 
all commas with spaces, then split on whitespace, discarding any leading 
or trailing empty tokens. I've opened 
https://salsa.debian.org/debian/devscripts/-/merge_requests/607 to make 
sadt do the same.

    smcv

Reply via email to