Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add runtime.osx-arm64.runtime.native.System.IO.Ports proj file for pa… #60310

Merged

Conversation

SeanMollet
Copy link
Contributor

Fixes #59972

Customer Impact

The native library for System.IO.Ports is current not built nor packaged for osx-arm64. The native library compiles cleanly and works correctly, but just isn't being built or packaged by the CI system.

This change adds a package project to build/package it, which also adds it as a dependency to the meta runtime.native.System.IO.Ports package

Testing

Tested on actual Apple Silicon hardware with various serial devices. Since no current M1 device offers native serial ports, all testing was conducted with various USB devices that convert to or present as serial ports. Works correctly.

Risk

Low. The alternative is that there is no support at all.

@ViktorHofer As requested

…ckaging.

Tested on actual osx-arm64 hardware, resulting package works correctly.
@ViktorHofer
Copy link
Member

cc @jozkee

@ghost ghost added the community-contribution Indicates that the PR has been added by a community member label Oct 12, 2021
@ghost
Copy link

ghost commented Oct 12, 2021

Tagging subscribers to this area: @dotnet/area-system-io
See info in area-owners.md if you want to be subscribed.

Issue Details

Fixes #59972

Customer Impact

The native library for System.IO.Ports is current not built nor packaged for osx-arm64. The native library compiles cleanly and works correctly, but just isn't being built or packaged by the CI system.

This change adds a package project to build/package it, which also adds it as a dependency to the meta runtime.native.System.IO.Ports package

Testing

Tested on actual Apple Silicon hardware with various serial devices. Since no current M1 device offers native serial ports, all testing was conducted with various USB devices that convert to or present as serial ports. Works correctly.

Risk

Low. The alternative is that there is no support at all.

@ViktorHofer As requested

Author: SeanMollet
Assignees: -
Labels:

area-System.IO

Milestone: -

<Import Project="runtime.native.System.IO.Ports.props" />
<PropertyGroup>
<!-- TODO: Remove when the package shipped. -->
<DisablePackageBaselineValidation>true</DisablePackageBaselineValidation>
Copy link
Member

Choose a reason for hiding this comment

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

If this will be shipping in 6.0, do we still need this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Right now, yes, because it's not out in 6.0 and there isn't a nuget package the build process can pull down. Once it's actually in 6.0 and there's a package, then it can be removed.

@jozkee jozkee merged commit 9dc925d into dotnet:main Oct 13, 2021
@jozkee
Copy link
Member

jozkee commented Oct 13, 2021

Thanks, @SeanMollet!

@ghost ghost locked as resolved and limited conversation to collaborators Nov 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-System.IO.Ports community-contribution Indicates that the PR has been added by a community member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

runtime.osx-arm64.runtime.native.System.IO.Ports missing
4 participants