Plugins Zip Builder Tags v. Branches namespacing conflict

[georgestephanis]

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

[nacin]

I am/was not aware that branches generated zips.

[mdawaffe]

I didn't even know we made ZIPs for branches :)

Should we just not?

[Otto42]

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.

[georgestephanis]

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.

[SergeyBiryukov]

Removing 'needs-patch' from components that are not open sourced yet, to help focus on tickets that actually need a patch.

[ocean90]

@Otto42 Any news here?

[dd32]

In 3081:

Plugin Directory: Add an initial set of scripts to build & serve ZIP files.

See #505 - The new directory doesn't build ZIPs for branches.
See #1578

[dd32]

Given this has existed for 2 years, i'm going to mark this as fixed in the new directory.