pub's binstubs should set #!/bin/bash explicitly

[DartBot]

This issue was originally filed by @seaneagan

---

From seaneagan/den#7:

This seems to be the culprit http://stackoverflow.com/a/17753098/217408 (/bin/sh -> dash) and dash seems not to support [[ according to the SO answer.

Do you consider changing the first line in /home/myuser/.pub-cache/bin/den to #!/bin/bash, which fixes the issue for me?

[[ is bash specific and according to http://lwn.net/Articles/343924/ the script file should set #!/bin/bash explicitly in this case. Contains some other interesting background I wasn't aware of.

[sethladd]

Not everyone uses bash, though?

---

Removed Type-Defect label.
Added Type-Enhancement, Area-Pub, Triaged labels.

[nex3]

We could also just use [ ] rather than [[ ]], right?

[DartBot]

This comment was originally written by @seaneagan

---

They don't have to use it though, just have to have it installed, so that
the binstubs can use it.

[nex3]

Set owner to @nex3.
Added Started label.

[nex3]

Fixed in r42308.

---

Added Fixed label.

[DartBot]

This comment was originally written by @zoechi

---

Great! Works fine now.

[DartBot]

This comment was originally written by @seaneagan

---

Thanks! Looks like the pub executable also uses [[, but it has #!/bin/bash. Should that be changed to be consistent with the binstubs? Not sure if the pub executable uses any other bash-specific features.

Cygwin is also not properly supported by the binstubs and the pub executable, see issue #20822. This means you have to call pub.bat or e.g. den.bat instead of just pub or den, and also that it's impossible to write shell scripts that call these executables and work cross-platform (cygwin and linux/mac).

[nex3]

Thanks! Looks like the pub executable also uses [[, but it has #!/bin/bash. Should that be changed to be consistent with the binstubs? Not sure if the pub executable uses any other bash-specific features.

Feel free to file an issue for this, although it's much less urgent since it's not actually breaking anyone.

Cygwin is also not properly supported by the binstubs and the pub executable, see issue #20822.

I believe Cygwin is not an officially supported platform right now.

[DartBot]

This issue has been moved to dart-lang/pub#1211.