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

Parallels: Support of FreeBSD and OmniOS. Some fixes #278

Merged
merged 3 commits into from
Oct 30, 2014

Conversation

legal90
Copy link
Contributor

@legal90 legal90 commented Oct 6, 2014

  • Minor fixes for centos-6.5-i386.json and debian-7.6-i386.json templates.
  • Added support of paralells-iso builder to the OmniOS template.
  • Added support of paralells-iso builder to all FreeBSD templates.

OmniOS & FreeBSD are supported only since hashicorp/packer#1502 changeset, which is not released yet. (Packer 0.7.2)

@juliandunn
Copy link
Contributor

Thanks! I will wait for Packer 0.7.2 before merging.

@juliandunn
Copy link
Contributor

This'll need a rebase before merge -- the minor fixes in CentOS and Debian have already been accepted.

I see Packer 0.7.2 still isn't out.

@legal90
Copy link
Contributor Author

legal90 commented Oct 30, 2014

@juliandunn Sure, I've just rebased it.
I guess TravisCI validation will pass after upgrading Packer to 0.7.2

I see Packer 0.7.2 still isn't out.

Hm... Packer's changelog says that it has been released in Oct 28th. Also, 0.7.2 is available on download page: https://packer.io/downloads.html

@legal90
Copy link
Contributor Author

legal90 commented Oct 30, 2014

I've just fixed the Travis build failure and rebased again.
Now it's ready to merge :)

@juliandunn
Copy link
Contributor

Oh, awesome! Thanks @legal90 -- guess I had a cached version of the page or something. Merging.

juliandunn added a commit that referenced this pull request Oct 30, 2014
Parallels: Support of FreeBSD and OmniOS. Some fixes
@juliandunn juliandunn merged commit 6e060a9 into chef:master Oct 30, 2014
@legal90 legal90 deleted the parallels-improve branch October 30, 2014 13:19
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.

2 participants