boxen / our-boxen

Copy me for your team.
https://github.com/boxen/our-boxen/
MIT License
2.58k stars 884 forks source link

Send path fixes upstream #820

Closed johnsyweb closed 8 years ago

johnsyweb commented 8 years ago

Context

Getting the latest script/bootstrap into our fork broke. I fixed this locally when I upgaded to El Capitan but forgot to send it upstream.

Change

Reapply PATH fixes for El Capitan, etc, from envato#2210

Considerations

I considered combining PATH=${PATH}:/usr/sbin:/sbin atop the file but ultimately chose to put the PATH updates close to where they are used. This makes the context clearer (to me) but also will facilitate refactoring.

Cc

@jacobbednarz

jacobbednarz commented 8 years ago

Looks good!