Opened 9 years ago
Closed 7 years ago
#505 closed defect (bug) (fixed)
Plugins Zip Builder Tags v. Branches namespacing conflict
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | Priority: | normal | |
Component: | Plugin Directory | Keywords: | dev-feedback |
Cc: |
Description
If a software project like bbPress runs tags and branches named the same -- for example, having a structure that looks something like
/branches/2.5/ /tags/2.5/ /tags/2.5.1/ /tags/2.5.2/ /tags/2.5.3/ /tags/2.5.4/
and keeps the 2.5 branch up to date, so that it is reflecting the current 2.5.4 release, the download of the 2.5 release will actually be the 2.5.4 release.
Test it for yourself: http://wordpress.org/plugins/bbpress/developers/
This is because of a naming conflict when branches and tags have their zip packages made.
Tags are generated as:
http://downloads.wordpress.org/plugin/plugin.{$tag_directory}.zip
Branches are generated as:
http://downloads.wordpress.org/plugin/plugin.{$branch_directory}.zip
If they have the same name, then I believe the zips are regenerated as the files in that directory is modified? Regardless, the zip of the 2.5 tagged release gets overwritten with the 2.5 branch.
This is particularly problematic if current stable is 2.5, and a 2.5.1 point release is being prepared in the 2.5 branch. Code that may not be yet meant for release into the world is getting delivered to every user that installs or upgrades to current stable.
This is bad.
Ideal and I suspect easiest resolution would be to change the branch zip filenames to
http://downloads.wordpress.org/plugin/plugin.branch.{$branch_directory}.zip
Change History (12)
#3
@
9 years ago
Yes, branches do get turned into ZIP files. I discovered this the hard way when Jetpack had an issue with this.
We could just turn off the ZIPs for branches. I'd be okay with that.
#4
@
9 years ago
Especially as feature branches are normally in active development, with a lot of commits going in, it seems silly to rebuild the zip after each commit.
+1 to just not building zips for branches.
This ticket was mentioned in IRC in #wordpress-meta by sams. View the logs.
9 years ago
#8
@
7 years ago
- Keywords needs-patch removed
Removing 'needs-patch' from components that are not open sourced yet, to help focus on tickets that actually need a patch.
I am/was not aware that branches generated zips.