From: Radon Rosborough <radon.neon@gmail.com>
To: Eli Zaretskii <eliz@gnu.org>
Cc: Ted Zlatanov <tzz@lifelogs.com>,
Lars Ingebrigtsen <larsi@gnus.org>,
emacs-devel <emacs-devel@gnu.org>
Subject: Re: git history tracking across renames (and emacs support)
Date: Thu, 12 Jul 2018 22:33:13 -0600 [thread overview]
Message-ID: <CADB4rJHyokt_NSMs9siM1AxdtEMUiGpFw16Wkfk=priM1qDEFA@mail.gmail.com> (raw)
In-Reply-To: <83o9fce4np.fsf@gnu.org>
> Is it really an issue with magit? And if it is, why isn't it
> documented in magit's documentation?
In my opinion, it doesn't really make sense for Magit, a
general-purpose Git frontend, to document the specific contribution
guidelines for GNU projects.
Sure, Magit should document the functionality it provides, and I
believe it does provide ChangeLog-related functionality. But in that
case, the Emacs CONTRIBUTE should most assuredly link to that
documentation. It can't simply say "please figure it out yourself"!
Since GNU projects are using a nonstandard commit message formatting
from the perspective of the rest of the world, I think the burden is
on them to tell contributors about that format and how to produce it
(if indeed the format is too burdensome to write by hand, which I
believe the ChangeLog format is).
>> As soon as I am linked to documentation about editing
>> ChangeLog files, I will assume I've gone astray, unless it's explained
>> why editing ChangeLog files is necessary in order to generate a commit
>> message.
>
> I don't understand why. How about providing detailed critique of
> what CONTRIBUTE says under "Generating ChangeLog entries"?
On re-reading very carefully, I see that I read "write ChangeLog
entries" as an operation that one would do when creating a ChangeLog
file, whereas it was meant more generally.
So, yes, the misunderstanding is my fault. However, I think it would
be much clearer if there were actual, literal step-by-step
instructions on how to go from a working directory with some changes
in it to a properly formatted commit. Directing people to pore through
the manual just so they can commit to Emacs seems a bit burdensome to
me; isn't there a reason we have CONTRIBUTE? To make contributing as
easy as possible?
I'm sure somebody will say that I am just being lazy. But the fact
remains that Emacs is much harder to contribute to than other
open-source projects, because of things like this.
>> It seems to me that ChangeLog is a "GNU thing". As are mailing
>> lists, avoidance of GitLab, using debbugs, etc.
>
> Well, we are talking about Emacs, and Emacs will always be a GNU
> project.
*cough* https://github.com/Wilfred/remacs *cough*
But seriously though, just because GNU projects usually have
ChangeLogs doesn't mean that ChangeLogs are a good idea. If they are
such a good idea, then why don't any other projects have them?
> Nevertheless, log messages provided by contributors are pretty close
> to what I'd like to see, after a couple of initial submissions,
> where we generally need to point out some minor nits.
But what do you want to see? Auto-generated ChangeLog entries? If
that's really what you want, then it sounds like you aren't open to
any other format.
Personally, what I want to see in commit messages is a properly
formatted summary line, links to any related bug reports, and an
explanation of anything important to know about the change that can't
be put in comments. These things could be included or not regardless
of whether ChangeLog format is used. They are what *I* want to see;
what do you want to see?
> > I think the blog does a much better job of getting across its
> > information clearly, concisely, and understandably:
>
> It's too long, IMO.
Surely the seven-line bullet-point list is not too long, is it?
Honestly, the rest of the article is just embellishment. If you
*really* want a shorter article, there are plenty of others that say
essentially the same thing in different words (because, as I've said,
essentially every open-source project uses these guidelines as a
baseline):
* https://gist.github.com/robertpainsi/b632364184e70900af4ab688decf6f53
* https://github.com/erlang/otp/wiki/writing-good-commit-messages
* https://medium.com/@steveamaza/how-to-write-a-proper-git-commit-message-e028865e5791
>> Plus, everybody knows this already, since virtually every open-source
>> project follows more or less these guidelines for commit messages.
>
> If so, why would someone need to write a blog, and why would we need
> to point to it? Evidently not everybody knows that.
I said "everybody knows" in the same sense that I say "everyone knows
you're not supposed to check the binaries into Git". Like, sure, there
are people who don't know, but in that case it's pretty much on them
because this is a really basic thing that you just have to learn if
you want to do open-source software development. As a corollary, if
"everybody knows" something, then we should certainly mention it, but
we shouldn't consider it burdensome to expect people to learn about it
if they didn't already know.
> If the proposal is to let people write what they want, then I don't
> think I'd agree, for reasons I tried to explain.
That's unfortunate. What would change your mind?
My position is based on the belief that the current format makes it
harder for new contributors than a more open format. If potential new
contributors to Emacs were to tell me they prefer the current format,
then that would change *my* mind.
next prev parent reply other threads:[~2018-07-13 4:33 UTC|newest]
Thread overview: 219+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <m2po9o273c.fsf@linux-m68k.org>
[not found] ` <87374jd66s.fsf@lifelogs.com>
[not found] ` <87bmj6dda0.fsf@linux-m68k.org>
[not found] ` <87vahe911g.fsf@lifelogs.com>
[not found] ` <87374id7jy.fsf@linux-m68k.org>
[not found] ` <877ett8g7k.fsf@lifelogs.com>
[not found] ` <mvm8te9nuh4.fsf@suse.de>
[not found] ` <E1eOWht-0005nF-DD@fencepost.gnu.org>
[not found] ` <87a7yn7tqp.fsf@lifelogs.com>
[not found] ` <E1eOsjM-0000PB-02@fencepost.gnu.org>
[not found] ` <878te75xa1.fsf@lifelogs.com>
2017-12-15 21:26 ` The name gnus-cloud.el Richard Stallman
2017-12-16 22:34 ` Ted Zlatanov
2017-12-17 22:22 ` Richard Stallman
2017-12-17 22:31 ` Lars Ingebrigtsen
2017-12-17 22:57 ` Eric Abrahamsen
2017-12-18 21:15 ` Richard Stallman
2017-12-17 23:20 ` Stephen Berman
2017-12-18 21:17 ` Richard Stallman
2017-12-17 23:06 ` Stefan Monnier
2017-12-18 0:33 ` Ted Zlatanov
2017-12-18 3:31 ` Eli Zaretskii
2017-12-18 21:16 ` Richard Stallman
2017-12-19 3:37 ` Eli Zaretskii
2017-12-21 16:50 ` Richard Stallman
2017-12-21 17:13 ` Eli Zaretskii
2017-12-22 18:47 ` Richard Stallman
2017-12-22 18:57 ` Eli Zaretskii
2017-12-22 19:20 ` Eli Zaretskii
2017-12-23 14:58 ` Richard Stallman
2017-12-23 15:04 ` Eli Zaretskii
2017-12-24 20:34 ` Richard Stallman
2017-12-25 16:06 ` Eli Zaretskii
2017-12-25 22:27 ` Paul Eggert
2017-12-26 19:40 ` Richard Stallman
2018-07-10 13:16 ` Ted Zlatanov
2017-12-24 20:34 ` Richard Stallman
2017-12-25 16:09 ` Eli Zaretskii
2017-12-31 16:27 ` Steinar Bang
2017-12-31 16:54 ` Eli Zaretskii
2017-12-31 18:12 ` Óscar Fuentes
2017-12-31 18:26 ` Eli Zaretskii
2018-01-01 0:48 ` Steinar Bang
2018-01-01 4:40 ` Eli Zaretskii
2017-12-31 18:27 ` Andreas Schwab
2018-01-01 1:05 ` Steinar Bang
2018-01-01 4:42 ` Eli Zaretskii
2018-01-01 10:07 ` git history tracking across renames (and emacs support) (Was: The name gnus-cloud.el) Steinar Bang
2018-01-01 10:59 ` git history tracking across renames (and emacs support) Andreas Schwab
2018-01-01 21:40 ` Steinar Bang
2018-01-01 11:00 ` Andreas Schwab
2018-01-01 21:37 ` Steinar Bang
2018-01-01 11:06 ` Andreas Schwab
2018-01-01 21:36 ` Steinar Bang
2018-01-01 21:59 ` Andreas Schwab
2018-01-01 22:42 ` Steinar Bang
2018-01-01 22:04 ` Óscar Fuentes
2018-01-01 22:45 ` Noam Postavsky
2018-01-02 0:02 ` Óscar Fuentes
2018-01-02 1:15 ` git history tracking across renames (and emacs support) (Was: The name gnus-cloud.el) Richard Stallman
2018-01-02 8:06 ` Paul Eggert
2018-01-02 8:49 ` git history tracking across renames (and emacs support) Lars Ingebrigtsen
2018-07-10 13:14 ` Ted Zlatanov
2018-07-10 15:05 ` Marcin Borkowski
2018-07-10 15:36 ` Eli Zaretskii
2018-07-10 20:54 ` Ted Zlatanov
2018-07-10 23:04 ` Basil L. Contovounesios
2018-07-11 3:32 ` Eli Zaretskii
2018-07-11 3:53 ` Stefan Monnier
2018-07-11 4:44 ` Marcin Borkowski
2018-07-11 8:22 ` Eli Zaretskii
2018-07-11 13:54 ` git history tracking across renames (and emacs support), Re: git history tracking across renames (and emacs support), Re: git history tracking across renames (and emacs support), " Ted Zlatanov
2018-07-11 15:42 ` Eli Zaretskii
2018-07-11 16:09 ` Clément Pit-Claudel
2018-07-11 17:13 ` Eric Abrahamsen
2018-07-11 22:50 ` Richard Stallman
2018-07-11 23:04 ` Noam Postavsky
2018-07-12 23:35 ` Richard Stallman
2018-07-13 20:13 ` Jonas Bernoulli
2018-07-14 7:05 ` Eli Zaretskii
2018-07-11 17:31 ` Ted Zlatanov
2018-07-11 18:13 ` Eli Zaretskii
2018-07-11 20:14 ` Paul Eggert
2018-07-11 22:36 ` João Távora
2018-07-12 2:42 ` Eli Zaretskii
2018-07-12 9:04 ` João Távora
2018-07-12 13:48 ` Eli Zaretskii
2018-07-12 16:26 ` João Távora
2018-07-12 16:38 ` Eli Zaretskii
2018-07-12 16:47 ` João Távora
2018-07-12 16:59 ` Eli Zaretskii
2018-07-12 16:40 ` Stefan Monnier
2018-07-12 16:56 ` João Távora
2018-07-12 17:01 ` Eli Zaretskii
2018-07-12 19:37 ` Ted Zlatanov
2018-07-13 6:16 ` Eli Zaretskii
2018-07-12 17:17 ` Stefan Monnier
2018-07-12 23:12 ` João Távora
2018-07-13 18:47 ` Stefan Monnier
2018-07-14 17:36 ` [PATCH] New option to make C-x 4 a use file-less ChangeLog buffers (was Re: git history tracking across renames (and emacs support)) João Távora
2018-07-15 5:33 ` [PATCH] New option to make C-x 4 a use file-less ChangeLog buffers Basil L. Contovounesios
2018-07-15 10:41 ` João Távora
2018-07-15 14:11 ` Basil L. Contovounesios
2018-07-16 2:15 ` [PATCH] New option to make C-x 4 a use file-less ChangeLog buffers (was Re: git history tracking across renames (and emacs support)) Stefan Monnier
2018-07-16 10:10 ` João Távora
2018-07-16 12:33 ` Stefan Monnier
2018-07-16 14:02 ` João Távora
2018-07-16 15:44 ` Stefan Monnier
2018-07-16 18:00 ` João Távora
2018-07-16 18:22 ` Stefan Monnier
2018-07-16 18:50 ` João Távora
2018-07-16 21:09 ` Stefan Monnier
2018-07-21 10:45 ` Eli Zaretskii
2018-07-16 15:50 ` Eli Zaretskii
2018-07-16 23:02 ` João Távora
2018-07-21 10:42 ` Eli Zaretskii
2018-07-21 13:02 ` João Távora
2018-07-21 13:30 ` Eli Zaretskii
2018-07-21 15:08 ` João Távora
2018-07-21 16:13 ` Eli Zaretskii
2018-10-08 1:10 ` Ted Zlatanov
2018-10-08 1:31 ` João Távora
2018-07-12 13:36 ` git history tracking across renames (and emacs support) Stefan Monnier
2018-07-12 15:53 ` Basil L. Contovounesios
2018-07-12 16:00 ` Stefan Monnier
2018-07-12 16:48 ` Robert Pluim
2018-07-12 17:03 ` Stefan Monnier
2018-07-12 20:01 ` Robert Pluim
2018-07-12 20:06 ` Stefan Monnier
2018-07-12 20:28 ` Robert Pluim
2018-07-13 3:46 ` Stefan Monnier
2018-07-13 8:18 ` Robert Pluim
2018-07-13 12:16 ` Stefan Monnier
2018-07-13 13:07 ` Robert Pluim
2018-07-13 15:29 ` Clément Pit-Claudel
2018-07-13 15:49 ` Stefan Monnier
2018-07-12 16:40 ` João Távora
2018-07-12 16:59 ` Stefan Monnier
2018-07-12 23:33 ` João Távora
2018-07-13 18:55 ` Stefan Monnier
2018-07-13 19:45 ` João Távora
2018-07-13 20:33 ` Stefan Monnier
2018-07-11 17:08 ` Radon Rosborough
2018-07-11 17:56 ` Paul Eggert
2018-07-11 18:10 ` Eli Zaretskii
2018-07-11 22:51 ` Richard Stallman
2018-07-12 14:10 ` Eli Zaretskii
2018-07-11 22:51 ` Richard Stallman
2018-07-12 19:46 ` Ted Zlatanov
2018-07-12 23:03 ` Paul Eggert
2018-07-12 23:39 ` Richard Stallman
2018-07-11 18:25 ` Eli Zaretskii
2018-07-12 17:03 ` Radon Rosborough
2018-07-12 17:37 ` Eli Zaretskii
2018-07-13 4:33 ` Radon Rosborough [this message]
2018-07-13 8:14 ` Eli Zaretskii
2018-07-13 14:15 ` Paul Eggert
2018-07-13 14:39 ` Eli Zaretskii
2018-07-13 16:39 ` Stefan Monnier
2018-07-13 23:11 ` Richard Stallman
2018-07-14 6:58 ` Commit header lines (was: git history tracking across renames (and emacs support)) Eli Zaretskii
2018-07-14 16:09 ` Paul Eggert
2018-07-14 16:23 ` Eli Zaretskii
2018-07-14 21:57 ` Paul Eggert
2018-07-12 19:42 ` git history tracking across renames (and emacs support) Ted Zlatanov
2018-07-13 6:16 ` Eli Zaretskii
2018-01-02 16:15 ` git history tracking across renames (and emacs support) (Was: The name gnus-cloud.el) Eli Zaretskii
2018-01-03 2:50 ` Paul Eggert
2018-01-03 3:24 ` git history tracking across renames (and emacs support) Stefan Monnier
2018-01-03 15:36 ` Eli Zaretskii
2018-01-03 18:20 ` Stefan Monnier
2018-01-03 18:56 ` Eli Zaretskii
2018-01-03 20:17 ` Stefan Monnier
2018-01-03 20:43 ` Eli Zaretskii
2018-01-03 22:02 ` Stefan Monnier
2018-01-04 3:33 ` Eli Zaretskii
2018-01-04 5:00 ` Stefan Monnier
2018-01-04 6:39 ` Eli Zaretskii
2018-01-04 8:03 ` Paul Eggert
2018-01-04 14:25 ` Stefan Monnier
2018-01-04 16:13 ` Eli Zaretskii
2018-01-04 13:15 ` Dmitry Gutov
2018-01-05 3:51 ` Stefan Monnier
2018-01-03 18:29 ` Alan Mackenzie
2018-01-03 22:45 ` Stefan Monnier
2018-01-04 12:02 ` Alan Mackenzie
2018-01-04 18:05 ` Stefan Monnier
2018-01-03 15:32 ` git history tracking across renames (and emacs support) (Was: The name gnus-cloud.el) Eli Zaretskii
2018-01-03 20:11 ` git history tracking across renames (and emacs support) Stefan Monnier
2018-01-03 16:42 ` Stephen Leake
2018-01-04 11:46 ` Richard Stallman
2018-01-03 3:08 ` git history tracking across renames (and emacs support) (Was: The name gnus-cloud.el) Richard Stallman
2018-01-03 3:28 ` git history tracking across renames (and emacs support) Stefan Monnier
2018-01-08 2:16 ` git history tracking across renames (and emacs support) (Was: The name gnus-cloud.el) Richard Stallman
2018-01-08 4:22 ` Paul Eggert
2018-01-08 18:30 ` Eli Zaretskii
2018-01-08 18:41 ` Paul Eggert
2018-01-08 19:13 ` Eli Zaretskii
2018-01-08 19:29 ` Paul Eggert
2018-01-08 21:08 ` Eli Zaretskii
2018-01-09 2:55 ` Richard Stallman
2018-01-09 7:27 ` Paul Eggert
2018-01-10 3:03 ` Richard Stallman
2018-01-10 8:22 ` Paul Eggert
2018-01-10 15:32 ` Eli Zaretskii
2018-01-11 22:08 ` git history tracking across renames (and emacs support) Stefan Monnier
2018-01-12 8:21 ` Eli Zaretskii
2018-01-13 2:33 ` Paul Eggert
2018-01-13 5:13 ` Stefan Monnier
2018-01-13 8:31 ` Eli Zaretskii
2018-01-13 10:31 ` Tim Landscheidt
2018-01-13 17:42 ` Stefan Monnier
2018-01-13 19:22 ` Eli Zaretskii
2018-01-13 21:43 ` Stefan Monnier
2018-01-13 19:35 ` Paul Eggert
2018-01-13 19:42 ` Eli Zaretskii
2018-01-13 20:22 ` Paul Eggert
2018-01-14 3:31 ` Eli Zaretskii
2018-01-13 8:26 ` Eli Zaretskii
2018-01-16 2:16 ` git history tracking across renames (and emacs support) (Was: The name gnus-cloud.el) Richard Stallman
2018-01-09 2:54 ` Richard Stallman
2018-01-02 22:04 ` git history tracking across renames (and emacs support) Steinar Bang
2018-01-02 5:20 ` Stefan Monnier
2018-01-02 17:34 ` The name gnus-cloud.el Karl Fogel
2018-01-02 20:42 ` Chad Brown
2017-12-18 21:15 ` Richard Stallman
2017-12-19 6:09 ` John Wiegley
2017-12-19 7:42 ` Eli Zaretskii
2017-12-21 16:51 ` Richard Stallman
2017-12-19 9:09 ` joakim
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='CADB4rJHyokt_NSMs9siM1AxdtEMUiGpFw16Wkfk=priM1qDEFA@mail.gmail.com' \
--to=radon.neon@gmail.com \
--cc=eliz@gnu.org \
--cc=emacs-devel@gnu.org \
--cc=larsi@gnus.org \
--cc=tzz@lifelogs.com \
/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).