Skip to content

Commit

Permalink
Merge pull request #64 from OakLabsInc/readLinkFix
Browse files Browse the repository at this point in the history
changed readlink -f to realpath -e.
  • Loading branch information
nucleardreamer authored Mar 20, 2019
2 parents ad40f98 + 6fb223d commit 8d97191
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 25 deletions.
12 changes: 0 additions & 12 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -29,18 +29,6 @@ npm install
oak-rebuild $(pwd) #directory path of wherever you want to rebuild
```

> Mac Tip: readlink is missing from OSX.
>
> To fix this run these commands:
>
> ```
> brew install coreutils
> sudo ln -s /usr/local/bin/greadlink /usr/local/bin/readlink
> ./node_modules/.bin/oak-rebuild
> ```
>
> This will allow readlink to work on OSX
### Locally

If you want to use a local version, you can install and execute oak from the `.bin` folder.
Expand Down
4 changes: 2 additions & 2 deletions bin/rebuild
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,8 @@
# at the right headers, not the local node version headers that are the default
# configuration of local npm

OAK_PATH=$(readlink -f "$(pwd)/$(dirname "${0}")/oak")
REBUILD_PATH=$(readlink -f ${1:-$(pwd)})
OAK_PATH=$(realpath -e "$(pwd)/$(dirname "${0}")/oak")
REBUILD_PATH=$(realpath -e ${1:-$(pwd)})

echo ""
echo "* Rebuilding node_modules at $REBUILD_PATH"
Expand Down
11 changes: 0 additions & 11 deletions readme.md
Original file line number Diff line number Diff line change
Expand Up @@ -30,17 +30,6 @@ oak-rebuild $(pwd) #directory path of wherever you want to rebuild
```

> Mac Tip: readlink is missing from OSX.
>
> To fix this run these commands:
>
> ```
> brew install coreutils
> sudo ln -s /usr/local/bin/greadlink /usr/local/bin/readlink
> ./node_modules/.bin/oak-rebuild
> ```
>
> This will allow readlink to work on OSX
>
> Also make sure you have installed the XCode Commandline Tools
> `xcode-select --install`
Expand Down

0 comments on commit 8d97191

Please sign in to comment.