Sitelet https://github.com/PowerShell/PowerShell/pull/5668
Skip to content

updated doc to build arm/arm64 versions of psrp.windows library - #5668

Merged
Dongbo Wang (daxian-dbw) merged 2 commits into
PowerShell:masterfrom
SteveL-MSFT:psrpwindows-doc
Dec 11, 2017
Merged

Dongbo Wang (daxian-dbw) merged 2 commits into
PowerShell:masterfrom
SteveL-MSFT:psrpwindows-doc

Conversation

@SteveL-MSFT

Copy link
Copy Markdown
Member

PR Summary

The doc to build and publish psrp.windows library needed to be updated to support arm and arm64 builds. It was also missing steps for signing and publishing. Added details on individual features needed for VS2017 and location to publish on MyGet.

PR Checklist

Note: Please mark anything not applicable to this PR NA.

Comment thread docs/building/internals.md Outdated
The `x64_arm` and `x64_arm64` architectures mean that the host system needs to be x64 to cross-compile to ARM.
When building for multiple architectures, be sure to use the `-clean` switch as cmake will cache the previous run and the wrong compiler will be used to generate the subsequent architecutes.

After that, the binary `pwrshplugin.dll` and its PDB file will be placed under 'src\powershell-native\bin\release\coreclr'.

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.

Maybe we should use the path 'src\powershell-win-core' here. The dll and pdb files will be bin-placed to "src\powershell-win-core" (see the script), and it's the same to 'PowerShell.Core.Instrumentation.dll' which will also be produced when running Start-BuildNativeWindowsBinaries.

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.

Ok, will change it.

Comment thread docs/building/internals.md Outdated
```

Lastly, run `nuget pack .` from within the folder. Note that you may need the latest `nuget.exe`.
Have the dlls signed and run `nuget pack` from the parent of the `runtimes` folder where `psrp.windows.nuspec` resides.

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.

Have the dlls signed

Please add "with authenticode dual certificate"

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.

Will add.

@daxian-dbw

Copy link
Copy Markdown
Member

Steve Lee (@SteveL-MSFT) I just released we don't have arm32/arm64 version of PowerShell.Core.Instrumentation.dll for the PowerShell.Core.Instrumentation nuget package ...

Can we build PowerShell.Core.Instrumentation.dll with x64_arm64 and x64_arm?

@SteveL-MSFT

Copy link
Copy Markdown
Member Author

Dongbo Wang (@daxian-dbw) I'll publish a new pkg for those dlls

@SteveL-MSFT

Copy link
Copy Markdown
Member Author

I'll merge the psrp.windows and powershell.core.instrumentation sections into a general Windows native section

address PR feedback
merge PowerShell.Core.Instrumentation section into PSRP.Windows section
@daxian-dbw
Dongbo Wang (daxian-dbw) merged commit 78271eb into PowerShell:master Dec 11, 2017
@SteveL-MSFT

Copy link
Copy Markdown
Member Author

@SteveL-MSFT
Steve Lee (SteveL-MSFT) deleted the psrpwindows-doc branch December 11, 2017 22:50
@daxian-dbw Dongbo Wang (daxian-dbw) added this to the 6.0.0-RC.2 milestone Dec 12, 2017
Dongbo Wang (daxian-dbw) pushed a commit that referenced this pull request Dec 12, 2017
@daxian-dbw Dongbo Wang (daxian-dbw) mentioned this pull request Dec 12, 2017
5 of 7 tasks
Dongbo Wang (daxian-dbw) pushed a commit that referenced this pull request Dec 12, 2017
Thatgfsj (Thatgfsj) pushed a commit to Thatgfsj-contribs/PowerShell that referenced this pull request Aug 6, 2026
Thatgfsj (Thatgfsj) pushed a commit to Thatgfsj-contribs/PowerShell that referenced this pull request Aug 6, 2026
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.

2 participants