On Sun, 2010-06-13 at 19:00 -0700, Russ Allbery wrote:
> Helps if I send this to the correct bug.
>
> Russ Allbery <rra@debian.org> writes:
>
> > * maintainer-name-missing and uploader-name-missing are both automatic
> > rejects in the ftp-master checks, which makes them automatically
> > severity: serious in Lintian. That's not the specific one that you're
> > asking about, but that's the rule that Changed-By references.
>
> > * The Policy description for Changed-By says "The name and email address
> > of the person who changed the said package." That's not a should.
> > That's a statement of what that field shall include, which means that if
> > it doesn't have the name and e-mail address, it's a syntax error and
> > therefore is a violation of an implicit must.
>
> > I see where your reading is coming from, but suspect the best fix is to
> > just change the Policy wording to make it clear that this is a must.
> > There's really no reason to use a different format, and Debian elsewhere
> > already requires names.
>
> Here's a proposed patch that cleans up the wording of Maintainer,
> Uploaders, and Changed-By to reflect current practice. There is another
> outstanding bug in this area to document further restrictions on
> Maintainer and Uploaders, but this is the easy part so I wanted to resolve
> this first.
>
> The following patch tightens the syntax of Maintainer to a must, tightens
> the use of comma as a separator in Uploaders to a must, permits people to
> use multi-line Uploaders fields (we were waiting for the lenny release),
> and is explicit that the syntax of Changed-By is the same as Maintainer
> and is a bit clearer about what goes into that field.
>
> Objections or seconds?
>
> diff --git a/policy.sgml b/policy.sgml
> index df6ae89..5a76cf3 100644
> --- a/policy.sgml
> +++ b/policy.sgml
> @@ -2672,7 +2672,7 @@ Package: libc6
>
> <p>
> The package maintainer's name and email address. The name
> - should come first, then the email address inside angle
> + must come first, then the email address inside angle
> brackets <tt><></tt> (in RFC822 format).
Missing semicolon:
brackets <tt><></tt> (in RFC822 format).
> </p>
>
> @@ -2690,17 +2690,16 @@ Package: libc6
> <sect1 id="f-Uploaders">
> <heading><tt>Uploaders</tt></heading>
>
> - <p>
> - List of the names and email addresses of co-maintainers of
> - the package, if any. If the package has other maintainers
> - beside the one named in the
> - <qref id="f-Maintainer">Maintainer field</qref>, their
> - names and email addresses should be listed here. The
> - format is the same as that of the Maintainer tag, and
> - multiple entries should be comma separated. Currently,
> - this field is restricted to a single line of data. This
> - is an optional field.
> - </p>
> + <p>
> + List of the names and email addresses of co-maintainers of
> + the package, if any. If the package has other maintainers
> + beside the one named in the
> + <qref id="f-Maintainer">Maintainer field</qref>, their names
> + and email addresses should be listed here. The format is the
> + same as that of the Maintainer tag, and multiple entries must
> + be comma separated. This is an optional field.
> + </p>
> +
> <p>
> Any parser that interprets the Uploaders field in
> <file>debian/control</file> must permit it to span multiple
> @@ -2714,9 +2713,10 @@ Package: libc6
> <heading><tt>Changed-By</tt></heading>
>
> <p>
> - The name and email address of the person who changed the
> - said package. Usually the name of the maintainer.
> - All the rules for the Maintainer field apply here, too.
> + The name and email address of the person who prepared this
> + version of the package, usually a maintainer. The syntax is
> + the same as for the <qref id="f-Maintainer">Maintainer
> + field</qref>.
> </p>
> </sect1>
Seconded.
Presuming you've fixed the missing semicolon that Sean spotted :-)
Cheers,
Andrew.
--
------------------------------------------------------------------------
andrew (AT) morphoss (DOT) com +64(272)DEBIAN
You are taking yourself far too seriously.
------------------------------------------------------------------------
Attachment:
signature.asc
Description: This is a digitally signed message part