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

Add servo/osmesa-src #484

Merged
merged 1 commit into from Sep 23, 2016
Merged

Add servo/osmesa-src #484

merged 1 commit into from Sep 23, 2016

Conversation

@larsbergstrom
Copy link
Contributor

larsbergstrom commented Sep 22, 2016

r? @aneeshusa @edunham


This change is Reviewable

@aneeshusa
Copy link
Member

aneeshusa commented Sep 23, 2016

r=me, not sure if you want to land servo/osmesa-src#3 first or this one first.

@aneeshusa aneeshusa self-assigned this Sep 23, 2016
@larsbergstrom
Copy link
Contributor Author

larsbergstrom commented Sep 23, 2016

@bors-servo r=aneeshusa

Yeah, typically I like to land and deploy this first so that I can land the .travis.yml fix as the first "homu approved" bit :-)

@bors-servo
Copy link
Contributor

bors-servo commented Sep 23, 2016

📌 Commit 065a45c has been approved by aneeshusa

@bors-servo
Copy link
Contributor

bors-servo commented Sep 23, 2016

Testing commit 065a45c with merge a79ad21...

bors-servo added a commit that referenced this pull request Sep 23, 2016
Add servo/osmesa-src

r? @aneeshusa @edunham

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/484)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Sep 23, 2016

💔 Test failed - status-travis

@larsbergstrom
Copy link
Contributor Author

larsbergstrom commented Sep 23, 2016

Hrm, this is a weird error:

----------

          ID: disable-homebrew-analytics

    Function: git.config

        Name: homebrew.analyticsdisabled

      Result: False

     Comment: An exception occurred in this state: Traceback (most recent call last):

                File "/usr/local/Cellar/saltstack/2015.5.5_1/libexec/lib/python2.7/site-packages/salt/state.py", line 1561, in call

                  **cdata['kwargs'])

                File "/usr/local/Cellar/saltstack/2015.5.5_1/libexec/lib/python2.7/site-packages/salt/states/git.py", line 573, in config

                  is_global=is_global)

                File "/usr/local/Cellar/saltstack/2015.5.5_1/libexec/lib/python2.7/site-packages/salt/modules/git.py", line 924, in config_set

                  return _git_run(cmd, cwd=cwd, runas=user)

                File "/usr/local/Cellar/saltstack/2015.5.5_1/libexec/lib/python2.7/site-packages/salt/modules/git.py", line 99, in _git_run

                  'Command {0!r} failed. Stderr: {1!r}'.format(cmd, stderr))

              CommandExecutionError: Command 'git config homebrew.analyticsdisabled "true"' failed. Stderr: 'error: could not lock config file .git/config: No such file or directory'

     Started: 15:01:04.872080

    Duration: 20.949 ms

     Changes:   
@larsbergstrom
Copy link
Contributor Author

larsbergstrom commented Sep 23, 2016

And the other mac builder just froze & died:

No output has been received in the last 10m0s, this potentially indicates a stalled build or something wrong with the build itself.

Check the details on how to adjust your build configuration on: https://docs.travis-ci.com/user/common-build-problems/#Build-times-out-because-no-output-was-received
@larsbergstrom larsbergstrom merged commit 5cddecb into servo:master Sep 23, 2016
0 of 2 checks passed
0 of 2 checks passed
continuous-integration/travis-ci/pr The Travis CI build failed
Details
homu Test failed
Details
@larsbergstrom
Copy link
Contributor Author

larsbergstrom commented Sep 23, 2016

r? @glennw

This should work once it's retested. Linux is now succeeding, homu support is in place, and we just need to use the updated macOS image from Travis.

@aneeshusa
Copy link
Member

aneeshusa commented Sep 23, 2016

@larsbergstrom, I think you may have commented on the wrong PR.

@larsbergstrom
Copy link
Contributor Author

larsbergstrom commented Sep 23, 2016

Thanks :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

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