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

(PE-14463) Support 32-bit Windows via pe_repo #94

Merged
merged 3 commits into from
Mar 25, 2016

Conversation

MikaelSmith
Copy link
Contributor

When installing packages in a PE environment via pe_repo, only 64-bit
packages were supported on Windows. Update to also support 32-bit
packages.

@MikaelSmith
Copy link
Contributor Author

Tested in a PE environment against Windows 10 Enterprise (32-bit).

@MikaelSmith
Copy link
Contributor Author

@highb updated; now it has some overlap with #86.

When installing packages in a PE environment via pe_repo, only 64-bit
packages were supported on Windows. Update to also support 32-bit
packages.
@highb
Copy link
Contributor

highb commented Mar 24, 2016

Going to re-test this against the Puppet 4+ upgrade changes that you just merged. :)

@MikaelSmith
Copy link
Contributor Author

I'm rebasing and fixing a few things.

@highb
Copy link
Contributor

highb commented Mar 24, 2016

Ah ok I'll wait up for that.

@MikaelSmith
Copy link
Contributor Author

Updated.

@highb
Copy link
Contributor

highb commented Mar 24, 2016

@MikaelSmith The specs are still failing.

Michael Smith added 2 commits March 24, 2016 15:01
Adds spec tests for Windows-specific preparation, which primarily
involves retrieving the MSI when in a PE environment.
Use the variable to ensure tests for same version use the same source.
@MikaelSmith
Copy link
Contributor Author

Ok, I think it's fixed now.

@highb
Copy link
Contributor

highb commented Mar 25, 2016

👍

@highb highb merged commit 88cb9e6 into puppetlabs:master Mar 25, 2016
@MikaelSmith MikaelSmith deleted the PE-14463 branch April 5, 2016 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants