Skip to content

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

Merged
fnichol merged 1 commit into
test-kitchen:masterfrom
someara:smartos-curl
Jan 12, 2014
Merged

adding /opt/local/bin to search path. smartmachines need this otherwise ...#309
fnichol merged 1 commit into
test-kitchen:masterfrom
someara:smartos-curl

Conversation

@someara

@someara someara commented Dec 27, 2013

Copy link
Copy Markdown

...curl cant find certificates

@damm

damm commented Dec 27, 2013

Copy link
Copy Markdown

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