Re: [PATCH] Added-support-for-multi-version-arch-in-temporary-labs

[Niels Thykier <niels@thykier.net>]

On 2011-07-20 22:43, Russ Allbery wrote:
> Niels Thykier <niels@thykier.net> writes:
> 

Hi

>> [...]
>> While we could leave out parts that are redundant when (parts of) the
>> feature is not used, I suspect it may make output parsers more
>> complicated (as they would need to know / deduce this).
>>   On the other hand, it may make output needlessly long.
> 
> Yeah, I'd kind of like to keep the output as short as possible for the
> common case, since it's nice to be able to glance at the tag and see the
> important information quickly.
> 
> What about something like:
> 
>     E: package source (1.0-1) [amd64]: some-tag
> 
> where the type is present or dropped based on the rules we have now, and
> the version and architecture are only present if necessary to
> disambiguate?  That has the advantage of using the same punctuation the
> project uses elsewhere for versions and architectures, which means the
> well-trained eye of a Debian developer will parse it easily.
> 

I have committed my patch more or less as is since it restored status
quo.  I will try to update the tag output so it is more useful soon.

Generally I like the output suggestion, so I will go with that.  When
you say that they should only be present "if necessary to disambiguate",
are we looking at:

 $ lintian pkg1.dsc pkg2_1.0-1.dsc pkg2_1.0-2.dsc
 W: pkg1 source: some-tag
 W: pkg2 source (1.0-1): some-tag
 W: pkg2 source (1.0-2): some-other-tag

Or would that just be versions on all three tags?

~Niels