Skip to content

Make some preliminary porting changes. #536

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

Merged
merged 1 commit into from
Aug 14, 2020

Conversation

3405691582
Copy link
Member

@3405691582 3405691582 commented Feb 1, 2020

These changes include:

  • hw_config.h: prevent referring to sysctlbyname on OpenBSD, as this
    is not available on all platforms.

  • transform.c: these stanzas referring to FreeBSD or Linux also apply
    to OpenBSD.

  • tests/dispatch_apply.c: stanza applying to Linux applies to OpenBSD
    and also tweak style for consistency.

  • tests/dispatch_io_net.c, tests/dispatch_test.h: stanzas applying to
    FreeBSD also apply to OpenBSD.

These changes include:

  * hw_config.h: prevent referring to sysctlbyname on OpenBSD, as this
    is not available on all platforms.

  * transform.c: these stanzas referring to FreeBSD or Linux also apply
    to OpenBSD.

  * tests/dispatch_apply.c: stanza applying to Linux applies to OpenBSD
    and also tweak style for consistency.

  * tests/dispatch_io_net.c, tests/dispatch_test.h: stanzas applying to
    FreeBSD also apply to OpenBSD.
@3405691582 3405691582 force-pushed the NoSysctlbyname_AddOSSymbol branch from 0ceaf5f to 35e8017 Compare February 1, 2020 01:37
@compnerd
Copy link
Member

compnerd commented Feb 4, 2020

@swift-ci please test

@compnerd
Copy link
Member

compnerd commented Feb 4, 2020

CC: @ktopley-apple

@compnerd compnerd merged commit f13ea5d into swiftlang:master Aug 14, 2020
rokhinip pushed a commit that referenced this pull request Nov 5, 2021
Make some preliminary porting changes.

Signed-off-by: Rokhini Prabhu <rokhinip@apple.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants