From: "Martin von Zweigbergk via GitGitGadget" <gitgitgadget@gmail.com>
To: git@vger.kernel.org
Cc: Martin von Zweigbergk <martinvonz@gmail.com>,
Martin von Zweigbergk <martinvonz@google.com>
Subject: [PATCH] BreakingChanges: say that `git diff X..Y` syntax will be removed in 3.0
Date: Wed, 15 Oct 2025 22:07:34 +0000 [thread overview]
Message-ID: <pull.1989.git.1760566054455.gitgitgadget@gmail.com> (raw)
From: Martin von Zweigbergk <martinvonz@google.com>
The `git diff X..Y` syntax is quite misleading because it looks like
it shows the diff of the commits in the X..Y range but it actually
shows the diff from X to Y. IMO, if that syntax is supported, it
should show a diff from the merge base of X and Y to Y. I hope Git 3.0
is a good time to remove support for the current syntax and
semantics. Then we can perhaps add the syntax back later with less
surprising semantics.
Signed-off-by: Martin von Zweigbergk <martinvonz@google.com>
---
BreakingChanges: say that git diff X..Y syntax will be removed in 3.0
Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-1989%2Fmartinvonz%2Fmz%2Fwtmnpolouvvz-v1
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-1989/martinvonz/mz/wtmnpolouvvz-v1
Pull-Request: https://github.com/gitgitgadget/git/pull/1989
Documentation/BreakingChanges.adoc | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/Documentation/BreakingChanges.adoc b/Documentation/BreakingChanges.adoc
index 90b53abcea..93fb968840 100644
--- a/Documentation/BreakingChanges.adoc
+++ b/Documentation/BreakingChanges.adoc
@@ -114,6 +114,10 @@ applications and forges.
+
There is no plan to deprecate the "sha1" object format at this point in time.
+
+Support for "git diff X..Y" syntax will be removed. Use "git diff X Y" instead.
+This will open up the syntax for a more consistent interpretation of
+"git diff $(git merge-base X Y) Y".
++
Cf. <2f5de416-04ba-c23d-1e0b-83bb655829a7@zombino.com>,
<20170223155046.e7nxivfwqqoprsqj@LykOS.localdomain>,
<CA+EOSBncr=4a4d8n9xS4FNehyebpmX8JiUwCsXD47EQDE+DiUQ@mail.gmail.com>.
base-commit: 143f58ef7535f8f8a80d810768a18bdf3807de26
--
gitgitgadget
next reply other threads:[~2025-10-15 22:07 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-10-15 22:07 Martin von Zweigbergk via GitGitGadget [this message]
2025-10-15 22:19 ` [PATCH] BreakingChanges: say that `git diff X..Y` syntax will be removed in 3.0 Junio C Hamano
2025-10-15 23:06 ` Martin von Zweigbergk
2025-10-16 17:42 ` Kristoffer Haugsbakk
2025-10-16 20:32 ` D. Ben Knoble
2025-10-15 22:28 ` brian m. carlson
2025-10-16 13:44 ` Junio C Hamano
2025-10-16 16:38 ` Martin von Zweigbergk
2025-10-16 17:02 ` Kristoffer Haugsbakk
2025-10-16 17:12 ` Martin von Zweigbergk
2025-10-16 16:56 ` Junio C Hamano
2025-10-16 20:42 ` D. Ben Knoble
2025-10-16 1:28 ` Justin Tobler
2025-10-16 16:34 ` Martin von Zweigbergk
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=pull.1989.git.1760566054455.gitgitgadget@gmail.com \
--to=gitgitgadget@gmail.com \
--cc=git@vger.kernel.org \
--cc=martinvonz@gmail.com \
--cc=martinvonz@google.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 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.