]> git.proxmox.com Git - mirror_ubuntu-eoan-kernel.git/commitdiff
UBUNTU: [Packaging]: config-check: some enforcement is ignored
authorThadeu Lima de Souza Cascardo <cascardo@canonical.com>
Thu, 19 Dec 2019 18:29:00 +0000 (19:29 +0100)
committerMarcelo Henrique Cerri <marcelo.cerri@canonical.com>
Fri, 24 Jan 2020 12:36:43 +0000 (09:36 -0300)
Whenever config-check finds an entry like:

CONFIG_VFIO mark<ENFORCED>note<LP#1636733>

it will consider mark<ENFORCED>note as the option name, as the regexp allows
any characters not in the whitespace class (\S). By using a class like [^\s<],
that is, not whitespace and not '<', it will be correctly parsed.

We could fix the entries that have this problem, but fixing the parser is more
robust.

Ignore: yes

Signed-off-by: Thadeu Lima de Souza Cascardo <cascardo@canonical.com>
Acked-by: Marcelo Henrique Cerri <marcelo.cerri@canonical.com>
Acked-by: Kleber Sacilotto de Souza <kleber.souza@canonical.com>
Signed-off-by: Kleber Sacilotto de Souza <kleber.souza@canonical.com>
debian/scripts/config-check

index 843f5999006b61a339ff42119409911702865b15..95edd39dc407f3d86c2cdc1d9b1a53f2cf4d40a7 100755 (executable)
@@ -113,7 +113,7 @@ for $config (keys %annot) {
        $options = $annot{$config};
 
        $policy = undef;
-       while ($options =~ /\s*(\S+)<(.*?)?>/g) {
+       while ($options =~ /\s*([^\s<]+)<(.*?)?>/g) {
                ($option, $value) = ($1, $2);
 
                if ($option eq 'mark' && $value eq 'ENFORCED') {