Skip to content

[FreeBSD] Fix target conditionals for FreeBSD aarch64 builds #984

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
May 22, 2025

Conversation

CodingCarpincho
Copy link
Contributor

@CodingCarpincho CodingCarpincho commented May 6, 2025

Despite being declared in the header unistd.h, the function sbrk is not available on FreeBSD when using the aarch64 architecture because FreeBSD made the decision to drop support for this deprecated function when porting the system to ARM.

The configuration script incorrectly hard-codes the support for this function as being always available, which is not correct. We need to ensure that it is not used on FreeBSD aarch64.

Despite being declared in the header unistd.h, the function sbrk is not available on FreeBSD when using the aarch64 architecture because FreeBSD make the decision to drop support for this deprecated function when porting the system to ARM.

The configuration script incorrectly hard-codes the support for this function as being always available, which is not correct. We need to ensure that it is not used on FreeBSD aarch64.
@CodingCarpincho CodingCarpincho requested a review from dmbryson as a code owner May 6, 2025 21:10
@jakepetroules
Copy link
Contributor

@swift-ci test

@jakepetroules
Copy link
Contributor

@swift-ci smoke test

@jakepetroules
Copy link
Contributor

@swift-ci smoke test linux

@jakepetroules
Copy link
Contributor

@swift-ci smoke test linux

1 similar comment
@jakepetroules
Copy link
Contributor

@swift-ci smoke test linux

@jakepetroules jakepetroules enabled auto-merge (rebase) May 16, 2025 00:48
@jakepetroules
Copy link
Contributor

@swift-ci test linux

@jakepetroules
Copy link
Contributor

@swift-ci smoke test linux

1 similar comment
@jakepetroules
Copy link
Contributor

@swift-ci smoke test linux

@jakepetroules jakepetroules merged commit 72b0343 into swiftlang:main May 22, 2025
6 checks passed
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.

3 participants