search

voxpupuli / vagrant-librarian-puppet

A Vagrant plugin to install Puppet modules using Librarian-Puppet.
MIT License
99 stars 26 forks source link

Upgraded puppet version to 3.7.4 to address windows 32/64 bit misdirection #32

Closed solcates closed 8 years ago

solcates commented 9 years ago

Simply upgrading to 3.7.4 avoids the issue, as Puppet put a lot of work into revamping factor 2.0.0 on windows (which is the later version of Puppet).

asifrc commented 9 years ago

+1

mlt commented 9 years ago

:+1: Current puppet pulls buggy facter (FACT-352 and alike). I didn't check particular version, but I just used to replace ~> with > for puppet gem and let it install the latest.

jhoblitt commented 8 years ago

@solcates Why does the puppet version spec need to be changed? Isn't a dep on facter 2.2 what's needed?

flamaral256 commented 8 years ago

I think this pull request may fix my problem (https://github.com/mhahn/vagrant-librarian-puppet/issues/44) on Windows 10 64 bits.

jhoblitt commented 8 years ago

I'm going to close this PR since it stale at that point. I would accept a PR to pin the facter version, if that's necessary for windows users.

mlt commented 8 years ago

@jhoblitt I think it is more about broken puppet version (that pulls broken facter) that needs to be bumped up. So you are right, there is no need to mess with puppet spec since newer puppet would pull newer facter.

As of now, it looks like it is enough to have only the first commit from this PR. It will pull puppet 3.7.5 though.

mlt commented 8 years ago

I retract my previous comment. I finally re-installed vagrant and plugins as I left a feedback in #12. Perhaps some gremlins were messing with old dependencies sitting around. Everything works for me now without any modifications on Windows 8.1.

Apparently things are getting cached in other locations as well. I deleted all dot smthng folders in my project as well as ~/.puppet/ and ~/.puppetlabs/ (I don't remember for sure). And I got problems again. Bumping puppet dependency to 3.7.5 did not help, but to 4.3.1 as in 4e78b9503c6591d2cb6382785b88c4fbcc760cce seems made a trick. I rebuilt & installed this plugin and all seems good.

jhoblitt commented 8 years ago

51 has been merged.