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

Skip tools installation in chocolatey mode (Windows installer) #1937

Merged
merged 3 commits into from Nov 13, 2019

Conversation

gilbertsoft
Copy link
Member

@gilbertsoft gilbertsoft commented Nov 11, 2019

The Problem/Issue/Bug:

Installing DDEV with Chocolaty should prefer the tools installed as dependencies instead of the ones bundled with the DDEV Windows installer

How this PR Solves The Problem:

Skips mkcert and nssm from installation in the new choco mode /S /C

Manual Testing Instructions:

Run the installer with options /S /C and check the tools are not installed. If they are already present they will be kept and prefered by DDEV

Automated Testing Overview:

Related Issue Link(s):

#1935

Release/Deployment notes:

Skips mkcert and nssm from installation in silent mode
@rfay rfay changed the title Skip tools installation in silent mode Skip tools installation in silent mode (Windows installer) Nov 11, 2019
@rfay
Copy link
Member

rfay commented Nov 11, 2019

It seems that this makes the assumption "silent mode" == "choco install". That might not be correct.

I hate to add too much complexity. Wonder if there's a way to detect choco install.

It's always possible that we should have a separate chocolatey installer :(

This patch forces the installtion of sudo as DDEV can normaly not run correctly and full function without it. The license page is now showed before the component selection because it is not optional as before.
@gilbertsoft
Copy link
Member Author

gilbertsoft commented Nov 11, 2019

It seems that this makes the assumption "silent mode" == "choco install". That might not be correct.

I hate to add too much complexity. Wonder if there's a way to detect choco install.

It's always possible that we should have a separate chocolatey installer :(

You're absolutely right but I don't think one will install DDEV in silent mode. So we can assume silent mode = choco install
As far as I know we can not check for an additonal cmd line parameter but will have a look. But I don't think we have take care

@gilbertsoft
Copy link
Member Author

As far as I know we can not check for an additonal cmd line parameter but will have a look. But I don't think we have take care

There is a way to get parameters and options from the command line. So I will add a choco parameter to the next commit...

@gilbertsoft
Copy link
Member Author

We have now a special Choco mode (see commit) enabled with the parameters /S /C

This patch adds a special Chocolatey mode which will prevent the installation of dependencies installed by Chocolatey.
@gilbertsoft gilbertsoft changed the title Skip tools installation in silent mode (Windows installer) Skip tools installation in chocolatey mode (Windows installer) Nov 11, 2019
@rfay
Copy link
Member

rfay commented Nov 13, 2019

Let's add this to the chocolatey build, please add to silentArgs, see https://github.com/drud/ddev/blob/master/winpkg/chocolatey/tools/chocolateyinstall.ps1

I see this was already done here. Not sure why I couldn't see it before.

@gilbertsoft
Copy link
Member Author

Let's add this to the chocolatey build, please add to silentArgs, see https://github.com/drud/ddev/blob/master/winpkg/chocolatey/tools/chocolateyinstall.ps1

That's already done with the related commit.

Currently I don't know why the test fails. Did a restart two times...

@rfay rfay merged commit 621efe2 into ddev:master Nov 13, 2019
@gilbertsoft gilbertsoft deleted the task-ignore-tools branch November 13, 2019 15:00
rfay pushed a commit to rfay/ddev that referenced this pull request Jan 15, 2020
…1937)

* Skip tools installation in silent mode

Skips mkcert and nssm from installation in silent mode

* Force installation of sudo

This patch forces the installtion of sudo as DDEV can normaly not run correctly and full function without it. The license page is now showed before the component selection because it is not optional as before.

* Add Chocolatey mode

This patch adds a special Chocolatey mode which will prevent the installation of dependencies installed by Chocolatey.
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.

None yet

2 participants