bug-standards
[Top][All Lists]
Advanced

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

Re: [gnu-prog-discuss] An experimental GNU Assembly


From: Thien-Thi Nguyen
Subject: Re: [gnu-prog-discuss] An experimental GNU Assembly
Date: Thu, 22 Dec 2011 18:19:53 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.0.50 (gnu/linux)

() Stefano Lattarini <address@hidden>
() Wed, 21 Dec 2011 09:52:55 +0100

   I have a few objections against making the format you propose the base of
   the ChangeLog entry format to be suggested in the GCS.  See my comments
   inline, below.

Thanks for looking it over.

   bug-standards perhaps?  (but I'm not any more sure that you are).

Fine w/ me.  CC changed.

   >       Most information about the change should be placed in the source
   >       code comments.  SHORT-PARAGRAPH is for referencing bug reports,
   >       giving credit (i.e., "Reported by" and "Suggested by" blurbs),
   >       and pointers to the origin of regressions
   >
   So your proposed format doesn't explicitly allow for a high-level, "global"
   explanation of what a change does, and perhaps even more importantly, why.
   This is an unacceptable limitation IMO.

After reading your explanation of this kind of text (in another part of
the thread), i have come to agree.  SHORT-PARAGRAPH should include
"motivation for change".  How about this?

 SHORT-PARAGRAPH is for describing the motivation for the change,
 i.e., "why", including perhaps a summary of the pre-change state.
 (Information about the post-change state should be placed in the
 source code comments.)  SHORT-PARAGRAPH should also reference bug
 reports, give credit (i.e., "Reported by" and "Suggested by"
 blurbs), and give pointers to the origin of regressions.

   > (using conventional format "Regression introduced YYYY-MM-DD [TITLE].")

   Some packages prefer to refer to the VCS revision as well (or exclusively).
   This is a point were we can give individual packages some slacks IMHO.
   WDYT?

Agreed.  I think ‘s/conventional/the suggested/’ is indicated, plus the
sentence: "(The intent of this format is to make it easy to search for
that particular ChangeLog entry as well as give an immediate sense of
how long the problem persisted.)"

   >         "U" means "Use ‘func’".
   >
   This just seems confusing to me; while I don't strongly object to individual
   packages using this "trick" if they find it useful, I'd rather not see it in
   an examples in the GCS.

Right, too ttn-specific.  Drop it.

   >       In this particular example, you can also write:
   > 
   >         Here, all C files now #include "header.h".
   >         * foo.c (foo): Use ‘func’.
   >         * bar.c (bar, baz): Likewise.
   >         (qux): Update call to ‘bar’.
   >         * doc.texi (ref): Mention "header.h".
   >
   This is what I'd do.

Then we can just drop the first part, or explain better the idea
behind it.

 ENTRY-CONVENTIONS describe entry-specific abbreviations or
 implicitly shared descriptions.  The idea is to factor out common
 bits of text, for readability, while preserving the full name of
 functions or other code elements.

   > ***** entries titled with suffix "; nfc."

   No strong opinion about this; but it seems to me that it might preferable to
   explicitly leave it to individual projects to decide whether to use this
   particular convention, or to always prefer something like:

           [maint] Add top-level file HACKING

           * HACKING: New file.

   instead.

Probably best to drop this entirely.

   > ***** conventions for TITLE
   > ******* tag
   >    To help identify the area of the change, include one or more
   >    tags in square braces at the beginning of the TITLE.
   >
   Many other gnu packages prefer the format:

      topic: brief description

   to the one you are proposing, that if I'm not mistaken is:

     [topic] brief description

   Should we try a poll of existing packages to decide which format to
   choose, or is this another area where to leave the packages some
   slack?

Hmm...

   >  For
   >    example, ‘maint’ for maintenance, ‘int’ for internal, ‘api’,
   >    ‘guile’, etc.  Because space in the TITLE is limited, tags
   >    should be short and (if more than one) separated by a space.
   >    For example:
   > 
   >    [guile int] Avoid deprecated libguile elements.

This example shows why i prefer tags: you can use them multiply.
Having said that, i realize that people see things differently.
How about adding:

 Alternatively, if there is no need for multiple tags, you can use
 the more succinct form:

    TOPIC: BRIEF-DESCRIPTION

This way, there is slack, but not too much slack.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]