I had another fix I wanted, but first thought I'd fix the build #23

Merged
merged 2 commits into from Apr 6, 2013

Projects

None yet

3 participants

@austinfromboston

No description provided.

@austinfromboston

Something changed, maybe in rvm or chef, over the past 2 months, and now a soloist script that used to work is failing, claiming it can't find 'chef-solo'. It's being run via cap in an rvm-shell instance, which all looks right. Switching the sudo calls to rvmsudo fixes, but is pretty darn ugly. Got ideas on a beter fix?

@mkocher mkocher merged commit bb63504 into mkocher:master Apr 6, 2013

1 check passed

Details default The Travis build passed
@mkocher
Owner
mkocher commented on bb63504 Apr 6, 2013

Huh, this commit is breaking the default_editor recipe which uses gem_package to install the plist gem. Frankly having chef install the gem has always been a little broken. I'll open an issue on pivotal_workstation to discuss.

@hiremaga
Collaborator
hiremaga commented Apr 6, 2013

@austinfromboston FWIW, assuming RVM isn't reasonable unfortunately. A number of teams using PW now use Rbenv instead.

@cunnie & I are working on some reorganization of PW that may end up offering a convenient solution for the plist gem dependency, at least in the case of the default_editor recipe. More on this soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment