Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Updating dockerfiles for Windows#3017

Closed
brycem wants to merge 3 commits into
PowerShell:masterPowerShell/PowerShell:masterfrom
brycem:masterbrycem/PowerShell:masterCopy head branch name to clipboard
Closed

Updating dockerfiles for Windows#3017
brycem wants to merge 3 commits into
PowerShell:masterPowerShell/PowerShell:masterfrom
brycem:masterbrycem/PowerShell:masterCopy head branch name to clipboard

Conversation

@brycem

@brycem brycem commented Jan 19, 2017

Copy link
Copy Markdown
Contributor

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

lzybkr commented Jan 23, 2017

Copy link
Copy Markdown
Contributor

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

brycem commented Jan 25, 2017 via email

Copy link
Copy Markdown
Contributor Author

@brycem

brycem commented Jan 27, 2017

Copy link
Copy Markdown
Contributor Author

@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
Copy Markdown

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
Copy Markdown
Member

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

@mirichmo

mirichmo commented Mar 2, 2017

Copy link
Copy Markdown
Member

@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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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

brycem commented Apr 8, 2017

Copy link
Copy Markdown
Contributor Author

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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.