Skip to content

Build Global tool for PowerShell and SDK container - #8984

Merged
Aditya Patwardhan (adityapatwardhan) merged 22 commits into
PowerShell:masterfrom
adityapatwardhan:FxDependentPkgOptimize
Feb 26, 2019
Merged

Build Global tool for PowerShell and SDK container#8984
Aditya Patwardhan (adityapatwardhan) merged 22 commits into
PowerShell:masterfrom
adityapatwardhan:FxDependentPkgOptimize

Conversation

@adityapatwardhan

@adityapatwardhan Aditya Patwardhan (adityapatwardhan) commented Feb 26, 2019

Copy link
Copy Markdown
Member

PR Summary

This PR builds a unified global tool to support all the runtimes as well as two separate reduced size global tools to be included in dotnet sdk container images.

PR Context

PR Checklist

Comment thread src/Microsoft.PowerShell.GlobalTool.Shim/GlobalToolShim.cs Outdated
[Parameter(Mandatory)] $DestinationPath
)
## Remove unnecessary files
$localeFolderToRemove = 'cs', 'de', 'es', 'fr', 'it', 'ja', 'ko', 'pl', 'pt-BR', 'ru', 'tr', 'zh-Hans', 'zh-Hant'

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

What if there will new folder? Maybe not "remove" but "use/filter only necessary folder"?

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.

Can I take this as a bug fix after this PR?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Yes.

Comment thread src/Microsoft.PowerShell.GlobalTool.Shim/GlobalToolShim.cs
Comment thread tools/packaging/packaging.strings.psd1 Outdated
Comment thread tools/releaseBuild/azureDevOps/templates/nuget.yml Outdated
@adityapatwardhan
Aditya Patwardhan (adityapatwardhan) merged commit fb05169 into PowerShell:master Feb 26, 2019
@adityapatwardhan
Aditya Patwardhan (adityapatwardhan) deleted the FxDependentPkgOptimize branch February 26, 2019 20:56
@iSazonov Ilya (iSazonov) added the CL-Tools Indicates that a PR should be marked as a tools change in the Change Log label Feb 27, 2019
Thatgfsj (Thatgfsj) pushed a commit to Thatgfsj/PowerShell that referenced this pull request Aug 6, 2026
* Add shim dll to redirect to appropriate runtime
* Build updates for creating global tool package
* Add building global tool to nuget.yml
* Add azure upload of nuget packages
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CL-Tools Indicates that a PR should be marked as a tools change in the Change Log

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants