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

The nightly upload script should also copy the newest binary to the name "servo-latest" #11987

Closed
larsbergstrom opened this issue Jul 1, 2016 · 6 comments

Comments

@larsbergstrom
Copy link
Contributor

@larsbergstrom larsbergstrom commented Jul 1, 2016

After uploading a nightly build with a specific date in its filename, we should also create a file named "servo-latest" (with the appropriate extension, e.g. .tar.gz) for easy linking. This change should be made here:
https://github.com/servo/servo/blob/master/etc/ci/upload_nightly.sh#L15

@highfive
Copy link

@highfive highfive commented Jul 1, 2016

@highfive
Copy link

@highfive highfive commented Jul 1, 2016

@aneeshusa aneeshusa removed the L-python label Jul 1, 2016
@aneeshusa aneeshusa changed the title The nightly upload script should also copy the newest binary to the name "latest" The nightly upload script should also copy the newest binary to the name "servo-latest" Jul 1, 2016
@highfive
Copy link

@highfive highfive commented Jul 1, 2016

Please make a comment here if you intend to work on this issue. Thank you!

@aneeshusa aneeshusa added the E-easy label Jul 1, 2016
@imjacobclark
Copy link
Contributor

@imjacobclark imjacobclark commented Jul 1, 2016

I'll pick this issue up.

@wafflespeanut
Copy link
Member

@wafflespeanut wafflespeanut commented Jul 1, 2016

Good to know! :)

@imjacobclark imjacobclark mentioned this issue Jul 1, 2016
3 of 5 tasks complete
@imjacobclark
Copy link
Contributor

@imjacobclark imjacobclark commented Jul 1, 2016

Initial pull request for this raised: #12004.

Let me know of any feedback on approach, concerns or changes that I need to make.

bors-servo added a commit that referenced this issue Jul 1, 2016
Add servo-latest on upload-nightly

Allows for easy linking over to servo-nightly builds.

cc/ @larsbergstrom

---
<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: -->
- [X] `./mach build -d` does not report any errors
- [X] `./mach test-tidy` does not report any errors
- [ ] These changes fix #11987 (github issue number if applicable).

<!-- Either: -->
- [ ] There are tests for these changes OR
- [X] These changes do not require tests because they are part of the CI infrastructure

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/12004)
<!-- Reviewable:end -->
bors-servo added a commit that referenced this issue Jul 1, 2016
Add servo-latest on upload-nightly

Allows for easy linking over to servo-nightly builds.

cc/ @larsbergstrom

---
<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: -->
- [X] `./mach build -d` does not report any errors
- [X] `./mach test-tidy` does not report any errors
- [ ] These changes fix #11987 (github issue number if applicable).

<!-- Either: -->
- [ ] There are tests for these changes OR
- [X] These changes do not require tests because they are part of the CI infrastructure

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/12004)
<!-- Reviewable:end -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

5 participants
You can’t perform that action at this time.