unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Lars Magne Ingebrigtsen <larsi@gnus.org>
Cc: emacs-devel@gnu.org
Subject: Re: Ensuring commit messages with correct copyright info
Date: Tue, 08 Mar 2016 21:22:39 +0200	[thread overview]
Message-ID: <8360wwrczk.fsf@gnu.org> (raw)
In-Reply-To: <m3y49shkn2.fsf@gnus.org> (message from Lars Magne Ingebrigtsen on Tue, 08 Mar 2016 19:47:29 +0100)

> From: Lars Magne Ingebrigtsen <larsi@gnus.org>
> Date: Tue, 08 Mar 2016 19:47:29 +0100
> 
> The thing that comes up, again and again, as the main reason we should
> be able to edit commit messages (and therefore we should have
> ChangeLogs, because we can edit those) is that we (er, mainly me :-))
> sometimes forget to put
> 
> Copyright-paperwork-exempt: yes
> 
> into the commit messages.

It's one problem, but it's not the only one.  Just look at the
generated ChangeLog file, and you will see the blunders.  We need to
educate contributors to do a better job, and I see no way of doing
that without some means of fixing the mistakes.

> So why not concentrate of fixing that particular problem?  I think
> something that might help is that if the committer is different from the
> author, then there should always be something in the commit message that
> explicitly says what the copyright status of the work is.
> 
> That is, unless the line
> 
> Copyright-paperwork-exempt: yes
> 
> or the line
> 
> Copyright-assigned: yes
> 
> is present in the commit message, then the commit... hooks? (the thing
> that refuses to check in files with trailing newlines etc; I don't know
> the details about how that works) should refuse the commit.
> 
> Then it should be pretty impossible to forget this stuff, and there's
> one less reason to want to edit the commit messages.

So you are suggesting that instead of doing TRT in perhaps 2% of
commit messages, we should now add a completely redundant text in 98%
of them?  How does this make sense?

> (Note: This hook should only be run if there's an Author: header in the
> vc log buffer.  Everybody who has commit rights have already assigned
> copyright, so it's unnecessary to add those if the committer and the
> author are the same person.)

I already use --author in those cases, why should I do anything else?



  parent reply	other threads:[~2016-03-08 19:22 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-03-08 18:47 Ensuring commit messages with correct copyright info Lars Magne Ingebrigtsen
2016-03-08 18:50 ` Lars Magne Ingebrigtsen
2016-03-08 19:22 ` Eli Zaretskii [this message]
2016-03-08 19:25   ` Lars Magne Ingebrigtsen
2016-03-08 19:58     ` Eli Zaretskii
2016-03-08 20:02       ` Lars Magne Ingebrigtsen
2016-03-08 21:22       ` John Wiegley

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=8360wwrczk.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=larsi@gnus.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).