Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.debian.bugs.dist > #1019357

Bug#725484: A way to override blhc false-positives

Show key headers only | View raw

Em dom., 26 de jul. de 2020 às 03:27, <simon@ruderich.org> escreveu:
> my first approach was that all false-positives should be
> handled/fixed in blhc so that nobody can simply "ignore" missing
> flags. But this approach doesn't scale.
>
> Please have a look at the attached patch. It permits embedding
> the "blhc: ignore-line-regexp: REGEXP" in the build log. All
> lines (fully) matching REGEXP are then ignored (just like the
> --ignore-line option). What do you think?
>
> Eriberto: Would this also work for the CI issues you mentioned in
> #962990?

Hi Simon,

My last suggestion in #962990 is inappropriate because some systems in
Debian, as Salsa CI pipelines, run blhc automatically. Please, see
examples here[1][2][3][4].

[1] https://salsa.debian.org/debian/blhc/-/pipelines/158444
[2] https://salsa.debian.org/debian/blhc/-/blob/debian/master/debian/salsa-ci.yml
[3] https://salsa.debian.org/debian/ngetty/-/pipelines/149456
[4] https://salsa.debian.org/debian/ngetty/-/blob/debian/master/debian/salsa-ci.yml

A manual option as --ignore-line will work for a final user but will
fail for automated systems as Salsa. So, I suggest a configuration
file in /etc/ with some rules. Thus, we will can send new rules to
allow you to release new versions with preinstalled rules. I think
this config file can have two sections, as shown below:

[GENERAL]
# To make blhc ignore this file, change to false.
enable = true

# To disable any blhc option below, change to false.
all = true
debian = true
color = true

[IGNORE]
# All lines matching with the regex shown below will be ignored by blhc.

# blhc
/t/logs/ignore-line --ignore-line.+prepare-script gcc test

# ngetty
^CC = diet -Os gcc -W$


Thanks a lot for your efforts to help.

Regards,

Eriberto

Back to linux.debian.bugs.dist | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

Bug#725484: A way to override blhc false-positives simon@ruderich.org - 2020-07-26 08:40 +0200
  Bug#725484: A way to override blhc false-positives Eriberto Mota <eriberto@debian.org> - 2020-07-26 17:30 +0200
    Bug#725484: A way to override blhc false-positives Eriberto <eriberto@eriberto.pro.br> - 2020-07-26 22:10 +0200
      Bug#725484: A way to override blhc false-positives simon@ruderich.org - 2020-07-27 08:20 +0200
        Bug#725484: A way to override blhc false-positives Eriberto <eriberto@eriberto.pro.br> - 2020-07-29 03:50 +0200
        Bug#725484: A way to override blhc false-positives Eriberto <eriberto@eriberto.pro.br> - 2020-07-29 04:00 +0200
          Bug#725484: A way to override blhc false-positives simon@ruderich.org - 2020-07-29 08:10 +0200

csiph-web