Skip to content

lexbor: Refresh patch files#22517

Merged
TimWolla merged 4 commits into
php:PHP-8.4from
TimWolla:lexbor-update-patch-handling
Jun 29, 2026
Merged

lexbor: Refresh patch files#22517
TimWolla merged 4 commits into
php:PHP-8.4from
TimWolla:lexbor-update-patch-handling

Conversation

@TimWolla

Copy link
Copy Markdown
Member

No description provided.

@TimWolla TimWolla requested a review from kocsismate as a code owner June 29, 2026 22:12
@TimWolla TimWolla requested review from iluuu1994 and ndossche and removed request for iluuu1994 and kocsismate June 29, 2026 22:12

@iluuu1994 iluuu1994 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you! This should probably be backported to 8.4. The script lives in the old directory though, so this can be done later.

TimWolla added 3 commits June 30, 2026 00:36
The filename of the patch file is derived from the commit message. When the
commit message changes or the patch was initially added with the wrong name the
patch file might be renamed and exist twice when it's not deleted.
This includes the git version number by default which makes the patches
unstable.
Including the commit ID of the temporary commit in the patch files makes them
unstable.
@TimWolla TimWolla changed the base branch from PHP-8.5 to PHP-8.4 June 29, 2026 22:36
@TimWolla TimWolla merged commit d447025 into php:PHP-8.4 Jun 29, 2026
15 of 18 checks passed
TimWolla added a commit that referenced this pull request Jun 29, 2026
* PHP-8.4:
  lexbor: Refresh patch files (#22517)
TimWolla added a commit that referenced this pull request Jun 29, 2026
* PHP-8.5:
  lexbor: Run `update_lexbor.sh` (PHP 8.5)
  lexbor: Refresh patch files (#22517)
@TimWolla TimWolla deleted the lexbor-update-patch-handling branch June 29, 2026 22:53
DanielEScherzer pushed a commit that referenced this pull request Jul 1, 2026
* PHP-8.4:
  lexbor: Refresh patch files (#22517)

(cherry picked from commit bc42635)
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.

2 participants