Skip to content
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

GH-14930: Revert "Reserve less file space if possible in a directory entry" #14933

Closed
wants to merge 2 commits into from

Conversation

joec4i
Copy link
Contributor

@joec4i joec4i commented Jul 12, 2024

This reverts 00c1e7b from #11577 which fixes #14930 .

@cmb69
Copy link
Member

cmb69 commented Jul 12, 2024

The bug ticket claims that this affects PHP 8.3, so this PR should probably target the PHP-8.3 branch (not the master branch).

@joec4i
Copy link
Contributor Author

joec4i commented Jul 12, 2024

The bug ticket claims that this affects PHP 8.3, so this PR should probably target the PHP-8.3 branch (not the master branch).

Hi @cmb69 The bug was introduced in PHP 8.3, but it affects the master branch as well.

@nielsdos
Copy link
Member

The bug ticket claims that this affects PHP 8.3, so this PR should probably target the PHP-8.3 branch (not the master branch).

Hi @cmb69 The bug was introduced in PHP 8.3, but it affects the master branch as well.

Bug fixes should always go against the lowest supported branch the bug is in. So this should indeed target PHP-8.3.

@joec4i
Copy link
Contributor Author

joec4i commented Jul 12, 2024

TIL. Let me update the PR. Cheers!

@joec4i joec4i force-pushed the revert-dir-name-len-change branch from 78d62b2 to 987ee94 Compare July 12, 2024 12:28
@joec4i joec4i changed the base branch from master to PHP-8.3 July 12, 2024 12:30
Copy link
Member

@nielsdos nielsdos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM thanks.
I guess in theory we could add a SKIPIF section that checks if MAXPATHLEN is at least 288, but I don't think there's a reasonable platform that has such a low limit.

@nielsdos nielsdos closed this in 38501ed Jul 12, 2024
@nielsdos
Copy link
Member

Merged into 8.3 and up, thanks!

@joec4i joec4i deleted the revert-dir-name-len-change branch July 14, 2024 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Custom stream wrapper dir_readdir output truncated to 255 characters in PHP 8.3
3 participants