#61665 closed task (blessed) (fixed)
Update the `$_old_files` array for 6.6
| Reported by: | dd32 | Owned by: | hellofromTonya |
|---|---|---|---|
| Priority: | normal | Milestone: | 6.6.1 |
| Component: | Upgrade/Install | Version: | 6.6 |
| Severity: | normal | Keywords: | has-patch dev-reviewed commit fixed-major |
| Cc: | Focuses: |
Description
It doesn't appear that the $_old_files array was updated prior to the 6.6 release.
The following files need to be added:
$ svn diff --summarize https://core.svn.wordpress.org/tags/6.5 https://core.svn.wordpress.org/tags/6.6 | grep '^D' D https://core.svn.wordpress.org/tags/6.5/wp-includes/blocks/block/editor.css D https://core.svn.wordpress.org/tags/6.5/wp-includes/blocks/block/editor.min.css D https://core.svn.wordpress.org/tags/6.5/wp-includes/blocks/block/editor-rtl.css D https://core.svn.wordpress.org/tags/6.5/wp-includes/blocks/block/editor-rtl.min.css
See https://make.wordpress.org/core/handbook/about/release-cycle/releasing-major-versions/#dry-run
Change History (21)
#2
@
2 years ago
- Keywords needs-patch added
- Owner set to
- Status new → assigned
Erf. When preparing the Dry Run we saw those files with @davidbaumwald and @hellofromTonya but assumed they were false positive. Sorry everyone about that.
https://wordpress.slack.com/archives/C06U06K50Q5/p1720968264541949
#3
@
2 years ago
Are these actually old files? Or are they artifacts causing false flags?
Copying parts of the slack discussion about these particular files:
@davidbaumwald observed these files keep getting reported for "a few majors":
^Dproduce a few false positives that have existed for a while now, and^Areports things that do not exist in the array previously.
...
They've been there for a few majors now
and was working with systems to help identify why:
I reached out to systems to see if there's some sort of artifact in core.svn.wordpress.org
Hey @davidbaumwald, any feedback from the systems team?
#4
@
2 years ago
Are these actually old files? Or are they artifacts causing false flags?
I compared 6.4.5, 6.5.5, and 6.6.0 to find out.
The files:
'wp-includes/blocks/block/editor.css', 'wp-includes/blocks/block/editor.min.css', 'wp-includes/blocks/block/editor-rtl.css', 'wp-includes/blocks/block/editor-rtl.min.css',
How:
- Downloaded each version from the release archives.
- Unzipped each, renaming the directory to append its version.
- Navigated to
wp-includes/blocks/block/directory in each version to check if the files were there or not.
Results:
- 6.4.5 ✅ yes these files are present.
- 6.5.5 ✅ yes these files are present.
- 6.6.0 ❌ no these files are not present.
Interesting. So they were removed in the 6.6 cycle.
#5
@
2 years ago
I understand now. These are built files, and only exist after a build, which core.svn.wordpress.org seems to be referencing the build repo(currently at rev 58145).
The $_old_files should include them. This is on me for conflating two different issues in the past.
Going to get this committed to trunk now.
This ticket was mentioned in PR #7049 on WordPress/wordpress-develop by @hellofromTonya.
2 years ago
#6
- Keywords has-patch added; needs-patch removed
Adds the following files to the $_old_files array for 6.6:
$ svn diff --summarize https://core.svn.wordpress.org/tags/6.5 https://core.svn.wordpress.org/tags/6.6 | grep '^D' D https://core.svn.wordpress.org/tags/6.5/wp-includes/blocks/block/editor.css D https://core.svn.wordpress.org/tags/6.5/wp-includes/blocks/block/editor.min.css D https://core.svn.wordpress.org/tags/6.5/wp-includes/blocks/block/editor-rtl.css D https://core.svn.wordpress.org/tags/6.5/wp-includes/blocks/block/editor-rtl.min.css
Originally flagged as false positives, these specific files did exist in 6.4.5 and 6.5.5, but no longer exist in 6.6.
Trac ticket: https://core.trac.wordpress.org/ticket/61665
This ticket was mentioned in PR #7050 on WordPress/wordpress-develop by @davidbaumwald.
2 years ago
#7
Missed during the Dry Run(by me).
@hellofromTonya commented on PR #7049:
2 years ago
#9
Closing in favor of patch with same changes that was committed in https://core.trac.wordpress.org/changeset/58744.
#10
@
2 years ago
- Keywords dev-feedback added
- Resolution fixed
- Status closed → reopened
Reopening for backport to the 6.6 branch.
#11
@
2 years ago
- Keywords dev-reviewed commit added; dev-feedback removed
[58744] LGTM for backport to the 6.6-branch.
This ticket was mentioned in Slack in #core by hellofromtonya. View the logs.
2 years ago
#16
@
2 years ago
Manually added @jpnl to the commit props, i.e. who reported the same issue in #61678.
This ticket was mentioned in Slack in #core by costdev. View the logs.
2 years ago
#18
@
2 years ago
Hey Guys!
Thanks for jumping on this so quickly.
Are you going to release a WP patch (say, V 6.6.1) to close this finding?
Cheers!
#19
@
2 years ago
@generosus RC1 for 6.6.1 was made available yesterday and assuming there are no issues, it is planned to be released on 23 July
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Just noting, that has caused a number of "security plugins" to throw warnings about the WordPress installation having extra unexpected files.