Skip to content

CI: Use compile_like_murdock instead of buildtest#59

Merged
chrysn merged 1 commit intomainfrom
test-without-buildtest
Jun 25, 2025
Merged

CI: Use compile_like_murdock instead of buildtest#59
chrysn merged 1 commit intomainfrom
test-without-buildtest

Conversation

@chrysn
Copy link
Member

@chrysn chrysn commented Jun 25, 2025

See-Also: RIOT-OS/RIOT#21359

cargo update -p riot-sys -p riot-wrappers --aggressive
cargo tree
make buildtest BUILDTEST_MAKE_REDIRECT=''
./dist/tools/compile_test/compile_like_murdock.py -b $BOARDS
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
./dist/tools/compile_test/compile_like_murdock.py -b $BOARDS
./dist/tools/compile_test/compile_like_murdock.py -b $BOARDS -j16

The Python script does not set the thread count by default. 16 Threads is what the Docker workflow uses too.

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks; updating that. (Gotta update a bit more b/c this used to build example-by-example to enhance visibility of bugs, and now I have to move compile_like_murdock out of the loop -- given how much runway it needs to get started, that's probably a good thing anyway).

Copy link
Contributor

Choose a reason for hiding this comment

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

You can still build it in the loop:

for D in ${DIRS}; do
          cd ${D}
          echo "::group::Building ${D}"
          cargo update -p riot-sys -p riot-wrappers --aggressive
          cargo tree
+          cd -
+          ./dist/tools/compile_test/compile_like_murdock.py -b $BOARDS -a ${D} -j16
-          ./dist/tools/compile_test/compile_like_murdock.py -b $BOARDS -j16
-          cd -
          echo "::endgroup::"
        done

Just go back to the root directory before building and specify the directory in the -a application command.

Copy link
Member Author

Choose a reason for hiding this comment

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

Nice trick swapping things around, I'd have started storing PWD in $RIOT and things would be more messy.

But probably the more I let the tool do its job, the more things will just work -- and this way, the threads get used well. I guess that the old buildtest would have shown the errors, so I'd have to go an extra mile to make this one show the right output. Instead, I'll leave that for future me who will automatically have provided a test case for "build fails" and can work from there :-)

@chrysn chrysn force-pushed the test-without-buildtest branch from c8beb8d to a7c9877 Compare June 25, 2025 13:09
@chrysn chrysn enabled auto-merge June 25, 2025 13:36
@chrysn chrysn merged commit 605f20c into main Jun 25, 2025
5 of 7 checks passed
@chrysn chrysn deleted the test-without-buildtest branch June 25, 2025 13:40
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