Commit b8df4f5
committed
(PA-5849) Consistently use miniruby when native compiling ruby 3.2
Ruby's build process requires a "base" ruby. We also need an executable ruby
that can install gems into /opt/puppetlabs/puppet/lib.
Previously, native compilation used whatever system ruby happened to be present.
Since each distro ships a different ruby, our build process used everything from
ruby 2.7 to 3.1 and everything in between.
This inconsistency made it more likely for our rubygem components with native
extensions to fail to install.
It also meant bumping to ruby 3.3 in the future could fail because that version
requires base ruby 3.0 or later, but some system rubies are still 2.7.
Additionally, in the past we've made the mistake of using pl-ruby because we
thought we had to.
This commit ensures we use miniruby for all native compiles. Doing so eliminates
all of the issues mentioned.
When cross-compiling, we still have to use either system ruby or pl-ruby.1 parent 78d1786 commit b8df4f5
1 file changed
+21
-11
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
91 | 91 | | |
92 | 92 | | |
93 | 93 | | |
| 94 | + | |
| 95 | + | |
| 96 | + | |
| 97 | + | |
| 98 | + | |
| 99 | + | |
| 100 | + | |
| 101 | + | |
| 102 | + | |
| 103 | + | |
| 104 | + | |
| 105 | + | |
| 106 | + | |
| 107 | + | |
| 108 | + | |
| 109 | + | |
| 110 | + | |
94 | 111 | | |
95 | 112 | | |
96 | 113 | | |
97 | 114 | | |
98 | | - | |
99 | | - | |
100 | 115 | | |
101 | 116 | | |
102 | 117 | | |
103 | 118 | | |
104 | | - | |
| 119 | + | |
105 | 120 | | |
106 | | - | |
107 | | - | |
108 | | - | |
| 121 | + | |
109 | 122 | | |
110 | 123 | | |
111 | | - | |
| 124 | + | |
112 | 125 | | |
113 | 126 | | |
114 | | - | |
115 | | - | |
116 | | - | |
117 | | - | |
| 127 | + | |
118 | 128 | | |
119 | 129 | | |
120 | 130 | | |
| |||
0 commit comments