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

Softlink jruby to jruby.bash instead of hard copy. #1199

Merged
merged 1 commit into from Nov 11, 2013

Conversation

@voxik
Copy link
Contributor

@voxik voxik commented Nov 1, 2013

Softlink will do the same job better than copy.

@mkristian
Copy link
Member

@mkristian mkristian commented Nov 1, 2013

can it be that the 'r' option is not common with all unix flavours ? ln -sr

nope: http://www.openbsd.org/cgi-bin/man.cgi?query=ln

@voxik
Copy link
Contributor Author

@voxik voxik commented Nov 1, 2013

I am afraid you are right. It seems to be quite new feature:

http://lists.gnu.org/archive/html/coreutils/2012-02/msg00159.html
http://git.savannah.gnu.org/cgit/coreutils.git/commit/?id=15b8318e93a947d1422e9a927f699eda1085cf0d

I'll try to come with some universal solution, e.g. cd into the bin directory could be solution.

@mkristian
Copy link
Member

@mkristian mkristian commented Nov 2, 2013

ln -s jruby.bash /home/christian/projects/active/maven/jruby/bin/something

works with me, i.e. the first argument can be relative

@voxik
Copy link
Contributor Author

@voxik voxik commented Nov 5, 2013

I have updated the PR and added pushd/popd. Hope it is more universal.

mkristian added a commit that referenced this pull request Nov 11, 2013
Softlink jruby to jruby.bash instead of hard copy.
@mkristian mkristian merged commit e0099f7 into jruby:master Nov 11, 2013
1 check failed
1 check failed
@veganstraightedge
default The Travis CI build failed
Details
@mkristian
Copy link
Member

@mkristian mkristian commented Nov 11, 2013

let's see it that works everywhere . . .thanx

mkristian added a commit that referenced this pull request Nov 11, 2013
Softlink jruby to jruby.bash instead of hard copy.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants