You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sqoop.apache.org by Stuart Williams <st...@sidereal.ca> on 2015/08/27 04:23:04 UTC

Review Request 37828: SQOOP-2531: readlink -f not supported on OS X

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37828/
-----------------------------------------------------------

Review request for Sqoop.


Summary (updated)
-----------------

SQOOP-2531: readlink -f not supported on OS X


Repository: sqoop-trunk


Description (updated)
-------

Tool scripts (sqoop-toolname) rely on "readlink -f" to get the path to the script, which is not an available readlink option on OS X.

Functions to solve this are already in bin/sqoop (to follow a file link to the final target). Added those to the tool script template.


Diffs (updated)
-----

  src/scripts/tool-script.sh.template 56b5a1a 

Diff: https://reviews.apache.org/r/37828/diff/


Testing (updated)
-------

Manually tested updated tool scripts on OS X 10.10.5.


Thanks,

Stuart Williams


Re: Review Request 37828: SQOOP-2531: readlink -f not supported on OS X

Posted by Stuart Williams <st...@sidereal.ca>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37828/
-----------------------------------------------------------

(Updated Aug. 27, 2015, 2:24 a.m.)


Review request for Sqoop.


Repository: sqoop-trunk


Description
-------

Tool scripts (sqoop-toolname) rely on "readlink -f" to get the path to the script, which is not an available readlink option on OS X.

Functions to solve this are already in bin/sqoop (to follow a file link to the final target). Added those to the tool script template.


Diffs
-----

  src/scripts/tool-script.sh.template 56b5a1a 

Diff: https://reviews.apache.org/r/37828/diff/


Testing
-------

Manually tested updated tool scripts on OS X 10.10.5.


Thanks,

Stuart Williams