Skip to content

save reproducer script - #298

Merged
bhcopeland merged 2 commits into
kernelci:masterfrom
roxell:reproducer-script
Aug 6, 2026
Merged

save reproducer script#298
bhcopeland merged 2 commits into
kernelci:masterfrom
roxell:reproducer-script

Conversation

@roxell

@roxell roxell commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

The reproducer command line is in metadata.json and at the top of
build.log, but it cannot be run from there.

Write it to reproducer.sh in the output directory. The file is
executable, so it runs directly. It is written before the build starts.

@roxell
roxell requested a review from bhcopeland August 5, 2026 13:15
Comment thread tuxmake/build.py
roxell added 2 commits August 6, 2026 12:09
We set KCFLAGS to -ffile-prefix-map=<build dir>/= to keep the build
path out of __FILE__. The build dir is local, and the reproducer does
not pass --build-dir. So the next build gets a different build dir but
the old path in KCFLAGS. The prefix never matches, and __FILE__ keeps
the full path. In the kernel that string shows up in WARN and BUG
messages, so the build is not bit by bit identical.

Leave it out and let the next build set its own value. Keep KCFLAGS if
the user asked for it with --environment.

Signed-off-by: Anders Roxell <anders.roxell@linaro.org>
The reproducer command line is already in metadata.json and in the top
of build.log. Neither can be run. The line has to be copied out of the
log first.

Write it to reproducer.sh instead. The file is executable, so it runs
directly. It is written before the build starts, so it is there also
when the build fails.

Signed-off-by: Anders Roxell <anders.roxell@linaro.org>
@roxell
roxell force-pushed the reproducer-script branch from d42db75 to 3c2c500 Compare August 6, 2026 10:09
@bhcopeland
bhcopeland merged commit 4b3407f into kernelci:master Aug 6, 2026
29 checks passed
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