[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Bug#932753: tag2upload should record git tag signer info in .dsc [and 1 more messages]



Russ Allbery writes ("Bug#932753: tag2upload should record git tag signer info in .dsc [and 1 more messages]"):
> One thing that jumps out at me here is that this field isn't extensible,
> since anything after the first space-separated word has to be taken to be
> the tagger line.

Hi.  Thanks for the review.

How hilarious that you are making this point back to me now here :-).

> Unfortunately , doing something extensible within the field requires adding
> a separator, which in turn requires dealing with escaping, and thus is
> kind of a mess.  Given that, what if you instead used two fields:
> 
>     Git-Tag-Info: fingerprint=FINGERPRINT
>     Git-Tag-Tagger: Firstname Surname <email@address>

This LGTM if other people like the look.

> That also allows the tagger field to be defined has having the same syntax
> as Maintainer (or one of the other existing RFC-2822-style fields we
> have), which I think increases the chances that parsers will get this
> right.

Right.  I already need and have code to turn a git-style
committer/author/tagger line into deb822 form.

Ian.

-- 
Ian Jackson <ijackson@chiark.greenend.org.uk>   These opinions are my own.

If I emailed you from an address @fyvzl.net or @evade.org.uk, that is
a private address which bypasses my fierce spamfilter.


Reply to: