Skip to contentSkip to conversation

package/mke2img: "/bin/bash" => "/usr/bin/env bash" shebang

PATCH New Bjørn Forsman 2 participants 3 messages Jan 8, 2015

lore
Share & tools
lore

Bjørn Forsman

author
#0 lore raw

PATCH 1/1 package/mke2img: "/bin/bash" => "/usr/bin/env bash" shebang

For portability. All other Buildroot scripts (i.e. scripts that run on host) already use the "/usr/bin/env bash" shebang.

This change is needed for NixOS, which lacks a global /bin/bash.

Signed-off-by Bj?rn Forsman <[email protected]>
---
 package/mke2img/mke2img | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
package/mke2img/mke2img +1 -1
index 68da6ed..5e2f62f 100755
@@ -1,4 +1,4 @@
-#!/bin/bash
+#!/usr/bin/env bash
 
 # Buildroot wrapper to the collection of ext2/3/4 filesystem tools:
 # - genext2fs, to generate ext2 filesystem images
Signature · 1 line
2.1.3
Applied
#1 lore raw

PATCH 1/1 package/mke2img: "/bin/bash" => "/usr/bin/env bash" shebang

Dear Bj?rn Forsman,

Quoted 9 lines · Bj?rn Forsman

For portability. All other Buildroot scripts (i.e. scripts that run on host) already use the "/usr/bin/env bash" shebang.

This change is needed for NixOS, which lacks a global /bin/bash.

Signed-off-by Bj?rn Forsman <[email protected]>
---
 package/mke2img/mke2img | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Applied, thanks!

(See: we can be fast! :-))

Thomas

Signature · 3 lines
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
author
#2 lore raw

PATCH 1/1 package/mke2img: "/bin/bash" => "/usr/bin/env bash" shebang

Quoted 8 lines · Thomas Petazzoni

Dear Bj?rn Forsman,

Quoted 5 lines · Bj?rn Forsman

For portability. All other Buildroot scripts (i.e. scripts that run on host) already use the "/usr/bin/env bash" shebang.

This change is needed for NixOS, which lacks a global /bin/bash.

Quoted 8 lines
Quoted 4 lines
Signed-off-by Bj?rn Forsman <[email protected]>
---
 package/mke2img/mke2img | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Applied, thanks!

(See: we can be fast! :-))

Indeed! Thanks :-)

- Bj?rn