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

Use OS X 10.11 on Travis #382

Closed
aneeshusa opened this issue May 24, 2016 · 3 comments
Closed

Use OS X 10.11 on Travis #382

aneeshusa opened this issue May 24, 2016 · 3 comments
Assignees

Comments

@aneeshusa
Copy link
Member

@aneeshusa aneeshusa commented May 24, 2016

@edunham is switching all of our Mac builders to OS X 10.11. Because our Salt states change the underlying system in various ways, they could be affected by the specific OS version, so we should test using the same version.

In order to this, we should set the osx_image key in the .travis.yml file for any Mac builders. We'll need to set the value to either xcode7.3 or xcode7.2 to get OS X 10.11 - @edunham @larsbergstrom do you know which Xcode we install? (That should also get documented on the wiki.)

@vigneshsarma
Copy link

@vigneshsarma vigneshsarma commented Sep 24, 2016

Hi, can I do this?

Only thing missing seems to be what xcode version to use. Can we go with xcode7.3 since it is the newer one?

@aneeshusa
Copy link
Member Author

@aneeshusa aneeshusa commented Sep 24, 2016

Hi @vigneshsarma, unfortunately I already have a commit for this in an open PR I'm working on so this is no longer on the table! (It turns out this is a lot more involved than just adding the osx_image key due to rippling effects on the build, so this isn't an E-easy either). You're welcome to look at some of our other open issues though!

@aneeshusa aneeshusa added C-assigned and removed E-easy labels Sep 24, 2016
@aneeshusa
Copy link
Member Author

@aneeshusa aneeshusa commented Sep 24, 2016

Going to be fixed in #483.

@aneeshusa aneeshusa self-assigned this Sep 24, 2016
@aneeshusa aneeshusa removed the C-assigned label Sep 24, 2016
bors-servo added a commit that referenced this issue Sep 27, 2016
Fix Homebrew 1.0.0 fallout

Homebrew 1.0.0 has just been released with some breaking changes. To handle all of them, this PR has a lot of changes, here are the main points:
- Add a new `homebrew` execution module and `hombrew_analytics` state module to fix analytics disabling
- Fix `autoconf`/`autoconf213` installing and linking
- Use env vars to specify OpenSSL location because linking fails (fixes #479)
- Upgrade to XCode 8/OS X 10.11 on Travis (fixes #382) to fix installing git
- Update Salt, leaping two major versions ahead to 2016.3.3 (the latest), due to changes in Homebrew Python/Salt packaging not available in our pinned old Salt
- Make our install and dispatch scripts more robust:
  - Force apt-get to use existing configuration files and ignore updates in Salt packages during installation
  - Unlink Salt if we have installed it via Homebrew previously before installing a new one
  - Allow failures when building old revs when checking upgrades (upstreams change)
  - Manually invalidate the Salt cache between runs

See commit messages for full details.

<!-- 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/483)
<!-- 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.

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