Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Obtain baseDir through more portable means. #27

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

aeriksson
Copy link

readlink -f does not work on OS X.

This commit replaces the use of readlink -f with something more
portable.

While this approach may not always fully resolve symlinks, it should be
fine since java seems to resolve symlinks.

@aeriksson
Copy link
Author

This fixes #26

@aeriksson aeriksson closed this Jan 12, 2015
@aeriksson aeriksson reopened this Jan 12, 2015
`readlink -f` does not work on OS X.

This commit replaces the use of `readlink -f` with something more
portable.

While this approach may not always fully resolve symlinks, it should be
fine since java seems to resolve symlinks.
@rakeshnd
Copy link

I tried this but doesn't seem to resolve the issue. I still get the same error #26

@aeriksson
Copy link
Author

@rakeshchalasani: That shouldn't be possible (the error is readlink-specific, and this pull request removes the only use of readlink). Did you build from https://github.com/aeriksson/IScala ?

@aeriksson aeriksson closed this Feb 19, 2015
@aeriksson aeriksson reopened this Feb 19, 2015
@rakeshnd
Copy link

Sorry about that, I changed a couple of things too many before I tried this patch. Building from your repo works fine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants