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

dist: Support SmartOS #60

Closed
wants to merge 7 commits into from
Closed

dist: Support SmartOS #60

wants to merge 7 commits into from

Conversation

geek
Copy link
Member

@geek geek commented Feb 28, 2015

This only includes the basic setup for now. Still need to work on build

tags: general

- name: General | Install required packages
command: pkgin in {{ item }}

This comment was marked as off-topic.

@jbergstroem
Copy link
Member

LGTM

@jbergstroem
Copy link
Member

Another minor thing: Since path doesn't seem to be provided through /etc/profilewe might want to add it as a envvar in the manifest (otherwise jenkins won't find path to git). For now, I've changed tooling path through jenkins admin.

Edit: tooling path nor env injection seemed to work. We'll going to have to edit the manifest. Suggesting adding this:

<envvar name='PATH' value='/usr/local/sbin:/usr/local/bin:/opt/local/sbin:/opt/local/bin:/usr/sbin:/usr/bin:/sbin' />

@jbergstroem
Copy link
Member

(btw, confirmed that adding path fixed Jenkins)

@geek
Copy link
Member Author

geek commented Mar 10, 2015

Closing in favor of #64

@geek geek closed this Mar 10, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants