Hi, On 2024-05-23 23:52, Louis-Philippe Véronneau wrote:
On 2024-05-22 16:36, Nilesh Patra wrote: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 listed in dpkg-buildpackage(1) (parallel=n, nocheck, noopt, nostrip, terse, hardening=..., reproducible=..., abi=..., future=..., qa=..., optimize=..., sanitize=...) and https://wiki.debian.org/BuildProfileSpec: nodocI concur. Thanks also to Andrius for +1. If Pollo/Andrius would like to work onI don't really mind the improvements proposed, but I don't have time to implement them.If no one has, I would suggest merging the code I wrote, as I believe it's better than the status quo :)
Since your MR performs a subset of the proposed general check for DEB_BUILD_OPTIONS, I think it can be merged right away and later extended to cover all known DEB_BUILD_OPTIONS. One suggestion would be to use a more generic lintian tag (something like 'debian-rules-invalid-build-profile') so as we can extend it later on instead of creating a new one and superseding your 'debian-rules-invalid-build-profile-nodocs'.
What do you think? Andrius