pkgsrc-Users archive

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

Re: pkgsrc Commit Message Policy



On 09/18, Thomas Klausner wrote:
> On Tue, Sep 12, 2017 at 03:40:00PM +0200, Thomas Klausner wrote:
> > For several years, there have been mirrors of pkgsrc in fossil, git,
> > and hg. Standard practise when using these tools is to make the first
> > line of a commit message function as a summary that can be read
> > without the rest, such as is commonly done with "git log --oneline".
> > Note that having a new commit message policy does not imply that a
> > decision to move to a DVCS has been made, but simply that we are now
> > in a world where DVCS usage is pervasive, even for repositories that
> > are authoritatively in CVS.
> > 
> > For this reason, pkgsrc-pmc has decided on the following guidelines
> > for pkgsrc commit messages:
> > 
> > Start the commit message with a line that explains the big picture in
> > 50 characters or less. When a commit is for one package, include the
> > name of the package. For updates, include the version to which it is
> > updated.
> 
> After listening to feedback, the limit has been increased to 65:
> 
> Start the commit message with a line that explains the big picture in
> 65 characters or less. When a commit is for one package, include the
> name of the package. For updates, include the version to which it is
> updated.

Hi, Thomas!

I'm not against a limit of 65 characters, but I am curious, would you
mind sharing the rationale behind it?  For example, maybe the initial
idea of 50 characters came from Tim Pope's "A Note About Git Commit
Messages" [1] blog post?  The rationale there seems to be that the
commit summary is used in various parts of Git output, and so a short
length for the summary makes that output look nicer by avoiding wrapping
or truncating.  With the pkgsrc-pmc guideline to include the name of the
package in the commit message if the commit is for a single package,
maybe a concern was that 50 characters could be challenging when the
package name eats up part of that right off the bat?  Again, just
curious.

Thanks!

Lewis

[1] http://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html


Home | Main Index | Thread Index | Old Index