-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
[Backport 7.x] (PUP-12031) Consolidate gem depenendencies & metadata into puppet.gemspec #9337
Conversation
2ab7a2a
to
df738ba
Compare
There are some discrepancies between the runtime dependencies versions in .gemspec:
project_data.yaml:
Maybe the .gemspec wasn't updated as ext/project_data.yaml was updated or vice versa?. Either way, I followed whatever runtime dependency versions were specified in |
Well this is not expected 🤔
|
Ah actually, I think the
|
df738ba
to
1273191
Compare
95cc3aa
to
217ef72
Compare
…spec This commit moves the puppet gem metadata from ext/project_yaml.data and .gemspec into puppet.gemspec. This change allows the puppet gem to be built using `gem build puppet.gemspec`. Additionally, for platform specific gems (e.g. puppet-universal-darwin.gem and puppet-x64-mingw32.gem), there is conditional logic in puppet.gemspec so platform specific dependecies such as ffi are appended when building those platform-specific gems. (cherry picked from commit 867ce9c)
This commit adds a Rake task, gem:build, that will build all puppet gems which includes the three platform specific puppet gems: x86-mingw32, x64-mingw32, and universal-darwin (cherry picked from commit e58bc11)
217ef72
to
57ea40c
Compare
Successfully built gem using 'gem build puppet.gemspec':
|
Diffs to verify that the gems are being built correctly:
|
Rake task works too!
|
Backport of #9325 to 7.x