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
