From: "Stephen J. Turnbull" <stephen@xemacs.org>
To: Glenn Morris <rgm@gnu.org>
Cc: Stefan Monnier <monnier@iro.umontreal.ca>, emacs-devel@gnu.org
Subject: Re: Recording the date at which a change was pushed to Savannah
Date: Wed, 03 Dec 2014 16:51:35 +0900 [thread overview]
Message-ID: <87r3whkw6w.fsf@uwakimon.sk.tsukuba.ac.jp> (raw)
In-Reply-To: <9e4mtd1cdi.fsf@fencepost.gnu.org>
Glenn Morris writes:
> Obviously the same sort of [commit date/push date lag] will apply
> if someone delays pushing their local commits to Savannah. Or the
> next time a long-lived feature branch gets merged to master.
As I wrote before, I think it's reasonable to want to know when a
commit was pushed. However, since merges are *always* done locally,
they are usually done very shortly before pushing. Similarly for
one-off commits. If anybody pushes before you do, you *can't* push in
most common public repo configurations.
AFAICT, the merge date is therefore highly likely to be an accurate
estimate of the push date for all of its parents back to the node at
the public branch (usually master, but the release branches would work
the same way).
> I've already been confused by this once (in trying to figure out what
> went on in some recent merge commits), and expect to be confused by it
> again in future. Basically all I can do is ignore the Date fields
> completely, or treat them as lower limits.
I don't see a problem with a push hook for adding notes, but I don't
see how it will solve your "confused by commit date" issue.
FWIW, my advice is to ignore those dates, unless you're trying to
compare to "external" events like mailing list posts. Commits on
parallel branches are *concurrent*; it's not useful to compare them by
dates. The commits where branches fork and where they merge are
points where the timelines are synchronized, but the interim commits
are not.
If you want commits linearly ordered, you need to rebase. This is
precisely why many people who favor rebasing workflows like those
workflows.
I suppose if you create your push hook, you will be able to order the
log by push date, and thus create groups of commits that were pushed
at the same time. But the relevant information for full understanding
of "what happened" in a merge is already present in the DAG, and you
will likely lose some relevant information by grouping by push date.
YMMV; grouping by push date may well be sufficient for your use case.
Regards,
next prev parent reply other threads:[~2014-12-03 7:51 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-12-02 2:55 Recording the date at which a change was pushed to Savannah Glenn Morris
2014-12-02 4:55 ` Yuri Khan
2014-12-02 5:05 ` Stephen J. Turnbull
2014-12-02 13:48 ` Stefan Monnier
2014-12-03 6:22 ` Glenn Morris
2014-12-03 6:47 ` David Kastrup
2014-12-03 7:51 ` Stephen J. Turnbull [this message]
2014-12-03 8:35 ` David Kastrup
2014-12-03 9:50 ` Stephen J. Turnbull
2014-12-03 10:04 ` David Kastrup
2014-12-03 14:18 ` Stefan Monnier
2014-12-03 16:19 ` Yuri Khan
2014-12-03 18:59 ` Stefan Monnier
2014-12-03 18:02 ` Eli Zaretskii
2014-12-03 19:00 ` Stefan Monnier
2014-12-03 19:06 ` Eli Zaretskii
2014-12-03 19:34 ` Stefan Monnier
2014-12-03 20:24 ` Eli Zaretskii
2014-12-03 21:10 ` Stefan Monnier
2014-12-04 2:52 ` Yuri Khan
2014-12-04 6:17 ` Eli Zaretskii
2014-12-04 2:58 ` Glenn Morris
2014-12-04 3:57 ` Stefan Monnier
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
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87r3whkw6w.fsf@uwakimon.sk.tsukuba.ac.jp \
--to=stephen@xemacs.org \
--cc=emacs-devel@gnu.org \
--cc=monnier@iro.umontreal.ca \
--cc=rgm@gnu.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 external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.