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

Homebrew nightly build not updated anymore (since 11/4) #535

Closed
paulrouget opened this issue Nov 7, 2016 · 4 comments
Closed

Homebrew nightly build not updated anymore (since 11/4) #535

paulrouget opened this issue Nov 7, 2016 · 4 comments

Comments

@paulrouget
Copy link
Contributor

@paulrouget paulrouget commented Nov 7, 2016

./etc/ci/update_brew.sh
 in dir /Users/servo/buildbot/slave/mac-nightly/build (timeout 1200 secs)
 watching logfiles {}
 argv: ['./etc/ci/update_brew.sh']
 using PTY: False
ls: target/brew/servo-????-??-??.tar.gz: No such file or directory
program finished with exit code 1
elapsedTime=0.017824
@aneeshusa
Copy link
Member

@aneeshusa aneeshusa commented Nov 7, 2016

Probably due to servo/servo#14058, will investigate later.

@paulrouget
Copy link
Contributor Author

@paulrouget paulrouget commented Nov 18, 2016

@aneeshusa anything I can do to help?

@paulrouget
Copy link
Contributor Author

@paulrouget paulrouget commented Nov 21, 2016

Now that etc/ci/upload_nightly.sh creates the timestamp, how can update_brew.sh know the proper url for that package?

I see 3 options:

  1. in upload_nightly, store the timestamp somewhere for re-use in update_brew.sh
  2. call update_brew from upload_nightly, not from buildbot_step (with timestamp or path of the tarball as a parameter)
  3. put together the timestamp earlier and store it in a shared env variable
@aneeshusa
Copy link
Member

@aneeshusa aneeshusa commented Nov 25, 2016

Sorry about the delay, that's indeed the root of the issue; I think option 2 is the best remediation.

bors-servo added a commit to servo/servo that referenced this issue Jan 5, 2017
Fix brew formula update

Fix servo/saltfs#535

I decided to merge update_brew within upload_nightly.

I hope this will fix the brew formula upadte.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/14403)
<!-- Reviewable:end -->
bors-servo added a commit to servo/servo that referenced this issue Jan 6, 2017
Fix brew formula update

Fix servo/saltfs#535

I decided to merge update_brew within upload_nightly.

I hope this will fix the brew formula upadte.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/14403)
<!-- Reviewable:end -->
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this issue Feb 4, 2017
…w); r=aneeshusa

Fix servo/saltfs#535

I decided to merge update_brew within upload_nightly.

I hope this will fix the brew formula upadte.

Source-Repo: https://github.com/servo/servo
Source-Revision: 207f9a5d3bb59cfc20b4455531325a3eb5a5ccf3
gecko-dev-updater pushed a commit to marco-c/gecko-dev-comments-removed that referenced this issue Oct 1, 2019
…w); r=aneeshusa

Fix servo/saltfs#535

I decided to merge update_brew within upload_nightly.

I hope this will fix the brew formula upadte.

Source-Repo: https://github.com/servo/servo
Source-Revision: 207f9a5d3bb59cfc20b4455531325a3eb5a5ccf3

UltraBlame original commit: 168b57b0e5500a14c68e8040c0ff57f8df9b711f
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified that referenced this issue Oct 1, 2019
…w); r=aneeshusa

Fix servo/saltfs#535

I decided to merge update_brew within upload_nightly.

I hope this will fix the brew formula upadte.

Source-Repo: https://github.com/servo/servo
Source-Revision: 207f9a5d3bb59cfc20b4455531325a3eb5a5ccf3

UltraBlame original commit: 168b57b0e5500a14c68e8040c0ff57f8df9b711f
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified-and-comments-removed that referenced this issue Oct 1, 2019
…w); r=aneeshusa

Fix servo/saltfs#535

I decided to merge update_brew within upload_nightly.

I hope this will fix the brew formula upadte.

Source-Repo: https://github.com/servo/servo
Source-Revision: 207f9a5d3bb59cfc20b4455531325a3eb5a5ccf3

UltraBlame original commit: 168b57b0e5500a14c68e8040c0ff57f8df9b711f
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.

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