{"thread":{"id":"32963","subject":"[PATCH 1/2] git-filter-branch.txt: clarify ident variables usage","startedAt":"2013-02-21T20:22:50Z","lastAt":"2013-02-21T20:46:34Z","messageCount":2,"participants":["Tadeusz Andrzej Kadłubowski","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":2},"messages":[{"id":"209990","messageId":"5126821A.9020800@hell.org.pl","threadId":"32963","inReplyTo":null,"subject":"[PATCH 1/2] git-filter-branch.txt: clarify ident variables usage","fromName":"Tadeusz Andrzej Kadłubowski","fromEmail":"yess@hell.org.pl","sentAt":"2013-02-21T20:22:50Z","receivedAt":"2013-02-21T20:22:50Z","isPatch":true,"sender":{"key":"yess@hell.org.pl","avatar":"https://avatars.githubusercontent.com/u/207712?v=4"},"body":"There is a rare edge case of git-filter-branch: a filter that unsets\nidentity variables from the environment. Link to git-commit-tree\nclarifies how Git would fall back in this situation.\n\nSigned-off-by: Tadeusz Andrzej Kadłubowski <yess@hell.org.pl>\n---\n Documentation/git-filter-branch.txt | 6 +++---\n 1 file changed, 3 insertions(+), 3 deletions(-)\n\ndiff --git a/Documentation/git-filter-branch.txt b/Documentation/git-filter-branch.txt\nindex dfd12c9..e50ee2f 100644\n--- a/Documentation/git-filter-branch.txt\n+++ b/Documentation/git-filter-branch.txt\n@@ -65,9 +65,9 @@ Prior to that, the $GIT_COMMIT environment variable will be set to contain\n the id of the commit being rewritten.  Also, GIT_AUTHOR_NAME,\n GIT_AUTHOR_EMAIL, GIT_AUTHOR_DATE, GIT_COMMITTER_NAME, GIT_COMMITTER_EMAIL,\n and GIT_COMMITTER_DATE are set according to the current commit.  The values\n-of these variables after the filters have run, are used for the new commit.\n-If any evaluation of <command> returns a non-zero exit status, the whole\n-operation will be aborted.\n+of these variables after the filters have run, are used for the new commit\n+(see linkgit:git-commit-tree[1] for details).  If any evaluation of <command>\n+returns a non-zero exit status, the whole operation will be aborted.\n  A 'map' function is available that takes an \"original sha1 id\" argument\n and outputs a \"rewritten sha1 id\" if the commit has been already\n-- \n1.7.11.7\n"},{"id":"209995","messageId":"7vwqu1tmxh.fsf@alter.siamese.dyndns.org","threadId":"32963","inReplyTo":"5126821A.9020800@hell.org.pl","subject":"Re: [PATCH 1/2] git-filter-branch.txt: clarify ident variables usage","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-02-21T20:46:34Z","receivedAt":"2013-02-21T20:46:34Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Tadeusz Andrzej Kadłubowski  <yess@hell.org.pl> writes:\n\n> There is a rare edge case of git-filter-branch: a filter that unsets\n> identity variables from the environment. Link to git-commit-tree\n> clarifies how Git would fall back in this situation.\n\nI find it unclear in the updated text _why_ the reader may want to\nrefer to that other documentation.\n\n> Signed-off-by: Tadeusz Andrzej Kadłubowski <yess@hell.org.pl>\n> ---\n>  Documentation/git-filter-branch.txt | 6 +++---\n>  1 file changed, 3 insertions(+), 3 deletions(-)\n>\n> diff --git a/Documentation/git-filter-branch.txt b/Documentation/git-filter-branch.txt\n> index dfd12c9..e50ee2f 100644\n> --- a/Documentation/git-filter-branch.txt\n> +++ b/Documentation/git-filter-branch.txt\n> @@ -65,9 +65,9 @@ Prior to that, the $GIT_COMMIT environment variable will be set to contain\n>  the id of the commit being rewritten.  Also, GIT_AUTHOR_NAME,\n>  GIT_AUTHOR_EMAIL, GIT_AUTHOR_DATE, GIT_COMMITTER_NAME, GIT_COMMITTER_EMAIL,\n>  and GIT_COMMITTER_DATE are set according to the current commit.  The values\n> -of these variables after the filters have run, are used for the new commit.\n> -If any evaluation of <command> returns a non-zero exit status, the whole\n> -operation will be aborted.\n> +of these variables after the filters have run, are used for the new commit\n> +(see linkgit:git-commit-tree[1] for details).  If any evaluation of <command>\n> +returns a non-zero exit status, the whole operation will be aborted.\n\nHere is my attempt to clarify it a bit.\n\n\tAlso, ...<variables>... are taken from the current commit\n\tand exported to the environment, in order to affect the\n\tauthor and committer identities of the replacement commit\n\tcreated by linkgit:git-commit-tree[1] after the filters have\n\trun.\n\nA user who contemplates to unset them should be able to guess the\nconsequences, even though the above text does not single out such an\ninsane misuse to put an undue stress on it.\n\n>   A 'map' function is available that takes an \"original sha1 id\" argument\n>  and outputs a \"rewritten sha1 id\" if the commit has been already\n"}]}