# contrib/diff-highlight: stop hard-coding perl location 4 messages from 2016-02-22 to 2016-02-22. Participants: Peter Dave Hello, Eric Sunshine. Thread: https://gitlist.dev/t/41467 ## Peter Dave Hello, 2016-02-22 05:54 Subject: contrib/diff-highlight: stop hard-coding perl location Message-ID: URL: https://gitlist.dev/e/CA%2BboQ6YWWkudgwC7zn-3UPWqfkw%2BuJZgwcBWcGvJDcCUakjzHA%40mail.gmail.com ``` From: Peter Dave Hello Use `#!/usr/bin/env perl` instead of `#!/usr/bin/perl`, the util "env" can help located the "perl", so that it can work on FreeBSD and other platforms. Signed-off-by: Peter Dave Hello --- contrib/diff-highlight/diff-highlight | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/contrib/diff-highlight/diff-highlight b/contrib/diff-highlight/diff-highlight index ffefc31..b57b0fd 100755 --- a/contrib/diff-highlight/diff-highlight +++ b/contrib/diff-highlight/diff-highlight @@ -1,4 +1,4 @@ -#!/usr/bin/perl +#!/usr/bin/env perl use 5.008; use warnings FATAL => 'all'; -- ``` ## Eric Sunshine, 2016-02-22 05:56 Subject: Re: contrib/diff-highlight: stop hard-coding perl location Message-ID: URL: https://gitlist.dev/e/CAPig%2BcSE77JbD9CShrxnFsDLd9DrG9L15BSecERk_Qih9CMOig%40mail.gmail.com In-Reply-To: ``` On Mon, Feb 22, 2016 at 12:54 AM, Peter Dave Hello wrote: > From: Peter Dave Hello > > Use `#!/usr/bin/env perl` instead of `#!/usr/bin/perl`, > the util "env" can help located the "perl", > so that it can work on FreeBSD and other platforms. > > Signed-off-by: Peter Dave Hello Better, thanks. > --- > contrib/diff-highlight/diff-highlight | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/contrib/diff-highlight/diff-highlight > b/contrib/diff-highlight/diff-highlight > index ffefc31..b57b0fd 100755 > --- a/contrib/diff-highlight/diff-highlight > +++ b/contrib/diff-highlight/diff-highlight > @@ -1,4 +1,4 @@ > -#!/usr/bin/perl > +#!/usr/bin/env perl > > use 5.008; > use warnings FATAL => 'all'; ``` ## Peter Dave Hello, 2016-02-22 06:49 Subject: Re: contrib/diff-highlight: stop hard-coding perl location Message-ID: URL: https://gitlist.dev/e/CA%2BboQ6bCH35hu9GfQY_P5bNZCdJ8Knr8zQ%2BS41RGF0z-JamVgw%40mail.gmail.com In-Reply-To: ``` Hi Eric, Thank you, is there anything I should do? Or I should just wait? Best, Peter 2016/2/22 下午1:56於 "Eric Sunshine" 寫道： > > On Mon, Feb 22, 2016 at 12:54 AM, Peter Dave Hello > wrote: > > From: Peter Dave Hello > > > > Use `#!/usr/bin/env perl` instead of `#!/usr/bin/perl`, > > the util "env" can help located the "perl", > > so that it can work on FreeBSD and other platforms. > > > > Signed-off-by: Peter Dave Hello > > Better, thanks. > > > --- > > contrib/diff-highlight/diff-highlight | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/contrib/diff-highlight/diff-highlight > > b/contrib/diff-highlight/diff-highlight > > index ffefc31..b57b0fd 100755 > > --- a/contrib/diff-highlight/diff-highlight > > +++ b/contrib/diff-highlight/diff-highlight > > @@ -1,4 +1,4 @@ > > -#!/usr/bin/perl > > +#!/usr/bin/env perl > > > > use 5.008; > > use warnings FATAL => 'all'; ``` ## Eric Sunshine, 2016-02-22 07:13 Subject: Re: contrib/diff-highlight: stop hard-coding perl location Message-ID: URL: https://gitlist.dev/e/CAPig%2BcQp9H4K2D88bVxRRnwD_rY-8hROzVOp6aBCp9Y%3DY6HwUA%40mail.gmail.com In-Reply-To: ``` [please don't top-post on this list] On Mon, Feb 22, 2016 at 1:49 AM, Peter Dave Hello wrote: > Thank you, is there anything I should do? Or I should just wait? Best is to wait for other review comments, particularly from Junio. Reviewers work at their own pace as time permits, so don't be surprised if it takes days or a week or more. If you don't hear back from Junio after a week or two, and if you don't see that he has silently picked it up in his "pu" branch and don't see mention of it in Junio's "What's cooking" email after that time, then it can be helpful to send a reminder about the patch to the list. > Best, > Peter > > 2016/2/22 下午1:56於 "Eric Sunshine" 寫道： >> >> On Mon, Feb 22, 2016 at 12:54 AM, Peter Dave Hello >> wrote: >> > From: Peter Dave Hello >> > >> > Use `#!/usr/bin/env perl` instead of `#!/usr/bin/perl`, >> > the util "env" can help located the "perl", >> > so that it can work on FreeBSD and other platforms. >> > >> > Signed-off-by: Peter Dave Hello >> >> Better, thanks. >> >> > --- >> > contrib/diff-highlight/diff-highlight | 2 +- >> > 1 file changed, 1 insertion(+), 1 deletion(-) >> > >> > diff --git a/contrib/diff-highlight/diff-highlight >> > b/contrib/diff-highlight/diff-highlight >> > index ffefc31..b57b0fd 100755 >> > --- a/contrib/diff-highlight/diff-highlight >> > +++ b/contrib/diff-highlight/diff-highlight >> > @@ -1,4 +1,4 @@ >> > -#!/usr/bin/perl >> > +#!/usr/bin/env perl >> > >> > use 5.008; >> > use warnings FATAL => 'all'; > ```