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

Set file-system as device-agnostic in build manifest #7052

Merged
merged 1 commit into from Nov 14, 2023

Conversation

pusukuri
Copy link
Contributor

@pusukuri pusukuri commented Nov 3, 2023

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.

@MaxDesiatov
Copy link
Member

@swift-ci test

Copy link
Member

@MaxDesiatov MaxDesiatov left a 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.

@MaxDesiatov MaxDesiatov added the build system Changes to interactions with build systems label Nov 5, 2023
@pusukuri
Copy link
Contributor Author

pusukuri commented Nov 6, 2023

Looking into the tests. Thank you, @MaxDesiatov

@MaxDesiatov
Copy link
Member

@swift-ci test

@MaxDesiatov MaxDesiatov changed the title sets file-system as device-agnostic in build manifest rdar://96403757 Set file-system as device-agnostic in build manifest Nov 13, 2023
@MaxDesiatov
Copy link
Member

@swift-ci test windows

Copy link
Member

@MaxDesiatov MaxDesiatov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SGTM, thanks!

@neonichu
Copy link
Member

@swift-ci test windows

@pusukuri pusukuri merged commit 9c290df into apple:main Nov 14, 2023
5 checks passed
MaxDesiatov pushed a commit that referenced this pull request Dec 12, 2023
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.
MaxDesiatov added a commit that referenced this pull request Dec 13, 2023
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>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build system Changes to interactions with build systems enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants