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

STORM-203 Adding paths to default java library path #1735

Merged
merged 3 commits into from Nov 3, 2016
Merged

Conversation

@picadoh
Copy link
Contributor

picadoh commented Oct 13, 2016

Adding /usr/lib64 and /usr/lib/x86_64-linux-gnu to default java library path

@knusbaum

This comment has been minimized.

Copy link
Contributor

knusbaum commented Oct 17, 2016

/usr/lib64 is FHS standard. /usr/lib/x86_64-linux-gnu is Debian-specific. I think we should avoid setting distro-specific defaults.

…y path as it is Debian specific
@picadoh

This comment has been minimized.

Copy link
Contributor Author

picadoh commented Oct 17, 2016

yeah, makes total sense 👍 removed it from the path. thanks @knusbaum

@revans2

This comment has been minimized.

Copy link
Contributor

revans2 commented Nov 3, 2016

@knusbaum are you OK with the change?

I am +1

@knusbaum

This comment has been minimized.

Copy link
Contributor

knusbaum commented Nov 3, 2016

+1

@asfgit asfgit merged commit a3799d4 into apache:master Nov 3, 2016
1 check failed
1 check failed
continuous-integration/travis-ci/pr The Travis CI build failed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.