{"thread":{"id":"8032","subject":"[PATCH] git-gui: Call changes \"Staged\" and \"Unstaged\" in file list titles.","startedAt":"2007-05-08T11:33:06Z","lastAt":"2007-05-08T19:29:42Z","messageCount":4,"participants":["Johannes Sixt","Shawn O. Pearce","Johannes Schindelin","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"41444","messageId":"46405FF2.1B600CBE@eudaptics.com","threadId":"8032","inReplyTo":null,"subject":"[PATCH] git-gui: Call changes \"Staged\" and \"Unstaged\" in file list titles.","fromName":"Johannes Sixt","fromEmail":"j.sixt@eudaptics.com","sentAt":"2007-05-08T11:33:06Z","receivedAt":"2007-05-08T11:33:06Z","isPatch":true,"sender":{"key":"j6t@kdbg.org","avatar":"https://avatars.githubusercontent.com/u/14810926?v=4"},"body":"From: Johannes Sixt \n\nAll menu entries talk about \"staging\" and \"unstaging\" changes, but the\ntitles of the file lists use different wording, which may confuse\nnewcomers.\n\nSigned-off-by: Johannes Sixt \n---\nI hope the patch went through without my MUA's \"improvements\".\n\nHannes\n\n git-gui/git-gui.sh | 4 ++--\n 1 files changed, 2 insertions(+), 2 deletions(-)\n\ndiff --git a/git-gui/git-gui.sh b/git-gui/git-gui.sh\nindex 60e79ca..45d251e 100755\n--- a/git-gui/git-gui.sh\n+++ b/git-gui/git-gui.sh\n@@ -5494,7 +5494,7 @@ pack .vpane -anchor n -side top -fill both -expand 1\n # -- Index File List\n #\n frame .vpane.files.index -height 100 -width 200\n-label .vpane.files.index.title -text {Changes To Be Committed} \\\n+label .vpane.files.index.title -text {Staged Changes (Will Be Committed)} \\\n \t-background green \\\n \t-font font_ui\n text $ui_index -background white -borderwidth 0 \\\n@@ -5516,7 +5516,7 @@ pack $ui_index -side left -fill both -expand 1\n # -- Working Directory File List\n #\n frame .vpane.files.workdir -height 100 -width 200\n-label .vpane.files.workdir.title -text {Changed But Not Updated} \\\n+label .vpane.files.workdir.title -text {Unstaged Changes (Will Not Be Committed)} \\\n \t-background red \\\n \t-font font_ui\n text $ui_workdir -background white -borderwidth 0 \\\n-- \n1.5.1.1.27.g91776\n"},{"id":"41455","messageId":"20070508143859.GO11311@spearce.org","threadId":"8032","inReplyTo":"46405FF2.1B600CBE@eudaptics.com","subject":"Re: [PATCH] git-gui: Call changes \"Staged\" and \"Unstaged\" in file list titles.","fromName":"Shawn O. Pearce","fromEmail":"spearce@spearce.org","sentAt":"2007-05-08T14:38:59Z","receivedAt":"2007-05-08T14:38:59Z","isPatch":true,"sender":{"key":"spearce@spearce.org","avatar":"https://avatars.githubusercontent.com/u/34844?v=4"},"body":"Johannes Sixt wrote:\n> From: Johannes Sixt \n> \n> All menu entries talk about \"staging\" and \"unstaging\" changes, but the\n> titles of the file lists use different wording, which may confuse\n> newcomers.\n> \n> Signed-off-by: Johannes Sixt \n> ---\n> I hope the patch went through without my MUA's \"improvements\".\n\nYes. But...\n \n> git-gui/git-gui.sh | 4 ++--\n> 1 files changed, 2 insertions(+), 2 deletions(-)\n\nThat's the wrong path. Clearly you made this patch in git.git and\nnot the git-gui.git repository itself. Fortunately for me I added\n-3 to git-am out of habit:\n\n $ git am -3 -s gg\n Applying 'git-gui: Call changes \"Staged\" and \"Unstaged\" in file list titles.'\n \n error: git-gui/git-gui.sh: does not exist in index\n Using index info to reconstruct a base tree...\n Falling back to patching base and 3-way merge...\n Renamed git-gui/git-gui.sh => git-gui.sh\n Wrote tree 8e034a94d9533af707279766e174b78ddf3874cc\n Committed: a1a4975824e8f9f88a8c96ae908c488a2c6047c5\n\nYay Git! Go go gadget rename detection! ;-)\n\nI didn't even realize the git-gui/ prefix on the path in the patch\nuntil after I saw the rename detection message.\n\n-- \nShawn.\n"},{"id":"41462","messageId":"Pine.LNX.4.64.0705081713260.4167@racer.site","threadId":"8032","inReplyTo":"20070508143859.GO11311@spearce.org","subject":"Re: [PATCH] git-gui: Call changes \"Staged\" and \"Unstaged\" in file list titles.","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2007-05-08T15:13:59Z","receivedAt":"2007-05-08T15:13:59Z","isPatch":true,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi,\n\nOn Tue, 8 May 2007, Shawn O. Pearce wrote:\n\n> Yay Git! Go go gadget rename detection! ;-)\n\nNow, how cool is _that_!\n\nCiao,\nDscho\n"},{"id":"41484","messageId":"7vmz0f2jx5.fsf@assigned-by-dhcp.cox.net","threadId":"8032","inReplyTo":"20070508143859.GO11311@spearce.org","subject":"Re: [PATCH] git-gui: Call changes \"Staged\" and \"Unstaged\" in file list titles.","fromName":"Junio C Hamano","fromEmail":"junkio@cox.net","sentAt":"2007-05-08T19:29:42Z","receivedAt":"2007-05-08T19:29:42Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"\"Shawn O. Pearce\" writes:\n\n> Yes. But...\n> \n>> git-gui/git-gui.sh | 4 ++--\n>> 1 files changed, 2 insertions(+), 2 deletions(-)\n>\n> That's the wrong path. Clearly you made this patch in git.git and\n> not the git-gui.git repository itself. Fortunately for me I added\n> -3 to git-am out of habit:\n>\n> $ git am -3 -s gg\n> Applying 'git-gui: Call changes \"Staged\" and \"Unstaged\" in file list titles.'\n> \n> error: git-gui/git-gui.sh: does not exist in index\n> Using index info to reconstruct a base tree...\n> Falling back to patching base and 3-way merge...\n> Renamed git-gui/git-gui.sh => git-gui.sh\n> Wrote tree 8e034a94d9533af707279766e174b78ddf3874cc\n> Committed: a1a4975824e8f9f88a8c96ae908c488a2c6047c5\n>\n> Yay Git! Go go gadget rename detection! ;-)\n>\n> I didn't even realize the git-gui/ prefix on the path in the patch\n> until after I saw the rename detection message.\n\nYeah, -3 is a magic, isn't it?\n\nFor the benefit of people who are unfamiliar with the internal,\nhere is how it works.\n\n - The patch Hannes gave Shawn had a change to\n git-gui/git-gui.sh, like this:\n\n git-gui/git-gui.sh | 4 ++--\n 1 files changed, 2 insertions(+), 2 deletions(-)\n\n diff --git a/git-gui/git-gui.sh b/git-gui/git-gui.sh\n index 60e79ca..45d251e 100755\n --- a/git-gui/git-gui.sh\n +++ b/git-gui/git-gui.sh\n @@ -5494,7 +5494,7 @@ pack .vpane -anchor n -side top -fill both -expand 1\n\n\n - Shawn had a tree that looked like this:\n\n GIT-VERSION-GEN\n Makefile\n git-gui.sh\n lib/blame.tcl\n lib/branch.tcl\n ...\n\n - The patch obviously does not apply, as Shawn does not have\n git-gui/git-gui.sh. However, Shawn instructed \"git am\" to\n fall back on 3-way merge, which prepared two \"fake\" trees.\n\n Fake tree 1 (the ancestor) is prepared by reading the index\n line to figure out what blob the patch applies to. The\n answer is 60e79ca, which Shawn has in his repository. The\n resulting tree looks like this:\n\n 040000 tree 0df2c7b... git-gui\n 100755 blob 60e79ca... git-gui/git-gui.sh\n\n Fake tree 2 (update) is prepared by applying the patch to the\n ancestor fake tree.\n\n 040000 tree 950e50d...\tgit-gui\n 100755 blob 45d251e...\tgit-gui/git-gui.sh\n\n - Then, it falls back to the 3-way merge, using the fake tree 1\n as the common ancestor, Shawn's repository as \"our\" tree, and\n fake tree 2 as the update being merged into \"our\" tree.\n\n To this merge, most of the paths in \"our\" tree appear as if\n Shawn added them since the common ancestor, except that\n git-gui.sh at the toplevel in \"our\" tree was moved up by\n Shawn from git-gui subdirectory (and then possibly further\n modified), while git-gui/git-gui.sh was modified by Hannes.\n\n At the path level, this is the classic case of \"one side\n (Shawn) modifies, while the other side (Hannes) kept it at\n the place it was before\", 3-way merge at the path level takes\n Shawn's change (which is to \"move the file up one level\").\n\n At the same time, at the contents level, this is just a\n normal \"common ancestor 60e79ca was modified to Shawn's\n git-gui.sh on one side while Hannes's side modified it to\n 45d251e\", and the contents are merged using the file level\n 3-way merge. The result is left in git-gui.sh.\n\n All the other paths look as if Shawn added them to the common\n ancestor that did not have any of them while Hannes did not\n do any such change, so the merge algorithm takes Shawn's\n \"additions\".\n\nSo in the end, Shawn ends up with most of his paths intact,\nexcept that Hannes's patch which were meant to apply to\ngit-gui/git-gui.sh gets applied to his git-gui.sh.\n"}]}