Skip to content
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

Update min required PHP version to 7.3 #2648

Closed
ssddanbrown opened this issue Mar 19, 2021 · 3 comments
Closed

Update min required PHP version to 7.3 #2648

ssddanbrown opened this issue Mar 19, 2021 · 3 comments

Comments

@ssddanbrown
Copy link
Member

ssddanbrown commented Mar 19, 2021

Very much related to #2388 (Upgrade to support PHP8), Specifically my comments here.

I was initially looking to jump direct to 7.4 but that would mean only php7.4 would be support on previous and next releases, Plus php7.3 is still on security support, so we'll stick with it to provide a more gradual change. This would just be dropping php7.2. We'll drop 7.3 next year unless there's a specific reason to drop it sooner.

Will update this with more specific steps soon, but we should probably recommend jumping direct to PHP8 for future compatibility and longevity. We'll need to provide scripts for people on relevant LTS systems we support and ensure popular community maintained hosting/docker projects are ready.

Systems PHP Versions

  • Ubuntu 16.04 (Almost End of life)
    • php7.0 native, Scripts been installing php7.4 and we provided upgrade instructions for that on previous version change.
  • Ubuntu 18.04
    • php7.2 native, script still installing php7.2. Will need to support.
  • Ubuntu 20.04
    • php7.4 native, fine to leave as-is.
  • CentOS 7 (Don't link to this in docs but we have it in the scripts repo)
    • php5.4 native, Script uses remi-php73, Should be fine to leave as-is for now.
  • Linuxserver.io (Docker)
    • Checking inside container reports as PHP 7.3.27. Looks like they're prepared to move that up anyway.
  • Solidnerd (Docker)
    • Based on `php:7.4-apache-buster, No update needed.
  • Home Assistant Addon
    • Based on PHP 7.4.15, Not update needed.
@ssddanbrown
Copy link
Member Author

Main app work now done in 829fecd, 0310b86, 8e5067e & c548c06.

This was referenced Mar 20, 2021
@ssddanbrown
Copy link
Member Author

The following works to update a fresh script-installed Ubuntu 18.04 instance to php8:

sudo add-apt-repository ppa:ondrej/php
sudo apt update
sudo apt install -y php8.0 php8.0-fpm php8.0-curl php8.0-mbstring php8.0-ldap php8.0-xml php8.0-zip php8.0-gd php8.0-mysql libapache2-mod-php8.0
sudo a2dismod php7.2
sudo a2enmod php8.0
systemctl restart apache2

That's to provide old script users on update. Have tested in a VM. For the exiting script, we'll move that up to php7.4 now to allow some cross-over.

ssddanbrown added a commit to BookStackApp/devops that referenced this issue Mar 21, 2021
@ssddanbrown
Copy link
Member Author

Ubuntu 18.04 install script now updated in BookStackApp/devops@01c971c

All that's left to do is release documentation using the above so will close this off.

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

No branches or pull requests

1 participant