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

How to best handle custom installation paths #336

Open
travispaul opened this issue Mar 10, 2016 · 3 comments
Open

How to best handle custom installation paths #336

travispaul opened this issue Mar 10, 2016 · 3 comments

Comments

@travispaul
Copy link

I need to install most packages (aside from salt itself) into the path D:\Program Files to achieve this I have used a custom grain called program_files in my package sls files:

nodejs:
  '5.7.1':
    full_name: Node.js
    installer: 'https://nodejs.org/dist/v5.7.1/node-v5.7.1-x64.msi'
    install_flags: 'INSTALLDIR="{{ grains['program_files'] }}\\Node.js" /qb'
    msiexec: True
    locale: en_US
    reboot: False

But using the grain in this fashion requires me to fork any packages that I want to change and prevents me from sharing these packages upstream. Is there a better way to override the installation path? I experimented with using the environ state to override the %ProgramFiles% environment variable without any success.

@rickh563
Copy link

ZD-689

@TheBigBear
Copy link
Contributor

TheBigBear commented May 17, 2016

@rickh563 not sure what ZD-689 means, but this and similar winrepo-ng improvements should be possible without any changes required? Just using jinja2 power and syntax in combination with salt environments and pillar lookup properly, right?
other places would be here #398 and here #350 and here #144 ?

@twangboy
Copy link
Collaborator

@TheBigBear ZD numbers reference our internal tracking system for enterprise customers.

TheBigBear added a commit that referenced this issue May 21, 2016
…2015.8.10-1

Update salt-minion to ver. 2015.8.10
TheBigBear added a commit that referenced this issue May 21, 2016
Merge pull request #336 from TheBigBear/TheBigBear-upd-salt-minion-v-2015.8.10-1
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

No branches or pull requests

4 participants