{"thread":{"id":"45269","subject":"[PATCH] Do not require Python for the git-remote-{bzr,hg} placeholder scripts","startedAt":"2017-03-03T10:59:50Z","lastAt":"2017-03-03T20:14:52Z","messageCount":6,"participants":["Sebastian Schuberth","Jeff King","Marc Branchaud","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"313156","messageId":"0102015a93d18443-c574aaf8-5656-49e8-8848-810e1fadc87a-000000@eu-west-1.amazonses.com","threadId":"45269","inReplyTo":null,"subject":"[PATCH] Do not require Python for the git-remote-{bzr,hg} placeholder scripts","fromName":"Sebastian Schuberth","fromEmail":"sschuberth@gmail.com","sentAt":"2017-03-03T10:57:46Z","receivedAt":"2017-03-03T10:59:50Z","isPatch":true,"sender":{"key":"sschuberth@gmail.com","avatar":"https://avatars.githubusercontent.com/u/349154?v=4"},"body":"It does not make sense for these placeholder scripts to depend on Python\njust because the real scripts do. At the example of Git for Windows, we\nwould not even be able to see those warnings as it does not ship with\nPython. So just use plain shell scripts instead.\n\nSigned-off-by: Sebastian Schuberth \n---\n contrib/remote-helpers/git-remote-bzr | 16 +++++++---------\n contrib/remote-helpers/git-remote-hg | 16 +++++++---------\n 2 files changed, 14 insertions(+), 18 deletions(-)\n\ndiff --git a/contrib/remote-helpers/git-remote-bzr b/contrib/remote-helpers/git-remote-bzr\nindex 712a137..ccc4aea 100755\n--- a/contrib/remote-helpers/git-remote-bzr\n+++ b/contrib/remote-helpers/git-remote-bzr\n@@ -1,13 +1,11 @@\n-#!/usr/bin/env python\n+#!/bin/sh\n \n-import sys\n-\n-sys.stderr.write('WARNING: git-remote-bzr is now maintained independently.\\n')\n-sys.stderr.write('WARNING: For more information visit https://github.com/felipec/git-remote-bzr\\n')\n-\n-sys.stderr.write('''WARNING:\n+cat <<'EOT'\n+WARNING: git-remote-bzr is now maintained independently.\n+WARNING: For more information visit https://github.com/felipec/git-remote-bzr\n+WARNING:\n WARNING: You can pick a directory on your $PATH and download it, e.g.:\n-WARNING: $ wget -O $HOME/bin/git-remote-bzr \\\\\n+WARNING: $ wget -O $HOME/bin/git-remote-bzr \\\n WARNING: https://raw.github.com/felipec/git-remote-bzr/master/git-remote-bzr\n WARNING: $ chmod +x $HOME/bin/git-remote-bzr\n-''')\n+EOT\ndiff --git a/contrib/remote-helpers/git-remote-hg b/contrib/remote-helpers/git-remote-hg\nindex 4255ad6..dfda44f 100755\n--- a/contrib/remote-helpers/git-remote-hg\n+++ b/contrib/remote-helpers/git-remote-hg\n@@ -1,13 +1,11 @@\n-#!/usr/bin/env python\n+#!/bin/sh\n \n-import sys\n-\n-sys.stderr.write('WARNING: git-remote-hg is now maintained independently.\\n')\n-sys.stderr.write('WARNING: For more information visit https://github.com/felipec/git-remote-hg\\n')\n-\n-sys.stderr.write('''WARNING:\n+cat <<'EOT'\n+WARNING: git-remote-hg is now maintained independently.\n+WARNING: For more information visit https://github.com/felipec/git-remote-hg\n+WARNING:\n WARNING: You can pick a directory on your $PATH and download it, e.g.:\n-WARNING: $ wget -O $HOME/bin/git-remote-hg \\\\\n+WARNING: $ wget -O $HOME/bin/git-remote-hg \\\n WARNING: https://raw.github.com/felipec/git-remote-hg/master/git-remote-hg\n WARNING: $ chmod +x $HOME/bin/git-remote-hg\n-''')\n+EOT\n\n--\nhttps://github.com/git/git/pull/333\n"},{"id":"313168","messageId":"20170303112104.2n3hu23sajpn2x2z@sigill.intra.peff.net","threadId":"45269","inReplyTo":"0102015a93d18443-c574aaf8-5656-49e8-8848-810e1fadc87a-000000@eu-west-1.amazonses.com","subject":"Re: [PATCH] Do not require Python for the git-remote-{bzr,hg} placeholder scripts","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2017-03-03T11:21:04Z","receivedAt":"2017-03-03T12:23:41Z","isPatch":true,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Fri, Mar 03, 2017 at 10:57:46AM +0000, Sebastian Schuberth wrote:\n\n> It does not make sense for these placeholder scripts to depend on Python\n> just because the real scripts do. At the example of Git for Windows, we\n> would not even be able to see those warnings as it does not ship with\n> Python. So just use plain shell scripts instead.\n\nYeah, this seems like an obvious improvement. I think we got here\nbecause the originals issued a warning but kept working, and then it was\nslowly whittled down to remove the \"working\" part.\n\nAt some point these can probably go away. It's been 3 years since they\nturned into nothing but warnings, so presumably most people have\nupgraded by now (I know people often go a long time on old\ndistro-packaged versions, but the distro packagers would presumably\nfigure this out in the meantime).\n\nOTOH, it is not really hurting much, so I do not mind keeping them\naround for another 3 years (or more) just to catch any stragglers.\n\n-Peff\n"},{"id":"313187","messageId":"a9273d37-388a-8c27-1cf1-51e9cb28ef55@xiplink.com","threadId":"45269","inReplyTo":"0102015a93d18443-c574aaf8-5656-49e8-8848-810e1fadc87a-000000@eu-west-1.amazonses.com","subject":"Re: [PATCH] Do not require Python for the git-remote-{bzr,hg} placeholder scripts","fromName":"Marc Branchaud","fromEmail":"marcnarc@xiplink.com","sentAt":"2017-03-03T15:32:29Z","receivedAt":"2017-03-03T15:43:15Z","isPatch":true,"sender":{"key":"marcnarc@xiplink.com","avatar":"https://avatars.githubusercontent.com/u/14980203?v=4"},"body":"On 2017-03-03 05:57 AM, Sebastian Schuberth wrote:\n> It does not make sense for these placeholder scripts to depend on Python\n> just because the real scripts do. At the example of Git for Windows, we\n> would not even be able to see those warnings as it does not ship with\n> Python. So just use plain shell scripts instead.\n\nJust a niggle: This change moves the warning message from stderr to stdout.\n\n\t\tM.\n\n> Signed-off-by: Sebastian Schuberth \n> ---\n> contrib/remote-helpers/git-remote-bzr | 16 +++++++---------\n> contrib/remote-helpers/git-remote-hg | 16 +++++++---------\n> 2 files changed, 14 insertions(+), 18 deletions(-)\n>\n> diff --git a/contrib/remote-helpers/git-remote-bzr b/contrib/remote-helpers/git-remote-bzr\n> index 712a137..ccc4aea 100755\n> --- a/contrib/remote-helpers/git-remote-bzr\n> +++ b/contrib/remote-helpers/git-remote-bzr\n> @@ -1,13 +1,11 @@\n> -#!/usr/bin/env python\n> +#!/bin/sh\n>\n> -import sys\n> -\n> -sys.stderr.write('WARNING: git-remote-bzr is now maintained independently.\\n')\n> -sys.stderr.write('WARNING: For more information visit https://github.com/felipec/git-remote-bzr\\n')\n> -\n> -sys.stderr.write('''WARNING:\n> +cat <<'EOT'\n> +WARNING: git-remote-bzr is now maintained independently.\n> +WARNING: For more information visit https://github.com/felipec/git-remote-bzr\n> +WARNING:\n> WARNING: You can pick a directory on your $PATH and download it, e.g.:\n> -WARNING: $ wget -O $HOME/bin/git-remote-bzr \\\\\n> +WARNING: $ wget -O $HOME/bin/git-remote-bzr \\\n> WARNING: https://raw.github.com/felipec/git-remote-bzr/master/git-remote-bzr\n> WARNING: $ chmod +x $HOME/bin/git-remote-bzr\n> -''')\n> +EOT\n> diff --git a/contrib/remote-helpers/git-remote-hg b/contrib/remote-helpers/git-remote-hg\n> index 4255ad6..dfda44f 100755\n> --- a/contrib/remote-helpers/git-remote-hg\n> +++ b/contrib/remote-helpers/git-remote-hg\n> @@ -1,13 +1,11 @@\n> -#!/usr/bin/env python\n> +#!/bin/sh\n>\n> -import sys\n> -\n> -sys.stderr.write('WARNING: git-remote-hg is now maintained independently.\\n')\n> -sys.stderr.write('WARNING: For more information visit https://github.com/felipec/git-remote-hg\\n')\n> -\n> -sys.stderr.write('''WARNING:\n> +cat <<'EOT'\n> +WARNING: git-remote-hg is now maintained independently.\n> +WARNING: For more information visit https://github.com/felipec/git-remote-hg\n> +WARNING:\n> WARNING: You can pick a directory on your $PATH and download it, e.g.:\n> -WARNING: $ wget -O $HOME/bin/git-remote-hg \\\\\n> +WARNING: $ wget -O $HOME/bin/git-remote-hg \\\n> WARNING: https://raw.github.com/felipec/git-remote-hg/master/git-remote-hg\n> WARNING: $ chmod +x $HOME/bin/git-remote-hg\n> -''')\n> +EOT\n>\n> --\n> https://github.com/git/git/pull/333\n>\n"},{"id":"313219","messageId":"xmqqzih289js.fsf@gitster.mtv.corp.google.com","threadId":"45269","inReplyTo":"20170303112104.2n3hu23sajpn2x2z@sigill.intra.peff.net","subject":"Re: [PATCH] Do not require Python for the git-remote-{bzr,hg} placeholder scripts","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-03-03T19:06:15Z","receivedAt":"2017-03-03T19:09:43Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jeff King writes:\n\n> On Fri, Mar 03, 2017 at 10:57:46AM +0000, Sebastian Schuberth wrote:\n>\n>> It does not make sense for these placeholder scripts to depend on Python\n>> just because the real scripts do. At the example of Git for Windows, we\n>> would not even be able to see those warnings as it does not ship with\n>> Python. So just use plain shell scripts instead.\n>\n> Yeah, this seems like an obvious improvement. I think we got here\n> because the originals issued a warning but kept working, and then it was\n> slowly whittled down to remove the \"working\" part.\n> ...\n> OTOH, it is not really hurting much, so I do not mind keeping them\n> around for another 3 years (or more) just to catch any stragglers.\n\nYup, let's queue it and remove it in a few years.\n\nThanks.\n"},{"id":"313225","messageId":"xmqqtw7a89dg.fsf@gitster.mtv.corp.google.com","threadId":"45269","inReplyTo":"a9273d37-388a-8c27-1cf1-51e9cb28ef55@xiplink.com","subject":"Re: [PATCH] Do not require Python for the git-remote-{bzr,hg} placeholder scripts","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2017-03-03T19:10:03Z","receivedAt":"2017-03-03T19:33:12Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Marc Branchaud writes:\n\n> On 2017-03-03 05:57 AM, Sebastian Schuberth wrote:\n>> It does not make sense for these placeholder scripts to depend on Python\n>> just because the real scripts do. At the example of Git for Windows, we\n>> would not even be able to see those warnings as it does not ship with\n>> Python. So just use plain shell scripts instead.\n>\n> Just a niggle: This change moves the warning message from stderr to stdout.\n\nRight. Here is what I'll queue.\n\n-- >8 --\nFrom: Sebastian Schuberth \nDate: Fri, 3 Mar 2017 10:57:46 +0000\nSubject: [PATCH] contrib: git-remote-{bzr,hg} placeholders don't need Python\n\nIt does not make sense for these placeholder scripts to depend on Python\njust because the real scripts do. At the example of Git for Windows, we\nwould not even be able to see those warnings as it does not ship with\nPython. So just use plain shell scripts instead.\n\nSigned-off-by: Sebastian Schuberth \nReviewed-by: Jeff King \nSigned-off-by: Junio C Hamano \n---\n contrib/remote-helpers/git-remote-bzr | 16 +++++++---------\n contrib/remote-helpers/git-remote-hg | 16 +++++++---------\n 2 files changed, 14 insertions(+), 18 deletions(-)\n\ndiff --git a/contrib/remote-helpers/git-remote-bzr b/contrib/remote-helpers/git-remote-bzr\nindex 712a1377e2..ccc4aea362 100755\n--- a/contrib/remote-helpers/git-remote-bzr\n+++ b/contrib/remote-helpers/git-remote-bzr\n@@ -1,13 +1,11 @@\n-#!/usr/bin/env python\n+#!/bin/sh\n \n-import sys\n-\n-sys.stderr.write('WARNING: git-remote-bzr is now maintained independently.\\n')\n-sys.stderr.write('WARNING: For more information visit https://github.com/felipec/git-remote-bzr\\n')\n-\n-sys.stderr.write('''WARNING:\n+cat >&2 <<'EOT'\n+WARNING: git-remote-bzr is now maintained independently.\n+WARNING: For more information visit https://github.com/felipec/git-remote-bzr\n+WARNING:\n WARNING: You can pick a directory on your $PATH and download it, e.g.:\n-WARNING: $ wget -O $HOME/bin/git-remote-bzr \\\\\n+WARNING: $ wget -O $HOME/bin/git-remote-bzr \\\n WARNING: https://raw.github.com/felipec/git-remote-bzr/master/git-remote-bzr\n WARNING: $ chmod +x $HOME/bin/git-remote-bzr\n-''')\n+EOT\ndiff --git a/contrib/remote-helpers/git-remote-hg b/contrib/remote-helpers/git-remote-hg\nindex 4255ad6312..dfda44f311 100755\n--- a/contrib/remote-helpers/git-remote-hg\n+++ b/contrib/remote-helpers/git-remote-hg\n@@ -1,13 +1,11 @@\n-#!/usr/bin/env python\n+#!/bin/sh\n \n-import sys\n-\n-sys.stderr.write('WARNING: git-remote-hg is now maintained independently.\\n')\n-sys.stderr.write('WARNING: For more information visit https://github.com/felipec/git-remote-hg\\n')\n-\n-sys.stderr.write('''WARNING:\n+cat >&2 <<'EOT'\n+WARNING: git-remote-hg is now maintained independently.\n+WARNING: For more information visit https://github.com/felipec/git-remote-hg\n+WARNING:\n WARNING: You can pick a directory on your $PATH and download it, e.g.:\n-WARNING: $ wget -O $HOME/bin/git-remote-hg \\\\\n+WARNING: $ wget -O $HOME/bin/git-remote-hg \\\n WARNING: https://raw.github.com/felipec/git-remote-hg/master/git-remote-hg\n WARNING: $ chmod +x $HOME/bin/git-remote-hg\n-''')\n+EOT\n-- \n2.12.0-368-g85767a6c71\n\n"},{"id":"313228","messageId":"CAHGBnuP8X3xOOwfkUrbnF8E9zPwkSGq2y0oQpdLZt-NTpGMuLg@mail.gmail.com","threadId":"45269","inReplyTo":"xmqqtw7a89dg.fsf@gitster.mtv.corp.google.com","subject":"Re: [PATCH] Do not require Python for the git-remote-{bzr,hg} placeholder scripts","fromName":"Sebastian Schuberth","fromEmail":"sschuberth@gmail.com","sentAt":"2017-03-03T20:13:08Z","receivedAt":"2017-03-03T20:14:52Z","isPatch":true,"sender":{"key":"sschuberth@gmail.com","avatar":"https://avatars.githubusercontent.com/u/349154?v=4"},"body":"On Fri, Mar 3, 2017 at 8:10 PM, Junio C Hamano wrote:\n\n>> Just a niggle: This change moves the warning message from stderr to stdout.\n>\n> Right. Here is what I'll queue.\n\nIndeed, thanks for the note, and also Junio for fixing while queuing.\n\n-- \nSebastian Schuberth\n"}]}