{"thread":{"id":"53659","subject":"[PATCH] GIT-VERSION-GEN: update for the post 2.27 cycle","startedAt":"2020-06-11T13:50:33Z","lastAt":"2020-06-12T23:15:01Z","messageCount":4,"participants":["Denton Liu","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"399533","messageId":"4946cf3650b95191455630f85f59e5f93156d0ac.1591883410.git.liu.denton@gmail.com","threadId":"53659","inReplyTo":null,"subject":"[PATCH] GIT-VERSION-GEN: update for the post 2.27 cycle","fromName":"Denton Liu","fromEmail":"liu.denton@gmail.com","sentAt":"2020-06-11T13:50:20Z","receivedAt":"2020-06-11T13:50:33Z","isPatch":true,"sender":{"key":"liu.denton@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9620836?v=4"},"body":"Signed-off-by: Denton Liu \n---\n GIT-VERSION-GEN | 2 +-\n 1 file changed, 1 insertion(+), 1 deletion(-)\n\ndiff --git a/GIT-VERSION-GEN b/GIT-VERSION-GEN\nindex 06a5333ee6..7b0cfeb92e 100755\n--- a/GIT-VERSION-GEN\n+++ b/GIT-VERSION-GEN\n@@ -1,7 +1,7 @@\n #!/bin/sh\n \n GVF=GIT-VERSION-FILE\n-DEF_VER=v2.27.0\n+DEF_VER=v2.27.GIT\n \n LF='\n '\n-- \n2.27.0.132.g321788e831\n\n"},{"id":"399563","messageId":"xmqq8sgtjsor.fsf@gitster.c.googlers.com","threadId":"53659","inReplyTo":"4946cf3650b95191455630f85f59e5f93156d0ac.1591883410.git.liu.denton@gmail.com","subject":"Re: [PATCH] GIT-VERSION-GEN: update for the post 2.27 cycle","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2020-06-11T19:41:56Z","receivedAt":"2020-06-11T19:42:01Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Denton Liu writes:\n\n> Signed-off-by: Denton Liu \n> ---\n> GIT-VERSION-GEN | 2 +-\n> 1 file changed, 1 insertion(+), 1 deletion(-)\n>\n> diff --git a/GIT-VERSION-GEN b/GIT-VERSION-GEN\n> index 06a5333ee6..7b0cfeb92e 100755\n> --- a/GIT-VERSION-GEN\n> +++ b/GIT-VERSION-GEN\n> @@ -1,7 +1,7 @@\n> #!/bin/sh\n> \n> GVF=GIT-VERSION-FILE\n> -DEF_VER=v2.27.0\n> +DEF_VER=v2.27.GIT\n\nI have been thinking about stopping this transition from \".0\" to\n\".GIT\" and left it at \".0\" deliberately.\n\n - The target to create a tarball (\"make dist\") places the \"version\"\n file that overrides whatever value is set here, so those who\n build from a tarball will not care what the value described here\n is.\n\n - Those who build from a repository would use \"git describe\".\n\n - In addition, I do not create tarballs and upload to public places\n for a random version with .GIT suffix---I only do so for -rcX and\n the releases. If anybody else is doing so with \"make dist\",\n there would be the \"version\" file included in the tarball,\n recording what is obtained from \"git describe HEAD\".\n\nSo there is no strong reason to care what this value is, and that\nwas why I was experimenting with the idea of leaving it at \".0\",\nin the hope of hearing from people who do want to see \".GIT\" why\nthey want it.\n\nUnfortunately, your patch does not say why, either, so it hasn't\nquite helped yet ;-)\n\n\n\n"},{"id":"399572","messageId":"20200612005217.GA3964@generichostname","threadId":"53659","inReplyTo":"xmqq8sgtjsor.fsf@gitster.c.googlers.com","subject":"Re: [PATCH] GIT-VERSION-GEN: update for the post 2.27 cycle","fromName":"Denton Liu","fromEmail":"liu.denton@gmail.com","sentAt":"2020-06-12T00:52:17Z","receivedAt":"2020-06-12T00:52:32Z","isPatch":true,"sender":{"key":"liu.denton@gmail.com","avatar":"https://avatars.githubusercontent.com/u/9620836?v=4"},"body":"Hi Junio,\n\nOn Thu, Jun 11, 2020 at 12:41:56PM -0700, Junio C Hamano wrote:\n> Denton Liu writes:\n> \n> > Signed-off-by: Denton Liu \n> > ---\n> > GIT-VERSION-GEN | 2 +-\n> > 1 file changed, 1 insertion(+), 1 deletion(-)\n> >\n> > diff --git a/GIT-VERSION-GEN b/GIT-VERSION-GEN\n> > index 06a5333ee6..7b0cfeb92e 100755\n> > --- a/GIT-VERSION-GEN\n> > +++ b/GIT-VERSION-GEN\n> > @@ -1,7 +1,7 @@\n> > #!/bin/sh\n> > \n> > GVF=GIT-VERSION-FILE\n> > -DEF_VER=v2.27.0\n> > +DEF_VER=v2.27.GIT\n> \n> I have been thinking about stopping this transition from \".0\" to\n> \".GIT\" and left it at \".0\" deliberately.\n> \n> - The target to create a tarball (\"make dist\") places the \"version\"\n> file that overrides whatever value is set here, so those who\n> build from a tarball will not care what the value described here\n> is.\n> \n> - Those who build from a repository would use \"git describe\".\n> \n> - In addition, I do not create tarballs and upload to public places\n> for a random version with .GIT suffix---I only do so for -rcX and\n> the releases. If anybody else is doing so with \"make dist\",\n> there would be the \"version\" file included in the tarball,\n> recording what is obtained from \"git describe HEAD\".\n> \n> So there is no strong reason to care what this value is, and that\n> was why I was experimenting with the idea of leaving it at \".0\",\n> in the hope of hearing from people who do want to see \".GIT\" why\n> they want it.\n\nThanks for the explanation, it makes sense.\n\n> Unfortunately, your patch does not say why, either, so it hasn't\n> quite helped yet ;-)\n\nI tried looking on the list but I didn't find any mention about this\nexperiment so I assumed that you unintentionally forgot to update the\nversion. Good to know that it was done deliberately :)\n\n-Denton\n"},{"id":"399634","messageId":"xmqq4krf27x8.fsf@gitster.c.googlers.com","threadId":"53659","inReplyTo":"20200612005217.GA3964@generichostname","subject":"Re: [PATCH] GIT-VERSION-GEN: update for the post 2.27 cycle","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2020-06-12T23:14:43Z","receivedAt":"2020-06-12T23:15:01Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Denton Liu writes:\n\n> Thanks for the explanation, it makes sense.\n>\n>> Unfortunately, your patch does not say why, either, so it hasn't\n>> quite helped yet ;-)\n>\n> I tried looking on the list but I didn't find any mention about this\n> experiment so I assumed that you unintentionally forgot to update the\n> version. Good to know that it was done deliberately :)\n\nWell having said all that, I do not have a strong reason not to\nupdate it. Perhaps when the next batch of topics graduates and\ncause the release notes to be updated, I'll update it as well.\n\nThanks.\n"}]}