Skip to content

Conversation

@benlangmuir
Copy link

For IncludeTreeFileSystem, contents (and recursive contents, since there is no distinction) prints the list of files in sorted order along with the casid of their contents.

For CASFileSystem, contents prints the root casid, while recursive contents additionally walks the full recursive contents printing each entry.

For CachingOnDiskFileSystem this is currently a stub that just prints the name of the filesystem with no contents. This is still helpful to know what kind of filesystem is in use for debugging.

@benlangmuir
Copy link
Author

@swift-ci please test llvm

Copy link

@jansvoboda11 jansvoboda11 left a comment

Choose a reason for hiding this comment

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

LGTM

For IncludeTreeFileSystem, contents (and recursive contents, since there is no
distinction) prints the list of files in sorted order along with the casid of
their contents.

For CASFileSystem, contents prints the root casid, while recursive contents
additionally walks the full recursive contents printing each entry.

For CachingOnDiskFileSystem this is currently a stub that just prints the name
of the filesystem with no contents. This is still helpful to know what kind of
filesystem is in use for debugging.
@benlangmuir benlangmuir force-pushed the eng/blangmuir/filesystem-print-impl-cas branch from 11f7f8c to 4d62632 Compare December 17, 2025 21:36
@benlangmuir
Copy link
Author

@swift-ci please test llvm

1 similar comment
@benlangmuir
Copy link
Author

@swift-ci please test llvm

@benlangmuir benlangmuir merged commit de3f24c into next Dec 18, 2025
0 of 2 checks passed
@benlangmuir benlangmuir deleted the eng/blangmuir/filesystem-print-impl-cas branch December 18, 2025 20:55
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