-
Notifications
You must be signed in to change notification settings - Fork 308
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
zephyr/cmake: fix llext.uuid growing from the last build #9159
Conversation
Reset the llext.uuid file so it does not start with content from the previous build. Fixes LLEXT incremental builds which apparently never worked (some people like it slow?) Fixes commit 05e69e1 ("llext: update to use add_llext_target()") and the ones before that. Signed-off-by: Marc Herbert <marc.herbert@intel.com>
Delete per-platform, "build-sof-staging/sof/intel/sof-ipc4-lib/mtl/" directories every time just like we've always done it for "build-sof-staging/sof/intel/sof-ipc4/mtl/". This is especially important considering the new LLEXT complexity, symbolic links etc. There is no way the build can be smart enough and extend "incremental" builds to the staging directory. Signed-off-by: Marc Herbert <marc.herbert@intel.com>
Even with the cryptographic salt, the ability to match a download with the corresponding build log is still useful. Signed-off-by: Marc Herbert <marc.herbert@intel.com>
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for fixing my crude file-removal code ;-)
Pending pass from mandatory CI jobs, otherwise good to go. |
@wszypelt , @lrudyX can you please re-run https://sof-ci.01.org/sof-pr-viewer/#/build/PR9159/build13991832 where LNL failed with unrelated "connection refused"? This PR should not affect these tests in any way. |
@marc-hb all green :) |
3 small LLEXT fixes. Main one:
Reset the llext.uuid file so it does not start with content from the
previous build.
Fixes incremental builds.