Thread (28 messages) 28 messages, 11 authors, 2016-06-15

Re: [PATCH v3] Porcelain scripts: Rewrite cryptic "needs update" error message

flat view

From: Ramkumar Ramachandra <hidden>
Date: 2016-06-15 22:49:55

Hi,

Ramkumar Ramachandra writes:
Although Git interally has the facility to differentiate between
porcelain and plubmbing commands and appropriately print errors,
several shell scripts invoke plubming commands triggering cryptic
plumbing errors to be displayed on a porcelain interface. This patch
replaces the "needs update" message in git-pull and git-rebase, when
`git update-index` is run, with a more friendly message.

Reported-by: Joshua Jensen <redacted>
Reported-by: Thore Husfeldt <redacted>
Signed-off-by: Ramkumar Ramachandra <redacted>
---
 Ref: [ref] for v2.
 Ref: [ref] for v1.

 Thanks to Matthieu for reviewing v1 and Junio for reviewing v2: I've
 tried to attack the problem more conservatively in this patch. It
 doesn't list paths, and doesn't print "generic" advice.
Junio: Is this patch alright?

-- Ram
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help