have illumos-gate build sun-solaris with perl 534 #7733
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This changes
components/openindiana/illumos-gate
so that instead of generatingruntime/perl-522/module/sun-solaris
, it generatesruntime/perl-534/module/sun-solaris
.There are a lot of relevant comments from Alexander in PR #7640 that relate to this update. The diff in 7640 should not be merged. I'll close that PR.
Note that I've rebuilt
illumos-gate
many times in the past couple weeks. I seem to get unpredictable behavior withillumos-gate
. It has built and successfully "installed' (the entiregmake build
process completes successfully) a couple of times, but most of the time everything builds successfully, but I get failures from the "install":I don't know if these relate to the chicken-or-egg problem that Alexander mentioned, but they seem unrelated to the perl changes I made.
Note that in addition to building
runtime/perl-534/module/sun-solaris
, I've also tested changing theperl
andsystem-perl
mediator to point to the 534 perl, and all ofillumos-gate
builds OK when usingperl-534
.As I said in my question on
oi-dev
, I don't know where this should fit into the overall process of updating perl, but I wanted to get the PR in place so that other eyes can check it out and hopefully have some suggestions about where this should fit in the overall process.