Bug#1070770: lintian: check for testing presence of "nodocs" in DEB_BUILD_OPTIONS

2024-06-03 Thread Andrius Merkys
On 2024-06-02 19:49, Louis-Philippe Véronneau wrote:> Sounds like a plan. I made the changed you proposed and also made sure the tag will output the problematic BUILD_OPTION. That way, when/if someone wants to make it more generic, it'll be possible to pass other invalid BUILD_OPTION. The tag

Bug#1070770: lintian: check for testing presence of "nodocs" in DEB_BUILD_OPTIONS

2024-05-22 Thread Nilesh Patra
Hi Julian! On Sun, May 19, 2024 at 01:48:17PM +0100, Julian Gilbey wrote: > But here I'd suggest doing the > opposite: checking for valid build options (and note: this is a check > for DEB_BUILD_OPTIONS, not for DEB_BUILD_PROFILES). There is a very > short list of standard build options: those

Bug#1070770: lintian: check for testing presence of "nodocs" in DEB_BUILD_OPTIONS

2024-05-20 Thread Andrius Merkys
On 2024-05-19 15:48, Julian Gilbey wrote: But here I'd suggest doing the opposite: checking for valid build options (and note: this is a check for DEB_BUILD_OPTIONS, not for DEB_BUILD_PROFILES). There is a very short list of standard build options: those listed in dpkg-buildpackage(1)

Bug#1070770: lintian: check for testing presence of "nodocs" in DEB_BUILD_OPTIONS

2024-05-19 Thread Julian Gilbey
Hi Nilesh, On Sun, May 19, 2024 at 12:27:02PM +0530, Nilesh Patra wrote: > Julian Gilbey : > > I have come across a number of packages with a line in their > > debian/rules like: > > > > ifeq (,$(findstring nodocs, $(DEB_BUILD_OPTIONS))) > > > > This should be "nodoc", according to the

Bug#1070770: lintian: check for testing presence of "nodocs" in DEB_BUILD_OPTIONS

2024-05-19 Thread Nilesh Patra
Julian Gilbey : > I have come across a number of packages with a line in their > debian/rules like: > > ifeq (,$(findstring nodocs, $(DEB_BUILD_OPTIONS))) > > This should be "nodoc", according to the "nodoc" entry in > https://wiki.debian.org/BuildProfileSpec#Registered_profile_names >

Bug#1070770: lintian: check for testing presence of "nodocs" in DEB_BUILD_OPTIONS

2024-05-09 Thread Julian Gilbey
On Thu, May 09, 2024 at 01:13:45PM -0400, Louis-Philippe Véronneau wrote: > tags 1070770 patch > thanks > > I've created a patch on Salsa that creates a new Lintian check for this. > > https://salsa.debian.org/lintian/lintian/-/merge_requests/504 Amazing, thanks! I've added a bunch of

Bug#1070770: lintian: check for testing presence of "nodocs" in DEB_BUILD_OPTIONS

2024-05-09 Thread Louis-Philippe Véronneau
tags 1070770 patch thanks I've created a patch on Salsa that creates a new Lintian check for this. https://salsa.debian.org/lintian/lintian/-/merge_requests/504 Cheers, -- ⢀⣴⠾⠻⢶⣦⠀ ⣾⠁⢠⠒⠀⣿⡁ Louis-Philippe Véronneau ⢿⡄⠘⠷⠚⠋ po...@debian.org / veronneau.org ⠈⠳⣄

Bug#1070770: lintian: check for testing presence of "nodocs" in DEB_BUILD_OPTIONS

2024-05-08 Thread Julian Gilbey
Package: lintian Version: 2.117.0 Severity: wishlist I have come across a number of packages with a line in their debian/rules like: ifeq (,$(findstring nodocs, $(DEB_BUILD_OPTIONS))) This should be "nodoc", according to the "nodoc" entry in