Page MenuHomeFreeBSD

php.mk: Create php extension ini files as sample
ClosedPublic

Authored by otis on Aug 29 2023, 11:05 AM.
Tags
None
Referenced Files
Unknown Object (File)
Apr 12 2024, 1:47 AM
Unknown Object (File)
Apr 9 2024, 4:14 PM
Unknown Object (File)
Apr 3 2024, 12:05 AM
Unknown Object (File)
Feb 7 2024, 7:46 AM
Unknown Object (File)
Dec 23 2023, 1:15 AM
Unknown Object (File)
Nov 18 2023, 10:17 AM
Unknown Object (File)
Nov 18 2023, 10:17 AM
Unknown Object (File)
Nov 18 2023, 5:02 AM
Subscribers

Details

Summary

Create php extension ini files with .sample extension
to prevent subsequent upgrades from clobbering the, possibly,
user-modified .ini files.

Diff Detail

Repository
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 53329
Build 50220: arc lint + arc unit

Event Timeline

otis requested review of this revision.Aug 29 2023, 11:05 AM

In theory those files were not designed to be modified by the user. What's the use case that requires it and that cannot be done via another php configuration file?

In D41630#949078, @ale wrote:

In theory those files were not designed to be modified by the user. What's the use case that requires it and that cannot be done via another php configuration file?

The actual use case, for me, is that I have an unified set of extensions of which one (opcache) is causing problems in one particular installation. That's why I want to be able to comment it out without subsequent update to activate it again.

Moreover, it's a handy place to put extension's configuration into.

I haven't tested the patch but it looks fine. What do you think @bofh ?

In D41630#949988, @ale wrote:

I haven't tested the patch but it looks fine. What do you think @bofh ?

As you approve it I will run a smaller exp-run on this and revert back after I finish my exp-run on newer version of php81 and php82. :)

I can't see any potential drawback in this change, so if it works as expected I approve it. Waiting for your exp-run feeback.

This revision is now accepted and ready to land.Sep 4 2023, 5:51 AM