Skip to content

Conversation

@legoktm
Copy link
Member

@legoktm legoktm commented Dec 5, 2022

No description provided.

@legoktm legoktm changed the title Use PHP 8.0 as the base now that 7.4 is EOL Use PHP 8.0 & 8.1 as the base now that 7.4 is EOL Dec 5, 2022
@legoktm legoktm merged commit fb21219 into master Dec 5, 2022
@legoktm legoktm deleted the php8 branch December 5, 2022 03:31
legoktm added a commit to legoktm/official-images that referenced this pull request Dec 5, 2022
* 1.39.0 is now the new "stable" and "lts" using PHP 8.1
* 1.38 is now "legacy"
* 1.37 is EOL and removed
* The remaining images are now based on PHP 8.0

See:
* wikimedia/mediawiki-docker#116
* wikimedia/mediawiki-docker#117
legoktm added a commit to legoktm/official-images that referenced this pull request Dec 5, 2022
* 1.39.0 is now the new "stable" and "lts" using PHP 8.1
* 1.38 is now "legacy"
* 1.37 is EOL and removed
* The remaining images are now based on PHP 8.0

See:
* wikimedia/mediawiki-docker#116
* wikimedia/mediawiki-docker#117
@gesinn-it-gea
Copy link

@JeroenDeDauw
Copy link
Member

Changing major PHP versions can break things for users of the image. If such changes are made, then I can only recommend people not to use the mediawiki image for anything serious. If you pull in mediawiki:1.35 you do not expect to suddenly get a PHP version that breaks your REL1_35 extensions.

@gesinn-it-gea
Copy link

@JeroenDeDauw ... I had the same issue and created https://github.com/gesinn-it/docker-mediawiki-base

@legoktm
Copy link
Member Author

legoktm commented Jul 14, 2023

Sorry for not responding here, I had replied on Phabricator. In short, Docker told us to move off 7.4 (c.f. docker-library/official-images#13622 (comment)). And of course, there's other people complaining that we were using an EOL version of PHP, wanting the upgrade!

Forking is encouraged if you have stricter requirements than what this image can provide.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants