Fix cr-bash-helpers for Mac w/ Homebrew, which doesn't have realpath.

Fix cr-bash-helpers for Mac w/ Homebrew, which doesn't have realpath.

readlink is in gnu coreutils, which should be more widely available. On my
machine, the gnu version is prefixed with 'g', while 'readlink' is a crippled
Mac version without the -e option.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=248853

[Jeffrey Yasskin, 2014-02-03 23:07:25]

It looks like `cr` doesn't actually work on Mac, but here's one step closer.

[bulach, 2014-02-04 05:51:24]

+ian, de facto owner...and lgtm % nit, thanks!

https://codereview.chromium.org/144243004/diff/1/tools/cr/cr-bash-helpers.sh
File tools/cr/cr-bash-helpers.sh (right):

https://codereview.chromium.org/144243004/diff/1/tools/cr/cr-bash-helpers.sh#...
tools/cr/cr-bash-helpers.sh:19: READLINK_e="greadlink -e"
nit: unindent (as 13-14 above)

[Jeffrey Yasskin, 2014-02-04 06:59:03]

https://codereview.chromium.org/144243004/diff/1/tools/cr/cr-bash-helpers.sh
File tools/cr/cr-bash-helpers.sh (right):

https://codereview.chromium.org/144243004/diff/1/tools/cr/cr-bash-helpers.sh#...
tools/cr/cr-bash-helpers.sh:19: READLINK_e="greadlink -e"
On 2014/02/04 05:51:25, bulach wrote:
> nit: unindent (as 13-14 above)

Oops, fixed.

[Jeffrey Yasskin, 2014-02-04 16:46:05]

The CQ bit was checked by jyasskin@chromium.org

[commit-bot: I haz the power, 2014-02-04 16:48:43]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jyasskin@chromium.org/144243004/60001

[commit-bot: I haz the power, 2014-02-04 17:37:06]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-02-04 17:37:06]

Retried try job too often on mac_rel for step(s) base_unittests, browser_tests,
interactive_ui_tests, net_unittests, unit_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[Jeffrey Yasskin, 2014-02-04 17:39:08]

The CQ bit was checked by jyasskin@chromium.org

[commit-bot: I haz the power, 2014-02-04 17:42:12]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jyasskin@chromium.org/144243004/60001

[commit-bot: I haz the power, 2014-02-04 19:28:37]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-02-04 19:28:38]

Retried try job too often on ios_dbg_simulator for step(s) url_unittests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=ios_dbg_si...

[Jeffrey Yasskin, 2014-02-04 21:46:10]

The CQ bit was checked by jyasskin@chromium.org

[commit-bot: I haz the power, 2014-02-04 22:29:13]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jyasskin@chromium.org/144243004/60001

[commit-bot: I haz the power, 2014-02-05 01:04:21]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jyasskin@chromium.org/144243004/60001

[commit-bot: I haz the power, 2014-02-05 05:05:18]

Change committed as 248853