On Sat, Oct 14, 2017 at 11:15:10AM -0700, Sean Whitton wrote: > I am seeking seconds for the following patch.
Thank you for working on this! > - I've included the ability to specify the architectures on which the > package is known /not/ to build. This seems useful because in many > cases the architectures that don't work is precisely what the > maintainer knows, and wants to document in machine-readable form. I > don't buy the argument that it creates any additional uncertainty Whilst you did this, I think the proper syntax is not clear by your wording. For example, I take that Build-Indep-Architecture: !amd64 means "this thing builds everywhere except amd64". But then, how can I specify "it builds on all 32-bit architectures except i386", for example? See the following inline comments: > +Debian machine architectures. If specified, it should be either > + > +- A unique single word identifying a Debian machine architecture as ^^^^^^^^^^^^^^^^^^ > + described in :ref:`s-arch-spec`. > + > +- An architecture wildcard identifying a set of Debian machine > + architectures, see :ref:`s-arch-wildcard-spec`. This also implies a single word. But… > +maintainer's control. The specification should entail that the > +architecture-independent packages are buildable on at least two > +architectures. this is not compatible with the above: both cases say that you have to provide at most one word, but how can you provide at least two architectures if you have only one word and can't use a wildcard? And if you can provide multiple words, you need to specify a separator (whitespace vs comma, I suppose). Also, I dislike the sentence in itself, I believe it should be more straightforward in conveying its meaning of "pretty please at least two arch". -- regards, Mattia Rizzolo GPG Key: 66AE 2B4A FCCF 3F52 DA18 4D18 4B04 3FCD B944 4540 .''`. more about me: https://mapreri.org : :' : Launchpad user: https://launchpad.net/~mapreri `. `'` Debian QA page: https://qa.debian.org/developer.php?login=mattia `-
signature.asc
Description: PGP signature