GHC 2018-04-08

7 comments.

, https://git.io/vx7UZ in Homebrew/homebrew-core
Ref: https://www.zsh.org/mla/zsh-announce/134

, https://git.io/vx7Un in Homebrew/homebrew-core
It's been released.

, https://git.io/vxQSD in jarun/googler
By the way, you should also quote `$1` to avoid shell injection, unless you use e.g. zsh in the shebang, where quoting is on by default.

, https://git.io/vxQSy in jarun/googler
No problem.

, https://git.io/vxQSS in jarun/googler
Okay, I thought your `reader` is just the `reader.py`. Apparently your `reader` doesn't even have a shebang, hence the exec format error.

, https://git.io/vxQXy in jarun/googler
Shouldn't matter, s/he claimed it's in PATH and executable, unless the claim's false. Python doesn't actively modify your PATH in runtime, we don't either, and there's no shell rc in the way since we don't do `shell=True`.

, https://git.io/vxQXS in jarun/googler
Can't reproduce. Your python just can't launch your `reader`, so your observation about `BROWSER` is expected, as we suppress both stdout and stderr by default for browsers not on the whitelist.

Try these:

```
$ python3 -c 'import subprocess; p = subprocess.Popen(["reader", "https://example.com"]); p.communicate()'
$ python3 -c 'import subprocess; p = subprocess.Popen(["/Users/me/.local/bin/reader", "https://example.com"]); p.communicate()'
$ python3 -c 'import subprocess; subprocess.check_call(["reader", "https://example.com"])'
$ head /Users/me/.local/bin/reader
```