From: Michael Haggerty <mhagger@alum.mit.edu>
To: Junio C Hamano <gitster@pobox.com>
Cc: "Stefan Beller" <sbeller@google.com>,
"Ronnie Sahlberg" <ronniesahlberg@gmail.com>,
"Jonathan Nieder" <jrnieder@gmail.com>,
"Nguyễn Thái Ngọc Duy" <pclouds@gmail.com>,
"Eric Sunshine" <sunshine@sunshineco.com>,
"Jeff King" <peff@peff.net>,
git@vger.kernel.org, "Michael Haggerty" <mhagger@alum.mit.edu>
Subject: [PATCH v2 0/7] Fix some problems with reflog expiration
Date: Mon, 2 Mar 2015 10:29:50 +0100 [thread overview]
Message-ID: <1425288597-20547-1-git-send-email-mhagger@alum.mit.edu> (raw)
This is v2 of the patch series. Thanks to Eric Sunshine, Stefan
Beller, Peff, and Junio for their comments about v1 [1].
The two main changes since v1:
* Don't change the locking policy for changing symbolic references.
Even though the existing policy is partly broken, the change
proposed in v1 would have introduced an incompatibility with older
versions of Git and with libgit2 and possibly other Git clients. I
will try to implement the transition plan described in [2], but I
will do it in a separate patch series.
* I rewrote the "git reflog" documentation more completely, following
Junio's suggestion.
Other changes since v1:
* Rebased onto a more recent version of upstream master (because
master now includes mh/reflog-expire and sb/atomic-push). The rebase
was conflict-free.
* Tweaked some commit messages and added some Reviewed-by lines.
* Appended a patch by Stefan Beller to delete the "force_write" member
of "struct ref_lock".
[1] http://thread.gmane.org/gmane.comp.version-control.git/263552
[2] http://article.gmane.org/gmane.comp.version-control.git/263811
Michael Haggerty (6):
write_ref_sha1(): remove check for lock == NULL
write_ref_sha1(): Move write elision test to callers
lock_ref_sha1_basic(): do not set force_write for missing references
reflog: improve and update documentation
reflog_expire(): ignore --updateref for symbolic references
reflog_expire(): never update a reference to null_sha1
Stefan Beller (1):
struct ref_lock: delete the force_write member
Documentation/git-reflog.txt | 139 +++++++++++++++++++++++++------------------
builtin/reflog.c | 4 +-
refs.c | 63 +++++++++++---------
3 files changed, 117 insertions(+), 89 deletions(-)
--
2.1.4
next reply other threads:[~2015-03-02 9:31 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-03-02 9:29 Michael Haggerty [this message]
2015-03-02 9:29 ` [PATCH v2 1/7] write_ref_sha1(): remove check for lock == NULL Michael Haggerty
2015-03-02 9:29 ` [PATCH v2 2/7] write_ref_sha1(): Move write elision test to callers Michael Haggerty
2015-03-02 9:29 ` [PATCH v2 3/7] lock_ref_sha1_basic(): do not set force_write for missing references Michael Haggerty
2015-03-02 9:29 ` [PATCH v2 4/7] struct ref_lock: delete the force_write member Michael Haggerty
2015-03-02 21:44 ` Junio C Hamano
2015-03-03 10:50 ` Michael Haggerty
2015-03-02 9:29 ` [PATCH v2 5/7] reflog: improve and update documentation Michael Haggerty
2015-03-02 22:04 ` Junio C Hamano
2015-03-03 11:35 ` Michael Haggerty
2015-03-02 9:29 ` [PATCH v2 6/7] reflog_expire(): ignore --updateref for symbolic references Michael Haggerty
2015-03-02 9:29 ` [PATCH v2 7/7] reflog_expire(): never update a reference to null_sha1 Michael Haggerty
2015-03-02 22:09 ` [PATCH v2 0/7] Fix some problems with reflog expiration Junio C Hamano
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=1425288597-20547-1-git-send-email-mhagger@alum.mit.edu \
--to=mhagger@alum.mit.edu \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=jrnieder@gmail.com \
--cc=pclouds@gmail.com \
--cc=peff@peff.net \
--cc=ronniesahlberg@gmail.com \
--cc=sbeller@google.com \
--cc=sunshine@sunshineco.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.
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 NNTP newsgroup(s).