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

Disable debug information on CI #6356

Merged
merged 1 commit into from
May 9, 2023

Conversation

alexcrichton
Copy link
Member

In #6338 that PR is bouncing on CI due to Windows running out of disk space. Locally a ./ci/run-tests.sh compile produces a ~13G build directory. Testing on CI it looks like Windows builders have ~13G of free space on them for GitHub Actions. I think something in that PR has tipped us just over the edge of requiring too much space, although I'm not sure exactly what.

To address the issue this commit disables DWARF debug information entirely on all builders on CI. Not only should this save a sliver of compile time but this reduces a local build directory to ~4.7G, over a 50% reduction. That should keep us fitting in CI for more time to come hopefully.

In bytecodealliance#6338 that PR is bouncing on CI due to Windows running out of disk
space. Locally a `./ci/run-tests.sh` compile produces a ~13G build
directory. Testing on CI it looks like Windows builders have ~13G of
free space on them for GitHub Actions. I think something in that PR has
tipped us just over the edge of requiring too much space, although I'm
not sure exactly what.

To address the issue this commit disables DWARF debug information
entirely on all builders on CI. Not only should this save a sliver of
compile time but this reduces a local build directory to ~4.7G, over a
50% reduction. That should keep us fitting in CI for more time to come
hopefully.
@alexcrichton alexcrichton requested a review from a team as a code owner May 8, 2023 22:06
@alexcrichton alexcrichton requested review from cfallin and removed request for a team May 8, 2023 22:06
@alexcrichton alexcrichton added this pull request to the merge queue May 9, 2023
Merged via the queue into bytecodealliance:main with commit 0e27887 May 9, 2023
@alexcrichton alexcrichton deleted the no-debug-ci branch May 9, 2023 14:46
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.

2 participants