[RFC]: fsync function

RFC ref: https://wiki.php.net/rfc/fsync_function
PR ref: https://github.com/php/php-src/pull/6650

I think this one is pretty much good now and passing builds; when I first
proposed adding fsync some time ago, there was a suggestion from a couple
contributors that an RFC might be overkill/unnecessary for this feature
(and I'd certainly be happy for it to just be accepted as a pull request)
but want to make sure everyone has had a good chance to look over the RFC
and implementation and specifically whether anyone has any significant
concerns or objections?

Thanks.

Dave

Hi internals,
I have updated the GitHub PR and wiki RFC to also include an
implementation of fdatasync.

On another note, I note my PR builds are failing on MACOS_DEBUG_NTS with a
test in ext/zend_test/tests/observer_error_05.phpt- this does not appear
to be related to my changes and I'm wondering if anyone can shed any light
on this? I'm seeing it on some other open PRs too so I'm guessing it's a
known issue? Cheers.

David Gebler
davidgebler@gmail.com

On Sat, Jan 30, 2021 at 6:34 PM David Gebler davidgebler@gmail.com
wrote:

Updated the PR https://github.com/php/php-src/pull/6650 following
comment from Nikita, added basic tests, would appreciate further review
from contributors, have updated the RFC
https://wiki.php.net/rfc/fsync_function to under discussion, cheers.

David Gebler
davidgebler@gmail.com

On Sat, Jan 30, 2021 at 1:58 AM David Gebler davidgebler@gmail.com
wrote:

Hi internals,
Some time back I asked for commentary on a proposal to add fsync() to
PHP, I have now submitted a draft RFC
https://wiki.php.net/rfc/fsync_function alongside an in-progress proof
of concept PR (current status: can compile locally and run but the branch
needs more work to bring it up to standard, as well as tests and
documentation) which I would like to open up for review and discussion.

Thanks.

David Gebler
davidgebler@gmail.com