-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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 support for SunOS-like platforms #12198
Conversation
cc @dsplaisted, @eerhardt, like FreeBSD, we can cross-compile SDK for illumos distros with these minimal set of changes. This matches the implementation done in host. |
@@ -13,7 +13,9 @@ internal enum Platform | |||
Windows = 1, | |||
Linux = 2, | |||
Darwin = 3, | |||
FreeBSD = 4 | |||
FreeBSD = 4, | |||
illumos = 5, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this supposed to start with a lower case character?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The official casing is lowercase (to disambiguate letters i
and lowercase l
, like iOS). In the output we print the value as is.
If there are no other comments here, can this be merged? Thanks. |
switch (versionDescription) | ||
{ | ||
case string version when version.StartsWith("omnios"): | ||
result.Id = "OmniOS"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like it to me... @am11 thoughts?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah it will. the testing was blocked by System.Diagnostics.Process port on this platform. I have opened #14395.
These are the changes required to cross-compile and use SDK on illumos distors and Oracle Solaris OS.