* [PATCH 0/1] rebase: warn about the correct tree's OID
@ 2018-11-20 9:44 Johannes Schindelin via GitGitGadget
2018-11-20 9:44 ` [PATCH 1/1] " Johannes Schindelin via GitGitGadget
2018-11-21 4:47 ` [PATCH 0/1] " Junio C Hamano
0 siblings, 2 replies; 4+ messages in thread
From: Johannes Schindelin via GitGitGadget @ 2018-11-20 9:44 UTC (permalink / raw)
To: git; +Cc: Junio C Hamano
A quick fix for a recent topic. Not overly critical, but I would deem this
v2.20.0-rc1 material.
Johannes Schindelin (1):
rebase: warn about the correct tree's OID
builtin/rebase.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
base-commit: bb75be6cb916297f271c846f2f9caa3daaaec718
Published-As: https://github.com/gitgitgadget/git/releases/tags/pr-85%2Fdscho%2Freset_head-typo-v1
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-85/dscho/reset_head-typo-v1
Pull-Request: https://github.com/gitgitgadget/git/pull/85
--
gitgitgadget
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH 1/1] rebase: warn about the correct tree's OID
2018-11-20 9:44 [PATCH 0/1] rebase: warn about the correct tree's OID Johannes Schindelin via GitGitGadget
@ 2018-11-20 9:44 ` Johannes Schindelin via GitGitGadget
2018-11-21 4:47 ` [PATCH 0/1] " Junio C Hamano
1 sibling, 0 replies; 4+ messages in thread
From: Johannes Schindelin via GitGitGadget @ 2018-11-20 9:44 UTC (permalink / raw)
To: git; +Cc: Junio C Hamano, Johannes Schindelin
From: Johannes Schindelin <johannes.schindelin@gmx.de>
This was a simple copy/paste error, and an obvious one at that: if we
cannot fill the tree descriptor, we should show an error message about
*that* tree, not another one.
Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
---
builtin/rebase.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/builtin/rebase.c b/builtin/rebase.c
index 1a2758756a..5b3e5baec8 100644
--- a/builtin/rebase.c
+++ b/builtin/rebase.c
@@ -582,7 +582,8 @@ static int reset_head(struct object_id *oid, const char *action,
}
if (!reset_hard && !fill_tree_descriptor(&desc[nr++], &head_oid)) {
- ret = error(_("failed to find tree of %s"), oid_to_hex(oid));
+ ret = error(_("failed to find tree of %s"),
+ oid_to_hex(&head_oid));
goto leave_reset_head;
}
--
gitgitgadget
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH 0/1] rebase: warn about the correct tree's OID
2018-11-20 9:44 [PATCH 0/1] rebase: warn about the correct tree's OID Johannes Schindelin via GitGitGadget
2018-11-20 9:44 ` [PATCH 1/1] " Johannes Schindelin via GitGitGadget
@ 2018-11-21 4:47 ` Junio C Hamano
2018-11-21 22:15 ` Johannes Schindelin
1 sibling, 1 reply; 4+ messages in thread
From: Junio C Hamano @ 2018-11-21 4:47 UTC (permalink / raw)
To: Johannes Schindelin via GitGitGadget; +Cc: git
"Johannes Schindelin via GitGitGadget" <gitgitgadget@gmail.com>
writes:
> A quick fix for a recent topic. Not overly critical, but I would deem this
> v2.20.0-rc1 material.
>
> Johannes Schindelin (1):
> rebase: warn about the correct tree's OID
>
Yup, it is kind of embarrasing that nobody caught it, but at the
same time, this typo is at so tiny level that I would not be
surprised if it survived for many years.
Will apply.
> builtin/rebase.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
>
> base-commit: bb75be6cb916297f271c846f2f9caa3daaaec718
> Published-As: https://github.com/gitgitgadget/git/releases/tags/pr-85%2Fdscho%2Freset_head-typo-v1
> Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-85/dscho/reset_head-typo-v1
> Pull-Request: https://github.com/gitgitgadget/git/pull/85
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 0/1] rebase: warn about the correct tree's OID
2018-11-21 4:47 ` [PATCH 0/1] " Junio C Hamano
@ 2018-11-21 22:15 ` Johannes Schindelin
0 siblings, 0 replies; 4+ messages in thread
From: Johannes Schindelin @ 2018-11-21 22:15 UTC (permalink / raw)
To: Junio C Hamano; +Cc: Johannes Schindelin via GitGitGadget, git
Hi Junio,
On Wed, 21 Nov 2018, Junio C Hamano wrote:
> "Johannes Schindelin via GitGitGadget" <gitgitgadget@gmail.com>
> writes:
>
> > A quick fix for a recent topic. Not overly critical, but I would deem this
> > v2.20.0-rc1 material.
> >
> > Johannes Schindelin (1):
> > rebase: warn about the correct tree's OID
> >
>
> Yup, it is kind of embarrasing that nobody caught it, but at the
> same time, this typo is at so tiny level that I would not be
> surprised if it survived for many years.
TBH I am quite mortified that it slipped through *my* multiple
pre-contribution reviews.
> Will apply.
Thanks.
Ciao,
Dscho
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-11-21 22:15 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-11-20 9:44 [PATCH 0/1] rebase: warn about the correct tree's OID Johannes Schindelin via GitGitGadget
2018-11-20 9:44 ` [PATCH 1/1] " Johannes Schindelin via GitGitGadget
2018-11-21 4:47 ` [PATCH 0/1] " Junio C Hamano
2018-11-21 22:15 ` Johannes Schindelin
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.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).