[arch-projects] [PATCH] [abs] Update shebangs to use /usr/bin instead of /bin due to merging of the "bin" directories
From: "Jason St. John" <jstjohn@purdue.edu> The "bin" directories are being merged into /usr/bin [1], so it is appropriate to update the shebangs as well. [1] https://mailman.archlinux.org/pipermail/arch-dev-public/2013-May/024908.html Signed-off-by: Jason St. John <jstjohn@purdue.edu> --- abs.in | 2 +- makeworld.in | 2 +- scripts/svn2abs.in | 2 +- scripts/update-abs.sh | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git a/abs.in b/abs.in index 19d01d6..ad81499 100644 --- a/abs.in +++ b/abs.in @@ -1,4 +1,4 @@ -#!/bin/bash -e +#!/usr/bin/bash -e # # abs - download a PKGBUILD tree from a CVS repository # diff --git a/makeworld.in b/makeworld.in index e5d9f0f..0ac8447 100644 --- a/makeworld.in +++ b/makeworld.in @@ -1,4 +1,4 @@ -#!/bin/bash +#!/usr/bin/bash # # makeworld - a makepkg wrapper to build multiple packages # diff --git a/scripts/svn2abs.in b/scripts/svn2abs.in index 0d4c74d..d8a60c9 100644 --- a/scripts/svn2abs.in +++ b/scripts/svn2abs.in @@ -1,4 +1,4 @@ -#!/bin/bash +#!/usr/bin/bash # # svn2abs - convert svn repo to ABS structure # diff --git a/scripts/update-abs.sh b/scripts/update-abs.sh index ff3510e..54ba42c 100644 --- a/scripts/update-abs.sh +++ b/scripts/update-abs.sh @@ -1,4 +1,4 @@ -#!/bin/bash +#!/usr/bin/bash SERVERS=('gerolde' 'sigurd') ARCHES=('i686' 'x86_64' 'any') -- 1.8.3
On Jun 2, 2013 10:01 AM, "Jason St. John" <jstjohn@purdue.edu> wrote:
From: "Jason St. John" <jstjohn@purdue.edu>
The "bin" directories are being merged into /usr/bin [1], so it is appropriate to update the shebangs as well.
[1]
https://mailman.archlinux.org/pipermail/arch-dev-public/2013-May/024908.html
Signed-off-by: Jason St. John <jstjohn@purdue.edu> --- abs.in | 2 +- makeworld.in | 2 +- scripts/svn2abs.in | 2 +- scripts/update-abs.sh | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/abs.in b/abs.in index 19d01d6..ad81499 100644 --- a/abs.in +++ b/abs.in @@ -1,4 +1,4 @@ -#!/bin/bash -e +#!/usr/bin/bash -e
Nak. /bin/bash will never be invalid.
# # abs - download a PKGBUILD tree from a CVS repository # diff --git a/makeworld.in b/makeworld.in index e5d9f0f..0ac8447 100644 --- a/makeworld.in +++ b/makeworld.in @@ -1,4 +1,4 @@ -#!/bin/bash +#!/usr/bin/bash # # makeworld - a makepkg wrapper to build multiple packages # diff --git a/scripts/svn2abs.in b/scripts/svn2abs.in index 0d4c74d..d8a60c9 100644 --- a/scripts/svn2abs.in +++ b/scripts/svn2abs.in @@ -1,4 +1,4 @@ -#!/bin/bash +#!/usr/bin/bash # # svn2abs - convert svn repo to ABS structure # diff --git a/scripts/update-abs.sh b/scripts/update-abs.sh index ff3510e..54ba42c 100644 --- a/scripts/update-abs.sh +++ b/scripts/update-abs.sh @@ -1,4 +1,4 @@ -#!/bin/bash +#!/usr/bin/bash
SERVERS=('gerolde' 'sigurd') ARCHES=('i686' 'x86_64' 'any') -- 1.8.3
On Sun, Jun 2, 2013 at 4:03 PM, Dave Reisner <d@falconindy.com> wrote:
On Jun 2, 2013 10:01 AM, "Jason St. John" <jstjohn@purdue.edu> wrote:
From: "Jason St. John" <jstjohn@purdue.edu>
The "bin" directories are being merged into /usr/bin [1], so it is appropriate to update the shebangs as well.
[1] https://mailman.archlinux.org/pipermail/arch-dev-public/2013-May/024908.html
Signed-off-by: Jason St. John <jstjohn@purdue.edu> --- abs.in | 2 +- makeworld.in | 2 +- scripts/svn2abs.in | 2 +- scripts/update-abs.sh | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/abs.in b/abs.in index 19d01d6..ad81499 100644 --- a/abs.in +++ b/abs.in @@ -1,4 +1,4 @@ -#!/bin/bash -e +#!/usr/bin/bash -e
Nak. /bin/bash will never be invalid.
# # abs - download a PKGBUILD tree from a CVS repository # diff --git a/makeworld.in b/makeworld.in index e5d9f0f..0ac8447 100644 --- a/makeworld.in +++ b/makeworld.in @@ -1,4 +1,4 @@ -#!/bin/bash +#!/usr/bin/bash # # makeworld - a makepkg wrapper to build multiple packages # diff --git a/scripts/svn2abs.in b/scripts/svn2abs.in index 0d4c74d..d8a60c9 100644 --- a/scripts/svn2abs.in +++ b/scripts/svn2abs.in @@ -1,4 +1,4 @@ -#!/bin/bash +#!/usr/bin/bash # # svn2abs - convert svn repo to ABS structure # diff --git a/scripts/update-abs.sh b/scripts/update-abs.sh index ff3510e..54ba42c 100644 --- a/scripts/update-abs.sh +++ b/scripts/update-abs.sh @@ -1,4 +1,4 @@ -#!/bin/bash +#!/usr/bin/bash
SERVERS=('gerolde' 'sigurd') ARCHES=('i686' 'x86_64' 'any') -- 1.8.3
But it isn't a problem to bypass the symlink, so why not update it? Jason
It then breaks on every system where bash isn't accessible via /usr/bin. The only acceptable bash shebangs are: #!/bin/bash #!/use/bin/env bash On Jun 2, 2013 10:30 AM, "Jason St. John" <jstjohn@purdue.edu> wrote:
On Sun, Jun 2, 2013 at 4:03 PM, Dave Reisner <d@falconindy.com> wrote:
On Jun 2, 2013 10:01 AM, "Jason St. John" <jstjohn@purdue.edu> wrote:
From: "Jason St. John" <jstjohn@purdue.edu>
The "bin" directories are being merged into /usr/bin [1], so it is appropriate to update the shebangs as well.
[1]
https://mailman.archlinux.org/pipermail/arch-dev-public/2013-May/024908.html
Signed-off-by: Jason St. John <jstjohn@purdue.edu> --- abs.in | 2 +- makeworld.in | 2 +- scripts/svn2abs.in | 2 +- scripts/update-abs.sh | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/abs.in b/abs.in index 19d01d6..ad81499 100644 --- a/abs.in +++ b/abs.in @@ -1,4 +1,4 @@ -#!/bin/bash -e +#!/usr/bin/bash -e
Nak. /bin/bash will never be invalid.
# # abs - download a PKGBUILD tree from a CVS repository # diff --git a/makeworld.in b/makeworld.in index e5d9f0f..0ac8447 100644 --- a/makeworld.in +++ b/makeworld.in @@ -1,4 +1,4 @@ -#!/bin/bash +#!/usr/bin/bash # # makeworld - a makepkg wrapper to build multiple packages # diff --git a/scripts/svn2abs.in b/scripts/svn2abs.in index 0d4c74d..d8a60c9 100644 --- a/scripts/svn2abs.in +++ b/scripts/svn2abs.in @@ -1,4 +1,4 @@ -#!/bin/bash +#!/usr/bin/bash # # svn2abs - convert svn repo to ABS structure # diff --git a/scripts/update-abs.sh b/scripts/update-abs.sh index ff3510e..54ba42c 100644 --- a/scripts/update-abs.sh +++ b/scripts/update-abs.sh @@ -1,4 +1,4 @@ -#!/bin/bash +#!/usr/bin/bash
SERVERS=('gerolde' 'sigurd') ARCHES=('i686' 'x86_64' 'any') -- 1.8.3
But it isn't a problem to bypass the symlink, so why not update it?
Jason
On Sun, Jun 2, 2013 at 4:33 PM, Dave Reisner <d@falconindy.com> wrote:
It then breaks on every system where bash isn't accessible via /usr/bin. The only acceptable bash shebangs are:
#!/bin/bash #!/use/bin/env bash
On Jun 2, 2013 10:30 AM, "Jason St. John" <jstjohn@purdue.edu> wrote:
On Sun, Jun 2, 2013 at 4:03 PM, Dave Reisner <d@falconindy.com> wrote:
On Jun 2, 2013 10:01 AM, "Jason St. John" <jstjohn@purdue.edu> wrote:
From: "Jason St. John" <jstjohn@purdue.edu>
The "bin" directories are being merged into /usr/bin [1], so it is appropriate to update the shebangs as well.
[1]
https://mailman.archlinux.org/pipermail/arch-dev-public/2013-May/024908.html
Signed-off-by: Jason St. John <jstjohn@purdue.edu> --- abs.in | 2 +- makeworld.in | 2 +- scripts/svn2abs.in | 2 +- scripts/update-abs.sh | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/abs.in b/abs.in index 19d01d6..ad81499 100644 --- a/abs.in +++ b/abs.in @@ -1,4 +1,4 @@ -#!/bin/bash -e +#!/usr/bin/bash -e
Nak. /bin/bash will never be invalid.
# # abs - download a PKGBUILD tree from a CVS repository # diff --git a/makeworld.in b/makeworld.in index e5d9f0f..0ac8447 100644 --- a/makeworld.in +++ b/makeworld.in @@ -1,4 +1,4 @@ -#!/bin/bash +#!/usr/bin/bash # # makeworld - a makepkg wrapper to build multiple packages # diff --git a/scripts/svn2abs.in b/scripts/svn2abs.in index 0d4c74d..d8a60c9 100644 --- a/scripts/svn2abs.in +++ b/scripts/svn2abs.in @@ -1,4 +1,4 @@ -#!/bin/bash +#!/usr/bin/bash # # svn2abs - convert svn repo to ABS structure # diff --git a/scripts/update-abs.sh b/scripts/update-abs.sh index ff3510e..54ba42c 100644 --- a/scripts/update-abs.sh +++ b/scripts/update-abs.sh @@ -1,4 +1,4 @@ -#!/bin/bash +#!/usr/bin/bash
SERVERS=('gerolde' 'sigurd') ARCHES=('i686' 'x86_64' 'any') -- 1.8.3
But it isn't a problem to bypass the symlink, so why not update it?
Jason
Should I resubmit using "#!/usr/bin/env bash"?
On Jun 2, 2013 11:21 AM, "Jason St. John" <jstjohn@purdue.edu> wrote:
On Sun, Jun 2, 2013 at 4:33 PM, Dave Reisner <d@falconindy.com> wrote:
It then breaks on every system where bash isn't accessible via
/usr/bin. The
only acceptable bash shebangs are:
#!/bin/bash #!/use/bin/env bash
On Jun 2, 2013 10:30 AM, "Jason St. John" <jstjohn@purdue.edu> wrote:
On Sun, Jun 2, 2013 at 4:03 PM, Dave Reisner <d@falconindy.com> wrote:
On Jun 2, 2013 10:01 AM, "Jason St. John" <jstjohn@purdue.edu> wrote:
From: "Jason St. John" <jstjohn@purdue.edu>
The "bin" directories are being merged into /usr/bin [1], so it is appropriate to update the shebangs as well.
[1]
https://mailman.archlinux.org/pipermail/arch-dev-public/2013-May/024908.html
Signed-off-by: Jason St. John <jstjohn@purdue.edu> --- abs.in | 2 +- makeworld.in | 2 +- scripts/svn2abs.in | 2 +- scripts/update-abs.sh | 2 +- 4 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/abs.in b/abs.in index 19d01d6..ad81499 100644 --- a/abs.in +++ b/abs.in @@ -1,4 +1,4 @@ -#!/bin/bash -e +#!/usr/bin/bash -e
Nak. /bin/bash will never be invalid.
# # abs - download a PKGBUILD tree from a CVS repository # diff --git a/makeworld.in b/makeworld.in index e5d9f0f..0ac8447 100644 --- a/makeworld.in +++ b/makeworld.in @@ -1,4 +1,4 @@ -#!/bin/bash +#!/usr/bin/bash # # makeworld - a makepkg wrapper to build multiple packages # diff --git a/scripts/svn2abs.in b/scripts/svn2abs.in index 0d4c74d..d8a60c9 100644 --- a/scripts/svn2abs.in +++ b/scripts/svn2abs.in @@ -1,4 +1,4 @@ -#!/bin/bash +#!/usr/bin/bash # # svn2abs - convert svn repo to ABS structure # diff --git a/scripts/update-abs.sh b/scripts/update-abs.sh index ff3510e..54ba42c 100644 --- a/scripts/update-abs.sh +++ b/scripts/update-abs.sh @@ -1,4 +1,4 @@ -#!/bin/bash +#!/usr/bin/bash
SERVERS=('gerolde' 'sigurd') ARCHES=('i686' 'x86_64' 'any') -- 1.8.3
But it isn't a problem to bypass the symlink, so why not update it?
Jason
Should I resubmit using "#!/usr/bin/env bash"?
Please don't. I see no value in doing so.
participants (2)
-
Dave Reisner
-
Jason St. John