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

adding /opt/local/bin to search path. smartmachines need this otherwise ... #309

Merged
merged 1 commit into from
Jan 12, 2014
Merged

Conversation

someara
Copy link

@someara someara commented Dec 27, 2013

...curl cant find certificates

@damm
Copy link

damm commented Dec 27, 2013

Dumb question; why are we even setting the path? Why not source /etc/profile or whatever login mechanism handle it and let the shell handle it?

Just seems bad idea to enforce the PATH. If you want to enforce it do it like.

-          PATH=/usr/local/sbin:/usr/local/bin:/sbin:/bin:/usr/sbin:/usr/bin
+          PATH=$PATH:/usr/local/bin

This is of course assuming smartmachines set set /opt/local/bin in the PATH?

fnichol added a commit that referenced this pull request Jan 12, 2014
adding /opt/local/bin to search path. smartmachines need this otherwise ...
@fnichol fnichol merged commit c094822 into test-kitchen:master Jan 12, 2014
@test-kitchen test-kitchen locked and limited conversation to collaborators Nov 16, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants