mudpi
Fixed an Issue Regarding PHP Version Requirements In The Installer
It seems that one of the PHP packages didn't specify a version in the install script. I updated this to follow the same version of PHP that the other PHP dependencies are being installed with.
Also noticed that there wasn't any handling for Rasbian 11 in the uninstall script. I updated this as well since this is the version I am using.
Tested this by running an install using a local copy of the install script on a Desktop running Rasbian 11 using the force-yes option and the install succeeded without intervention. Also ran the uninstall script and observed the environment being detected and the defined PHP package being removed.
Ill review this and get merged in asap.
Eric Davisson Developer Mudpi https://mudpi.app God Bless
On Mon, Feb 17, 2025 at 1:32 PM Daniel434 @.***> wrote:
It seems that one of the PHP packages didn't specify a version in the install script. I updated this to follow the same version of PHP that the other PHP dependencies are being installed with.
Also noticed that there wasn't any handling for Rasbian 11 in the uninstall script. I updated this as well since this is the version I am using.
Tested this by running an install using a local copy of the install script on a Desktop running Rasbian 11 using the force-yes option and the install succeeded without intervention. Also ran the uninstall script and observed the environment being detected and the defined PHP package being removed.
You can view, comment on, or merge this pull request online at:
https://github.com/mudpi/installer/pull/15 Commit Summary
- 6b136e6 https://github.com/mudpi/installer/pull/15/commits/6b136e6abc0b03bbd431d18f41c1caab757178e6 Update PHP version logic in install and uninstall scripts
- aecdca6 https://github.com/mudpi/installer/pull/15/commits/aecdca63fa310f100e1d1f0919e32c155680fa1c Update PHP version logic in install and uninstall scripts
File Changes
(2 files https://github.com/mudpi/installer/pull/15/files)
- M install.sh https://github.com/mudpi/installer/pull/15/files#diff-043df5bdbf6639d7a77e1d44c5226fd7371e5259a1e4df3a0dd5d64c30dca44f (6)
- M uninstall.sh https://github.com/mudpi/installer/pull/15/files#diff-794e65e0ee6053d90c158cf942f93caee8a9fb1473830f1ccd660cf865c90828 (5)
Patch Links:
- https://github.com/mudpi/installer/pull/15.patch
- https://github.com/mudpi/installer/pull/15.diff
— Reply to this email directly, view it on GitHub https://github.com/mudpi/installer/pull/15, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADNTLKM7OZHJ66MRV4MCCXD2QI2LBAVCNFSM6AAAAABXKBBVO6VHI2DSMVQWIX3LMV43ASLTON2WKOZSHA2TQNRVGE4DSNA . You are receiving this because you are subscribed to this thread.Message ID: @.***>
