Skip to content

Add support for .NET Framework 4.7#19

Merged
shirhatti merged 2 commits into
masterfrom
kmcgain-dotnet4.7
Jun 13, 2017
Merged

Add support for .NET Framework 4.7#19
shirhatti merged 2 commits into
masterfrom
kmcgain-dotnet4.7

Conversation

@richlander

@richlander richlander commented Jun 5, 2017

Copy link
Copy Markdown
Member

Comment thread 4.7/Dockerfile Outdated
Invoke-WebRequest -Uri "https://download.microsoft.com/download/D/D/3/DD35CC25-6E9C-484B-A746-C5BE0C923290/NDP47-KB3186497-x86-x64-AllOS-ENU.exe" \
-OutFile C:\NDP47-KB3186497-x86-x64-AllOS-ENU.exe

RUN ["C:\\NDP47-KB3186497-x86-x64-AllOS-ENU.exe", "/q", "/norestart", "/ChainingPackage", "ADMINDEPLOYMENT"] \

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why multiple layers? It seems inefficient that a layer is getting created that contains the .exe that is getting removed later.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops. I reduced the layers from the original PR but not this one. Not sure why I didn't do that. Thanks.

@richlander richlander force-pushed the kmcgain-dotnet4.7 branch from 8447b55 to d9a3a19 Compare June 7, 2017 02:22
@richlander

Copy link
Copy Markdown
Member Author

Squashed my commits into one, leaving just the two commits. This means that the commit from @kmcgain will remain in-tact. Will use a merge commit when we land the PR.

richlander added a commit to microsoft/wcf-docker that referenced this pull request Jun 12, 2017
- Add .NET Framework 4.7 Dockerfile
- Update 4.6.2 Dockerfile to FROM dotnet-framework:4.6.2
- Add IIS components directly in place of FROM iis
- Dependent on
microsoft/dotnet-framework-docker#19
- In support of https://github.com/dotnet/designs/issues/7
@shirhatti shirhatti merged commit 0d9973d into master Jun 13, 2017
@MichaelSimons MichaelSimons deleted the kmcgain-dotnet4.7 branch October 20, 2017 14:05
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.

5 participants