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.

Sign up for GitHub

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

Jump to bottom

/opt/online used instead of $lool_dir #158

Closed
quenenni opened this issue Feb 16, 2018 · 0 comments
Closed

/opt/online used instead of $lool_dir #158

quenenni opened this issue Feb 16, 2018 · 0 comments

Comments

@quenenni
Copy link

Could you use $lool_dir instead of /opt/online in bin/onlineInstall.sh
That sort of error can be very bad for those who compile several lool on a server (like me :)).

This was introduced here:
5153c0f#diff-517170e58b930e4e6896c007c959eaa6

Thanks
@quenenni quenenni changed the title /opt/online used instead of $lol_dir /opt/online used instead of $lool_dir Feb 16, 2018
@Yumekui Yumekui mentioned this issue Sep 19, 2018
Merged
husisusi added a commit that referenced this issue Sep 19, 2018
@husisusi
Merge pull request #166 from Yumekui/patch-1
d768b87 
replaced hardcoded paths to $lool_dir (fix #158)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@quenenni