Opened 3 years ago
Closed 3 years ago
#54004 closed task (blessed) (fixed)
Bump Akismet (5.9 Edition)
Reported by: | kraftbj | Owned by: | dd32 |
---|---|---|---|
Milestone: | 5.9 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Build/Test Tools | Keywords: | |
Focuses: | Cc: |
Change History (15)
#2
@
3 years ago
trunk: [build51339]
branches/5.8: [build51340]
This ticket was mentioned in Slack in #core by desrosj. View the logs.
3 years ago
#6
@
3 years ago
Thanks for the ping @desrosj
Trunk: [build51597]
5.8: [build51598]
#7
follow-up:
↓ 11
@
3 years ago
- Resolution set to fixed
- Status changed from assigned to closed
Thanks to @chadworthman manually bumping akismet is no longer required.
The svn:external for the latest branch, and trunk, will be updated automatically with every commit.
For example, [build51608] would've bumped the version if a new version of Akismet was releaed.
I'm marking this as fixed, with a request that @kraftbj (or someone else) checks that https://build.trac.wordpress.org/browser/trunk/wp-content/plugins links to the latest version of Akismet after the next core commit following an akismet release.
#8
@
3 years ago
@dd32 and @chadworthman — This is great news and a welcome enhancement.
I'll confirm the next release is bumped on build. Generally, I pretty regularly use a site set to track trunk
and will notify when Akismet has an update. I'll be sure to verify and report if anything ever seems to be broken.
#9
@
3 years ago
- Resolution fixed deleted
- Status changed from closed to reopened
Looks like a new release dropped today (4.2.2). @dd32 would it be possible for you update this prior to 5.9 on Tuesday.
This ticket was mentioned in Slack in #core by desrosj. View the logs.
3 years ago
#11
in reply to:
↑ 7
;
follow-up:
↓ 13
@
3 years ago
Replying to dd32:
I'm marking this as fixed, with a request that @kraftbj (or someone else) checks that https://build.trac.wordpress.org/browser/trunk/wp-content/plugins links to the latest version of Akismet after the next core commit following an akismet release.
In addition to trunk, does it also need to be bumped in the 5.9 branch? Still points to 4.2.1 at the moment:
https://build.trac.wordpress.org/browser/branches/5.9/wp-content/plugins
The RC4 commit was ~20 minutes after the Akismet 4.2.2 release:
https://plugins.trac.wordpress.org/changeset/2663649/akismet
https://build.trac.wordpress.org/changeset/52223
#12
@
3 years ago
I'm sorry about this y'all—I didn't realize this was on deck to ship until it did.
#13
in reply to:
↑ 11
@
3 years ago
Replying to SergeyBiryukov:
In addition to trunk, does it also need to be bumped in the 5.9 branch? Still points to 4.2.1 at the moment:
https://build.trac.wordpress.org/browser/branches/5.9/wp-content/plugins
I think it needs to be bumped in the 5.9 branch to avoid immediate upgrade notifications on a fresh WP installation.
#14
@
3 years ago
Hmm... The automatic bumping worked kind-of!
Trunk was automatically updated in https://build.trac.wordpress.org/changeset/52225/trunk/wp-content/plugins
branches/5.8 will be automatically updated next-time there's a commit to it.
branches/5.9 didn't get automatically bumped, as the script bumps trunk & the branch of the version that https://api.wordpress.org/core/version-check/1.7/ returns.
We didn't think about the notion of core being branched before release and there being an Akismet update in that time.
I'll manually bump it for 5.9, and get the automatic bumping fixed up.
The current code is:
svn propset svn:externals "akismet $LATEST_AKISMET_VERSION/" $CORE_CO/trunk/wp-content/plugins svn propset svn:externals "akismet $LATEST_AKISMET_VERSION/" $CORE_CO/branches/$LATEST_CORE_BRANCH/wp-content/plugins
I think I'll have to create a new API for it, and return [ trunk, WP_CORE_DEV_BRANCH (5.9), WP_CORE_STABLE_BRANCH (5.8) ]
for this scenario, and [ trunk, WP_CORE_STABLE_BRANCH (5.9) ]
for normal.. or just make $LATEST_CORE_BRANCH reflect the dev-branch instead of stable-branch after release branching..
Bumped it for branches/5.9 in https://build.trac.wordpress.org/changeset/52227
#15
@
3 years ago
- Resolution set to fixed
- Status changed from reopened to closed
Marking as closed, it was bumped for 5.9, and I've followed up with systems to get the auto-bumping bumping in this edge-case: https://make.wordpress.org/systems/?p=1717#comment-1973
4.1.12 is out:
akismet http://plugins.svn.wordpress.org/akismet/tags/4.1.12/