Skip to content
This repository has been archived by the owner on Sep 14, 2021. It is now read-only.

Incorrect parameter: microsoft-build-tools #813

Closed
jbest84 opened this issue Apr 28, 2016 · 8 comments
Closed

Incorrect parameter: microsoft-build-tools #813

jbest84 opened this issue Apr 28, 2016 · 8 comments

Comments

@jbest84
Copy link

jbest84 commented Apr 28, 2016

There is no /qn flag for the BuildTools_Full.exe installer. This causes the installer to open a winforms dialog with the list of params. The minion keeps the process open indefinitely.

@TheBigBear
Copy link
Contributor

@jbest84 thanks for reporting this

@TheBigBear
Copy link
Contributor

@jbest84 what is the 'quiet' or 'silent' switch for this installer? Do you know? Is there one listed in the open winforms dialog?

@jbest84
Copy link
Author

jbest84 commented Apr 28, 2016

/Quite (/Q) and /Silent (/S)

I took a screenshot of the full list:
builttools_params

@jbest84
Copy link
Author

jbest84 commented Apr 28, 2016

I am relatively new to salt. Is there a way these package flags can be overriden in my state files via an include/extend? I tried doing this with no luck:

include:
  - win.winrepo-ng.salt-winrepo-ng.microsoft-build-tools

extend:
  microsoft-build-tools:
    install_flags: '/Q /NoRestart'

dotnet:
  pkg.installed: 
    - pkgs:
      - microsoft-build-tools

TheBigBear added a commit to TheBigBear/salt-winrepo that referenced this issue Apr 28, 2016
…d-tools-silent-install

Update microsoft-build-tools.sls as per comment and fix by @jbest84 here saltstack#813 (comment)
@TheBigBear
Copy link
Contributor

TheBigBear commented Apr 28, 2016

@jbest84 can you please test and confirm that this can be closed?

@jbest84
Copy link
Author

jbest84 commented Apr 28, 2016

@TheBigBear It worked. Thank you!

@jbest84 jbest84 closed this as completed Apr 28, 2016
@TheBigBear
Copy link
Contributor

@twangboy @jfindlay do you guys know if parameters can be 'overriden' in the way @jbest84 describes above in #813 (comment) ?
IF this can't be done already, I think this would make a great addition, but I suspect this type of 'over-riding' is probably already possible? Rigth?

@twangboy
Copy link
Contributor

twangboy commented May 3, 2016

@TheBigBear I think it would be possible. I have added it to my list (which is getting large) of winrepo things to do.

TheBigBear added a commit that referenced this issue Nov 5, 2016
Merge pull request #1149 from TheBigBear/travis-ci_cron_daily
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants