Bug#917120: lintian: Lintianrc should contain valid configuration

2018-12-23 Thread Chris Lamb
retitle 917120 lintian: misleading line format description in lintianrc
thanks

Salvo,

> # Also note, that this file uses a special syntax:
> # Empty lines are allowed, comments are introduced by a hash sign (#).
> # All other lines must have the format
> #VAR=text
> # or
> #VAR="text"

I don't read this as forbidding spaces myself, but I would concede
that it could be ambiguous so very happy to update the comment.

For the avoidance of doubt, I was asking you to copy-paste exactly
what you were seeing and reading (text, errors, etc.) just in case we
had different file contents. When filing bugs, please do try and a) be
as explicit as possible and b) avoid making assumptions about what the
problem is (your bug title is misleading; the commented-out syntax is
perfectly valid).

We could have saved a number of round-trips at the very least. :)


Best wishes,

-- 
  ,''`.
 : :'  : Chris Lamb
 `. `'`  la...@debian.org / chris-lamb.co.uk
   `-



Bug#917120: lintian: Lintianrc should contain valid configuration

2018-12-23 Thread Salvo Tomaselli
# Also note, that this file uses a special syntax:
# Empty lines are allowed, comments are introduced by a hash sign (#).
# All other lines must have the format
#VAR=text
# or
#VAR="text"
# It is allowed to use `~' and `$HOME' in the variables, but not other
# shell/environment variables.

# Enable info tags by default (--display info)
#display-info = yes

# Limit the number of parallel unpacking jobs to X (--jobs)
#jobs = 8

# Enable pedantic tags by default (--pedantic)
#pedantic = yes


In the example syntax, spaces do not seem to be allowed.

Il giorno dom 23 dic 2018 alle ore 01:00 Chris Lamb 
ha scritto:
>
> Hi Salvo,
>
> > No error message, just that the comment at the top of the file states
> > that var assignments should have no spaces, and then all the example
> > assignments have spaces.
>
> I don't see any reference to spaces being illegal in this file. Please
> could you copy-paste what you are seeing? I'm struggling to see what
> the bug is here and we appear to be making a number of unnecessary
> round-trips. :)
>
>
> Regards,
>
> --
>   ,''`.
>  : :'  : Chris Lamb
>  `. `'`  la...@debian.org / chris-lamb.co.uk
>`-



-- 
Salvo Tomaselli

"Io non mi sento obbligato a credere che lo stesso Dio che ci ha dotato di
senso, ragione ed intelletto intendesse che noi ne facessimo a meno."
-- Galileo Galilei

http://ltworf.github.io/ltworf/



Bug#917120: lintian: Lintianrc should contain valid configuration

2018-12-22 Thread Chris Lamb
Hi Salvo,

> No error message, just that the comment at the top of the file states
> that var assignments should have no spaces, and then all the example
> assignments have spaces.

I don't see any reference to spaces being illegal in this file. Please
could you copy-paste what you are seeing? I'm struggling to see what
the bug is here and we appear to be making a number of unnecessary
round-trips. :)


Regards,

-- 
  ,''`.
 : :'  : Chris Lamb
 `. `'`  la...@debian.org / chris-lamb.co.uk
   `-



Bug#917120: lintian: Lintianrc should contain valid configuration

2018-12-22 Thread Salvo Tomaselli
No error message, just that the comment at the top of the file states
that var assignments should have no spaces, and then all the example
assignments have spaces.

Unless I am misinterpreting the meaning of the comment, but the
example syntax there has no spaces.

Il giorno dom 23 dic 2018 alle ore 00:24 Chris Lamb 
ha scritto:
>
> tags 917120 + moreinfo
> thanks
>
> Hi Salvo,
>
> > According to the comments at the beginning of the file, /etc/lintianrc uses
> > shell syntax (it gets sourced I presume).
>
> Hm? It's not shell, it's parsed with:
>
>   
> https://salsa.debian.org/lintian/lintian/blob/master/commands/lintian.pm#L1240
>
> > It would be better if people could just uncomment, rather than needing to 
> > fix
> > the syntax as well.
>
> What error message(s) are you seeing, specifically?
>
>
> Regards,
>
> --
>   ,''`.
>  : :'  : Chris Lamb
>  `. `'`  la...@debian.org / chris-lamb.co.uk
>`-



-- 
Salvo Tomaselli

"Io non mi sento obbligato a credere che lo stesso Dio che ci ha dotato di
senso, ragione ed intelletto intendesse che noi ne facessimo a meno."
-- Galileo Galilei

http://ltworf.github.io/ltworf/



Bug#917120: lintian: Lintianrc should contain valid configuration

2018-12-22 Thread Chris Lamb
tags 917120 + moreinfo
thanks

Hi Salvo,

> According to the comments at the beginning of the file, /etc/lintianrc uses
> shell syntax (it gets sourced I presume).

Hm? It's not shell, it's parsed with:

  https://salsa.debian.org/lintian/lintian/blob/master/commands/lintian.pm#L1240

> It would be better if people could just uncomment, rather than needing to fix
> the syntax as well.

What error message(s) are you seeing, specifically?


Regards,

-- 
  ,''`.
 : :'  : Chris Lamb
 `. `'`  la...@debian.org / chris-lamb.co.uk
   `-



Bug#917120: lintian: Lintianrc should contain valid configuration

2018-12-22 Thread Salvo Tomaselli
Package: lintian
Version: 2.5.117
Severity: wishlist

Dear Maintainer,

According to the comments at the beginning of the file, /etc/lintianrc uses
shell syntax (it gets sourced I presume).

However all the later commented example options do not use said valid syntax.

It would be better if people could just uncomment, rather than needing to fix
the syntax as well.

Best

-- System Information:
Debian Release: buster/sid
  APT prefers unstable
  APT policy: (500, 'unstable'), (500, 'testing')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 4.18.16 (SMP w/4 CPU cores; PREEMPT)
Locale: LANG=it_IT.UTF-8, LC_CTYPE=it_IT.UTF-8 (charmap=UTF-8), LANGUAGE= 
(charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

Versions of packages lintian depends on:
ii  binutils   2.31.1-11
ii  bzip2  1.0.6-9
ii  diffstat   1.62-1
ii  dpkg   1.19.2
ii  dpkg-dev   1.19.2
ii  file   1:5.34-2
ii  gettext0.19.8.1-9
ii  gpg2.2.12-1
ii  intltool-debian0.35.0+20060710.4
ii  libapt-pkg-perl0.1.34+b1
ii  libarchive-zip-perl1.64-1
ii  libcgi-pm-perl 4.40-1
ii  libclass-accessor-perl 0.51-1
ii  libclone-perl  0.41-1+b1
pn  libdigest-sha-perl 
ii  libdpkg-perl   1.19.2
ii  libemail-valid-perl1.202-1
ii  libfile-basedir-perl   0.08-1
ii  libipc-run-perl20180523.0-1
ii  liblist-moreutils-perl 0.416-1+b4
ii  libparse-debianchangelog-perl  1.2.0-13
ii  libtext-levenshtein-perl   0.13-1
ii  libtimedate-perl   2.3000-2
ii  liburi-perl1.74-1
ii  libxml-simple-perl 2.25-1
ii  libyaml-libyaml-perl   0.75+repack-1
ii  man-db 2.8.4-3
ii  patchutils 0.3.4-2
ii  perl   5.28.1-3
ii  t1utils1.41-2
ii  xz-utils   5.2.2-1.3

Versions of packages lintian recommends:
pn  libperlio-gzip-perl  

Versions of packages lintian suggests:
pn  binutils-multiarch 
ii  libhtml-parser-perl3.72-3+b3
pn  libtext-template-perl  

-- no debconf information