Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wrong version number in released builds #869

Closed
boredom2 opened this issue Jun 11, 2017 · 5 comments
Closed

Wrong version number in released builds #869

boredom2 opened this issue Jun 11, 2017 · 5 comments
Assignees
Labels
status: archived Archived and locked; will not be updated type: bug Something isn't working correctly
Milestone

Comments

@boredom2
Copy link

Hi there,

the compiled Version on CDNJS here:
https://cdnjs.cloudflare.com/ajax/libs/shaka-player/2.1.3/shaka-player.compiled.js seems to be wrong - it shows v2.1.2-17...
Search the Code for "v2." and you will see, that this Version offers v2.1.2-17-gf11a54a-dirty.
This seems not to be a clean final Version of the 2.1.3 Code.

I already opened a Ticket on CDNJS, but they said, its something your Side must fix.
Can you please check that?

@joeyparrish
Copy link
Member

Yes, that is definitely a mistake we made in our release process. The build was done before the tag was created, which caused the released build to have the wrong version number.

I went back through previous releases and found that v2.0.3 and v2.0.4 also had a bad version number in the binary.

I will see what I can do to fix the process for future releases, and I will try to find a way to fix the three builds that are already out with the wrong version numbers: v2.1.3, v2.0.4, v2.0.3.

@joeyparrish joeyparrish added the type: bug Something isn't working correctly label Jun 11, 2017
@joeyparrish joeyparrish added this to the v2.2.0 milestone Jun 11, 2017
@joeyparrish joeyparrish changed the title Version 2.1.3 and 2.1.2-17-dirty Wrong version number in released builds Jun 11, 2017
@joeyparrish joeyparrish self-assigned this Jun 11, 2017
@joeyparrish
Copy link
Member

I will not be able to fix the version in the builds that went out via npm. According to the docs on npm publish:

Once a package is published with a given name and version, that specific name and version combination can never be used again, even if it is removed with npm-unpublish.

I may be able to fix what's in CDNJS and Google Hosted Libraries.

shaka-bot pushed a commit that referenced this issue Jun 12, 2017
This fixes an issue in which we released several builds (v2.1.3,
v2.0.4, and v2.0.3) with the wrong version number.  This happens when
we build, then tag, then release.

By forcing a rebuild at publication time, we make sure the build we
release has the right tag in it.

Issue #869

Change-Id: I23867ceef43162469c53006336b7f0226dbe16b6
@joeyparrish
Copy link
Member

The versions in Google Hosted Libraries are already correct. (Those go through a different release process that is not driven by npm.)

So it is only npm (which we can't fix) and CDNJS (which I am working to fix).

We have fixed our npm release scripts to avoid this issue in future.

@boredom2
Copy link
Author

Thanks for your Efforts!

@joeyparrish
Copy link
Member

According to @PeterDaveHello in cdnjs/cdnjs#11396 (comment), we can't update what's in CDNJS. So there's nothing more we can do.

I apologize for the error in those releases. We have fixed our processes so it should not happen again. Builds on Google Hosted Libraries are not affected, but builds on NPM and CDNJS are.

joeyparrish added a commit that referenced this issue Jun 15, 2017
This fixes an issue in which we released several builds (v2.1.3,
v2.0.4, and v2.0.3) with the wrong version number.  This happens when
we build, then tag, then release.

By forcing a rebuild at publication time, we make sure the build we
release has the right tag in it.

Issue #869

Change-Id: I23867ceef43162469c53006336b7f0226dbe16b6
@shaka-project shaka-project locked and limited conversation to collaborators Mar 22, 2018
@shaka-bot shaka-bot added the status: archived Archived and locked; will not be updated label Apr 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status: archived Archived and locked; will not be updated type: bug Something isn't working correctly
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants