-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Set file-system
as device-agnostic
in build manifest
#7052
Conversation
@swift-ci test |
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.
LLBuildManifestTests
need to be adjusted accordingly to make tests pass after the change.
Looking into the tests. Thank you, @MaxDesiatov |
@swift-ci test |
file-system
as device-agnostic
in build manifest
@swift-ci test windows |
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.
SGTM, thanks!
@swift-ci test windows |
sets file-system as device-agnostic in build manifest ### Motivation: This is the default mode in XCBuild, and allows incremental builds to work across reboots on APFS. It would also likely address #4651. ### Modifications: sets file-system as device-agnostic in build manifest ### Result: Restarting your system (or inode changes) doesn't invoke unnecessary builds Related to rdar://96403757.
Cherry-pick of #7052. ### Motivation: This is the default mode in XCBuild, and allows incremental builds to work across reboots on APFS. It would also likely address #4651. ### Modifications: Set `file-system` as `device-agnostic` in the generated build manifest ### Result: Restarting your system (or inode changes) doesn't invoke unnecessary builds Related to rdar://96403757. Co-authored-by: Kishore Pusukuri <kishoreguptaos@gmail.com>
sets file-system as device-agnostic in build manifest
Motivation:
This is the default mode in XCBuild, and allows incremental builds to work across reboots on APFS. It would also likely address #4651.
Modifications:
sets file-system as device-agnostic in build manifest
Result:
Restarting your system (or inode changes) doesn't invoke unnecessary builds
Related to rdar://96403757.