Skip to content

Conversation

@ahoppen
Copy link
Member

@ahoppen ahoppen commented May 24, 2024

The signposts aren’t easily visible in the log (you need to add --signpost to log show) and don’t get logged on non-Darwin platforms at all. Add logging for it.

@ahoppen ahoppen requested review from bnbarham and hamishknight May 24, 2024 05:50
@ahoppen ahoppen requested a review from benlangmuir as a code owner May 24, 2024 05:50
@ahoppen ahoppen force-pushed the generating-build-graph-logging branch from 6c74084 to f0be6d6 Compare May 24, 2024 14:46
@ahoppen
Copy link
Member Author

ahoppen commented May 24, 2024

@swift-ci Please test

@ahoppen ahoppen changed the title Log when the build graph starts being generated and when it finishes 🚥 #1342 Log when the build graph starts being generated and when it finishes May 24, 2024
@ahoppen
Copy link
Member Author

ahoppen commented May 24, 2024

@swift-ci Please test

@ahoppen
Copy link
Member Author

ahoppen commented May 24, 2024

@swift-ci Please test Windows

ahoppen added 2 commits May 24, 2024 11:23
…ild graph

Rebuilding the build graph can take a while (initial loading of the build graph takes ~7s for sourcekit-lsp) and it’s good to show some progress during this time.
The signposts aren’t easily visible in the log (you need to add `--signpost` to `log show`) and don’t get logged on non-Darwin platforms at all. Add logging for it.
@ahoppen ahoppen force-pushed the generating-build-graph-logging branch from 6067c88 to dd67a3b Compare May 24, 2024 18:25
@ahoppen
Copy link
Member Author

ahoppen commented May 24, 2024

@swift-ci Please test

@ahoppen
Copy link
Member Author

ahoppen commented May 24, 2024

@swift-ci Please test Windows

1 similar comment
@ahoppen
Copy link
Member Author

ahoppen commented May 24, 2024

@swift-ci Please test Windows

…he SourceKitLSPServer are modified

`IndexProgressManager` iterates over all workspaces in the SourceKitLSPServer. Modifying workspaces might thus update the index progress status.
@ahoppen ahoppen force-pushed the generating-build-graph-logging branch from dd67a3b to 069e8a2 Compare May 25, 2024 00:31
@ahoppen
Copy link
Member Author

ahoppen commented May 25, 2024

@swift-ci Please test

@ahoppen ahoppen merged commit 84007ca into swiftlang:main May 25, 2024
@ahoppen ahoppen deleted the generating-build-graph-logging branch August 4, 2025 10:48
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