Skip to content

Update Pi demo instructions if libunwind8 is not installed#4974

Merged
adityapatwardhan merged 2 commits intoPowerShell:masterfrom
SteveL-MSFT:pi-update
Oct 4, 2017
Merged

Update Pi demo instructions if libunwind8 is not installed#4974
adityapatwardhan merged 2 commits intoPowerShell:masterfrom
SteveL-MSFT:pi-update

Conversation

@SteveL-MSFT
Copy link
Copy Markdown
Member

added instructions to install libunwind8

Copy link
Copy Markdown

@anmenaga anmenaga left a comment

Choose a reason for hiding this comment

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

LGTM


Note that until arm32 is [fully supported by CoreCLR](https://github.com/dotnet/coreclr/issues/3977), it's not supported by PowerShell Core.

If you get an error complaining about `libunwind.so.8` not being found, you'll need to install it as it's required by CoreCLR
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Period missing at the end of the sentence.

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 fix.

Copy link
Copy Markdown
Contributor

@markekraus markekraus left a comment

Choose a reason for hiding this comment

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

LGTM

@GoateePFE
Copy link
Copy Markdown

Can you guys go ahead and merge this today? I'm doing a conference presentation including this tomorrow, and I would like the attendees to have the update. Thanks. Ashley

@SteveL-MSFT
Copy link
Copy Markdown
Member Author

@adityapatwardhan please merge today

@adityapatwardhan adityapatwardhan merged commit ffec170 into PowerShell:master Oct 4, 2017
@SteveL-MSFT SteveL-MSFT deleted the pi-update branch October 16, 2017 21:53
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.

6 participants