Skip to content
This repository has been archived by the owner on Jan 23, 2023. It is now read-only.
/ corefx Public archive

Adds OSPlatform.FreeBSD. #2533

Closed
wants to merge 1 commit into from
Closed

Adds OSPlatform.FreeBSD. #2533

wants to merge 1 commit into from

Conversation

ghost
Copy link

@ghost ghost commented Jul 25, 2015

In #2068, it was decided to hold off adding public property for FreeBSD
because at the time there wasn't much progress made for the platform
support. Since then, there has been some progress made for FreeBSD.
Also with #2141 merged, I have added a missing test for S.RT.I.RI.

In #2068, it was decided to hold off adding public property for FreeBSD
because at the time there wasn't much progress made for the platform
support. Since then, there has been some progress made for FreeBSD.
Also with #2141 merged, I have added a missing test for S.RT.I.RI.
@@ -1,15 +1,13 @@
// Copyright (c) Microsoft. All rights reserved.
// Copyright (c) Microsoft. All rights reserved.
Copy link
Author

Choose a reason for hiding this comment

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

I have added 💣, as this was the only .cs file in this directory missing BOM.

@stephentoub
Copy link
Member

Thanks, @jasonwilliams200OK. It's great that some progress has been made, but I don't think it's far enough along yet, and progress has slowed of late. I think we need to establish some tripwire for when we accept adding such a property, e.g. 90% of corefx tests passing on the platform, or something like that. @weshaggard? @ellismg? @joshfree?

@ellismg
Copy link
Contributor

ellismg commented Jul 27, 2015

I would be interested in understanding what percent of the tests that pass on Linux/OSX today are passing on FreeBSD. I think for now we should be very close/at parity with this pass rate (which we already know is worse than the Windows pass rate, due to tests not running) before merging something like this.

@stephentoub
Copy link
Member

I'm going to close this for now. We can revisit once the FreeBSD port is closer to being completed. Thanks.

@stephentoub stephentoub closed this Aug 3, 2015
@karelz karelz added this to the 1.0.0-rtm milestone Jan 25, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants