-
Notifications
You must be signed in to change notification settings - Fork 6
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
php74 is deprecated #22
Comments
Hey @basilhendroff, it seems that some of the downstream forks have done the work and the changes could be integrated somewhat easily. Are you still maintaining this or should the truenas core community switch to one of the forks? |
I've used this fork and it works quite well. It's relatively straightforward to add to the script the ssmtp config and testing that's in the post-installation tasks. I've also added cleanup funcs based on this script to run the installation, clean up default plugins, remove default content etc. You can also perhaps secure the phpmyadmin installation too in the script, but I haven't got that far. Oh - one thing I found missing is setting up the default home and siteurl in the wordpress installation. Seems necessary if you are running wordpress in one jail that's communicating with the reverse proxy jail over http. These few lines have proved very useful to me to add to the
|
The script needs to be updated.
With these changes, it appears to build fine. Haven't yet tested the end result... but without the changes, I couldn't get any php dependency to install.
The text was updated successfully, but these errors were encountered: