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

macOS Sierra support #715

Merged
merged 2 commits into from
Nov 15, 2016
Merged

macOS Sierra support #715

merged 2 commits into from
Nov 15, 2016

Conversation

kameghamegha
Copy link
Contributor

Added Sierra packer template (macos-10.12.json) and updated scripts/macosx/updates.sh to support Sierra.

I've tested this on Virtualbox but don't have access to Parallels or VMware.

Just duplicated the 10.11 template and changed the title to remove the x (since it's just MacOS now) and update the version. Also updated the version inside the variables section to the same. Virtualbox guest_os_type stays as MacOS1011_64 because there is no 1012 official support yet from virtualbox, but the 1011 os type works.

Signed-off-by: Meg Cassidy <kameghamegha@users.noreply.github.com>
This section would hang at the "Searching for Software Updates" prompt and never finish. To fix it, I had to add the --verbose and --force flags to be compatible with 10.12, as well as triggering a background scan. It would then fail during the vboxmanage_post command for the IDE Controller, because the shutdown was failing to complete properly (the background scan didn't seem to be finished). The sleep line fixes this.

Signed-off-by: Meg Cassidy <kameghamegha@users.noreply.github.com>
@kameghamegha kameghamegha changed the title Macos sierra support macOS Sierra support Nov 11, 2016
@kameghamegha
Copy link
Contributor Author

I tested this with a 10.11 host, packer 0.11.0, virtualbox 5.0.26, and vagrant 1.8.1. I was receiving some rsync failures with the newest versions of virtualbox and vagrant when I tried to do kitchen create/vagrant up, so I rolled them back (not sure which one was giving me the errors).

@legal90
Copy link
Contributor

legal90 commented Nov 12, 2016

Seems like this PR supersedes #664, but here is softwareupdate issue fixed, so LGTM 👍

@cheeseplus cheeseplus merged commit 725e232 into chef:master Nov 15, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants