Skip to content

Conversation

@brycem
Copy link
Contributor

@brycem brycem commented Jan 19, 2017

Cleaned up RELEASE version Dockerfiles for Windows

Cleaned up RELEASE versions, and adding initial version of CI-friendly
NIGHTLY dockerfile for WindowsServerCore
Core build works, -FullCLR doesn't yet.
@PowerShellTeam PowerShellTeam added the Review - Needed The PR is being reviewed label Jan 19, 2017
@lzybkr
Copy link
Contributor

lzybkr commented Jan 23, 2017

Can you document how the image brycem/win10build was built?

We are discussing not building the C/C++ Windows components in PowerShell/PowerShell and instead pull packages from NuGet - would this eliminate the need for your custom image?

@brycem
Copy link
Contributor Author

brycem commented Jan 25, 2017 via email

@brycem
Copy link
Contributor Author

brycem commented Jan 27, 2017

@lzybkr - Where did you guys shake out on removing the Standalone Win10SDK dependency? It would definitely simplify. Looks like CMake is available in NuGet as well.

@codecov-io
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (master@f0a9581).


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f0a9581...d34a962. Read the comment docs.

@mirichmo
Copy link
Member

I filed issue #3014, but it hasn't progressed beyond that.

@mirichmo
Copy link
Member

mirichmo commented Mar 2, 2017

@raghushantha - Can you please take a look?

@mirichmo mirichmo requested a review from raghushantha March 2, 2017 01:22
If ($ZipHash -notmatch $Env:POWERSHELL_SHA256){`
Throw [String]$('['+$ZipHash+'] does not match ['+$Env:POWERSHELL_SHA256+']!')`
}`
[System.IO.DirectoryInfo]$PsFolder=New-Item -Path $Env:ProgramFiles\PowerShell -ItemType Directory -Force;`
Copy link
Member

Choose a reason for hiding this comment

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

Expand-Archive would simplify this. Here is an example.


ARG POWERSHELL_MSI=https://github.com/PowerShell/PowerShell/releases/download/v6.0.0-alpha.14/PowerShell_6.0.0.14-alpha.14-win10-x64.msi
ARG POWERSHELL_SHA256=503F3AD52223699765895D3E9615FBD7988194693BCB725BE90C9EF0CD594447
ARG POWERSHELL_MSI=https://github.com/PowerShell/PowerShell/releases/download/v6.0.0-alpha.15/PowerShell_6.0.0-alpha.15-win10-win2k16-x64.msi
Copy link
Member

Choose a reason for hiding this comment

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

@brycem Please pick up the latest release MSI here
https://github.com/PowerShell/PowerShell/releases/latest

Then the experience would be similar to Linux. There we simply apt-get install / yum install from packages.microsoft.com and the image is automatlically populated with latest PS

This comment applies all locations where the MSI is downloaded

ARG POWERSHELL_MSI=https://github.com/PowerShell/PowerShell/releases/download/v6.0.0-alpha.14/PowerShell_6.0.0.14-alpha.14-win10-x64.msi
ARG POWERSHELL_SHA256=503F3AD52223699765895D3E9615FBD7988194693BCB725BE90C9EF0CD594447
ARG POWERSHELL_MSI=https://github.com/PowerShell/PowerShell/releases/download/v6.0.0-alpha.15/PowerShell_6.0.0-alpha.15-win10-win2k16-x64.msi
ARG POWERSHELL_SHA256=CC52D21F3287E412B9C3B73C98BB5B06F8056D49D63201072216DF92B7F2E59B
Copy link
Member

Choose a reason for hiding this comment

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

Not really required to validate the hash. The MSI is authenticode signed & you are downloading from an HTTPS location

@brycem
Copy link
Contributor Author

brycem commented Apr 8, 2017

Looks like @TravisEz13 has updated the URLs in #3508 - I'll close this PR

@brycem brycem closed this Apr 8, 2017
@iSazonov iSazonov removed the Review - Needed The PR is being reviewed label Apr 8, 2017
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.

8 participants