search

krakjoe / uopz

User Operations for Zend
Other
356 stars 47 forks source link

Add a ini option to control the exit override default #106

Closed duncan3dc closed 5 years ago

duncan3dc commented 5 years ago

I would like to enable the extension but still use the exit() function. To achieve this I have to add calls to uopz_allow_exit(true) everywhere, it would be preferable to control this at the same time as enabling the extension, eg:

extension=uopz
uopz.allow_exit=1
cmb69 commented 5 years ago

What about changing the default instead, as has been suggested by @rlerdorf?

krakjoe commented 5 years ago

I'm reluctant to change the defaults, again ... but have introduced ini uopz.exit, and made exit behaviour consistent with and without opcache at the same time ... predictable++

Thanks for the PR ...

krakjoe commented 5 years ago

@cmb69 could you please pop a manual entry in for this, and I'll ask @remicollet to do a bug fix release in the next couple of days, once we've had a chance to do a few big test runs ...

cmb69 commented 5 years ago

Yes, of course. Will the next release be 6.0.1?

krakjoe commented 5 years ago

It will.

cmb69 commented 5 years ago

Done.

krakjoe commented 5 years ago

Ta

mustanggb commented 5 years ago

Just got bitten by this one as well, any chance of adding a warning or something to README.md?