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

[cmake] Hook up SWIFT_STDLIB_HAS_FILESYSTEM macro #78453

Merged

Conversation

etcwilde
Copy link
Contributor

@etcwilde etcwilde commented Jan 7, 2025

This patch hooks up the SWIFT_STDLIB_HAS_FILESYSTEM macro define in the new swiftCore build system, while also setting SWIFT_LIB_SUBDIR and SWIFT_ARCH, which is required as part of finding the backtracer in the swiftRuntime target.

The new build-system flags to set this up are

  • SwiftCore_ENABLE_FILESYSTEM_SUPPORT
  • SwiftCore_ARCH_SUBDIR
  • SwiftCore_PLATFORM_SUBDIR

rdar://142440689

@etcwilde
Copy link
Contributor Author

etcwilde commented Jan 7, 2025

@swift-ci please test

This hooks up the SWIFT_STDLIB_HAS_FILESYSTEM, which then requires
setting SWIFT_ARCH and SWIFT_LIB_SUBDIR macros for the backtracer.

Note: `FILESYSTEM` in this case does not refer to the C++ filesystem
header, but whether the platform you're building for has a filesystem.
Updating the Apple cmake caches to enable the filesystem support,
setting the appropriate architecture and platform subdirectories.
@etcwilde etcwilde force-pushed the ewilde/stdlib-rebuild-has-filesystem branch from afc7495 to aed70b3 Compare January 7, 2025 17:24
@etcwilde
Copy link
Contributor Author

etcwilde commented Jan 7, 2025

@swift-ci please smoke test

@etcwilde etcwilde requested a review from compnerd January 7, 2025 17:24
@etcwilde
Copy link
Contributor Author

etcwilde commented Jan 7, 2025

@swift-ci please smoke test Linux platform

@etcwilde etcwilde merged commit 51b3ffd into swiftlang:main Jan 8, 2025
3 checks passed
@etcwilde etcwilde deleted the ewilde/stdlib-rebuild-has-filesystem branch January 8, 2025 17:17
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