Bug#909510: [PATCH] New tag: script-uses-unversioned-python-in-shebang

2019-02-13 Thread Dmitry Bogatov
[2019-02-11 19:23] Chris Lamb > > part text/plain 414 > Hi Dmitry, > > > Yes. Sorry, but I do not understand, how to test tags with severity > > 'classification'. It seems to not appear in output, that is > > compared with t/tags/checks/*/tags. > > Look at how, for example:

Bug#909510: [PATCH] New tag: script-uses-unversioned-python-in-shebang

2019-02-11 Thread Chris Lamb
Hi Dmitry, > Yes. Sorry, but I do not understand, how to test tags with severity > 'classification'. It seems to not appear in output, that is > compared with t/tags/checks/*/tags. Look at how, for example: t/tags/checks/scripts/scripts-maintainer-script-classification … works. :)

Bug#909510: [PATCH] New tag: script-uses-unversioned-python-in-shebang

2019-02-11 Thread Dmitry Bogatov
[2019-02-10 12:46] Chris Lamb > > part text/plain 238 > Hi Dmitry, > > > +Severity: classification > > Replying quickly, excuse terseness; you appear to have removed the > tests? Yes. Sorry, but I do not understand, how to test tags with severity 'classification'. It

Bug#909510: [PATCH] New tag: script-uses-unversioned-python-in-shebang

2019-02-10 Thread Chris Lamb
Hi Dmitry, > +Severity: classification Replying quickly, excuse terseness; you appear to have removed the tests? Regards, -- ,''`. : :' : Chris Lamb `. `'` la...@debian.org  chris-lamb.co.uk `-

Bug#909510: [PATCH] New tag: script-uses-unversioned-python-in-shebang

2019-02-09 Thread Dmitry Bogatov
[2019-02-07 16:34] Chris Lamb > > [Dmitry Bogatov] > > Sorry, I understood phrase "get numbers" as [...] > > I think I've confused you further now, alas. We can use Lintian to > accurate discover these numbers — my point is basically that it > cannot be "Severity: wishlist" as this would be

Bug#909510: [PATCH] New tag: script-uses-unversioned-python-in-shebang

2019-02-07 Thread Chris Lamb
Dmitry, > > > +Severity: wishlist > > > > Again, weren't we just doing this to get numbers at first? […] > Sorry, I understood phrase "get numbers" as > > let's create check, set severity to low and see numbers on > lintial.debian.org > > According to codesearch, there is 18

Bug#909510: [PATCH] New tag: script-uses-unversioned-python-in-shebang

2019-02-07 Thread Dmitry Bogatov
[2019-02-05 18:31] Chris Lamb > > part text/plain 432 > Hi Dmitry, > > > +Tag: script-uses-unversioned-python-in-shebang > > +Severity: wishlist > > Again, weren't we just doing this to get numbers at first? Unless > I'm missing something (quite possible!) I do not believe

Bug#909510: [PATCH] New tag: script-uses-unversioned-python-in-shebang

2019-02-05 Thread Chris Lamb
Hi Dmitry, > +Tag: script-uses-unversioned-python-in-shebang > +Severity: wishlist Again, weren't we just doing this to get numbers at first? Unless I'm missing something (quite possible!) I do not believe we are at a point in time where we wish to be implicitly asking maintainers to "fix" these

Bug#909510: [PATCH] New tag: script-uses-unversioned-python-in-shebang

2019-02-04 Thread Dmitry Bogatov
* checks/python.desc: Add description for new tag * checks/python.pm: check shebang of script for references to unversioned python, either directly or via /usr/bin/env. * t/tags/tests/python-script-uses-unversioned-python-in-shebang: new test directory. --- checks/python.desc

Bug#909510: [PATCH] New tag: script-uses-unversioned-python-in-shebang

2019-02-02 Thread Chris Lamb
Hi Dmitry, > +Tag: script-uses-unversioned-python-in-shebang > +Severity: normal ^^ Hmm, I thought we were just going to use this to see the numbers rather than ask maintainers to "fix" this? > +# Check for unversioned python shebang (#909510). > +for my $file

Bug#909510: [PATCH] New tag: script-uses-unversioned-python-in-shebang

2019-02-02 Thread Dmitry Bogatov
* checks/python.desc: Add description for new tag * checks/python.pm: check shebang of script for references to unversioned python, either directly or via /usr/bin/env. * t/tags/tests/python-script-uses-unversioned-python-in-shebang: new test directory. --- checks/python.desc