upgrade perl to v5.40.0 #35
Conversation
review-deriv
left a comment
review-deriv
left a comment
There was a problem hiding this comment.
PR description is too short. You can use this format:
# Description
This is a summary of the changes. This should describe why do we need this change.
Whys and Hows come here.
# Cards
* https://clickup.com/12345
* https://clickup.com/12346
|
Why changing the base image for myriad? The package should have been installed automatically during the build process |
review-deriv
left a comment
review-deriv
left a comment
There was a problem hiding this comment.
PR description is too short. You can use this format:
# Description
This is a summary of the changes. This should describe why do we need this change.
Whys and Hows come here.
# Cards
* https://clickup.com/12345
* https://clickup.com/12346
diego-deriv
left a comment
diego-deriv
left a comment
There was a problem hiding this comment.
should this build both branches?
|
Yes and no, we need at least the changes for perl_40 but I simply update the one for master so it is easily to reconcile the changes later |
| ENV DEBIAN_FRONTEND=noninteractive | ||
| ENV PERL_VERSION=5.36.0 | ||
| ENV PERL_SHA256=0f386dccbee8e26286404b2cca144e1005be65477979beb9b1ba272d4819bcf0 | ||
| ENV PERL_VERSION=5.40.0 |
There was a problem hiding this comment.
how sure are we regarding this working on the usual images depending on this, like dzil & myriad
There was a problem hiding this comment.
This is tested at least on our stuffs, on the latest myriad upgrade attempt
Co-authored-by: Diego Ramírez <[email protected]>
6 participants
No description provided.