Skip to content

[CI] Tee Ninja Output to Log Files #152331

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

Merged

Conversation

boomanaiden154
Copy link
Contributor

This patch makes all of the ninja commands in the monolithic-* scripts
write to log files in the current working directory. The plan is to use
this to feed the ninja log into generate_test_report_github.py so we can
surface compilation errors.

Related to #152246.

Created using spr 1.3.6
boomanaiden154 added a commit to boomanaiden154/llvm-project that referenced this pull request Aug 6, 2025
This patch makes all of the ninja commands in the monolithic-* scripts
write to log files in the current working directory. The plan is to use
this to feed the ninja log into generate_test_report_github.py so we can
surface compilation errors.

Related to llvm#152246.

Pull Request: llvm#152331
Created using spr 1.3.6
@boomanaiden154 boomanaiden154 merged commit 536e414 into main Aug 7, 2025
10 checks passed
@boomanaiden154 boomanaiden154 deleted the users/boomanaiden154/ci-tee-ninja-output-to-log-files branch August 7, 2025 21:02
llvm-sync bot pushed a commit to arm/arm-toolchain that referenced this pull request Aug 7, 2025
This patch makes all of the ninja commands in the monolithic-* scripts
write to log files in the current working directory. The plan is to use
this to feed the ninja log into generate_test_report_github.py so we can
surface compilation errors.

Related to #152246.

Reviewers: Keenuts, lnihlen, cmtice, dschuff, gburgessiv

Reviewed By: Keenuts, cmtice

Pull Request: llvm/llvm-project#152331
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