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

xtensa-build-zephyr.py: add flexibility to generate tools/ only #8916

Merged
merged 1 commit into from
Apr 22, 2024

Conversation

marc-hb
Copy link
Collaborator

@marc-hb marc-hb commented Mar 7, 2024

Installing most of the tools/ directory does not technically require building any platform, so add that possibility.

This is useful because the Jenkins-based CI builds the (userspace) tools separately. So far it has been picking hardcoded source paths but of course the HWMv2 transition just broke that:

While it's too late this time (we want to keep CI able to test stable branches for some time), this commit prepares a future where all CIs can stop hardcoding Zephyr source paths and pick the output of the xtensa-build-zephyr.py installer and indirection layer instead.

@marc-hb marc-hb marked this pull request as ready for review March 7, 2024 05:57
@marc-hb
Copy link
Collaborator Author

marc-hb commented Mar 7, 2024

@wszypelt
Copy link

wszypelt commented Mar 8, 2024

@marc-hb hmmm problem when creating compilation and empty logs ;/(I need to analyze it) I have already added this PR to the queue. I think there will be new results within an hour or two

@marc-hb
Copy link
Collaborator Author

marc-hb commented Mar 8, 2024

I have already added this PR to the queue. I think there will be new results within an hour or two

https://sof-ci.01.org/sof-pr-viewer/#/build/PR8916/build13691475 is reported green but when looking at the internal site I see some red in a mostly green and I discovered that... no tests were run at all

Triggered build: [SOF_FW_null_01.01.00.6961] build/13691711

I'm going to refresh the commit date and force-push to re-run.

@marc-hb
Copy link
Collaborator Author

marc-hb commented Mar 10, 2024

Installing most of the tools/ directory does not technically require
building any platform, so add that possibility.

This is useful because the Jenkins-based CI builds the (userspace) tools
separately. So far it has been picking hardcoded source paths but of
course the HWMv2 transition just broke that:
thesofproject#8913

While it's too late this time (we want to keep CI able to test stable
branches for some time), this commit prepares a future where all CIs can
stop hardcoding Zephyr source paths and pick the output of the
xtensa-build-zephyr.py installer and indirection layer instead.

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
@marc-hb
Copy link
Collaborator Author

marc-hb commented Mar 16, 2024

New, much better results after a lot of Zephyr water under the bridge.

ACE https://sof-ci.01.org/sofpr/PR8916/build3487/devicetest/index.html has only one known and unrelated pause/resume failure

https://sof-ci.01.org/sofpr/PR8916/build3488/devicetest/index.html is 100% green.

Everything else including https://sof-ci.01.org/sof-pr-viewer/#/build/PR8916/build13728357 is green.

@kv2019i kv2019i merged commit 665817f into thesofproject:main Apr 22, 2024
42 of 43 checks passed
@marc-hb marc-hb deleted the zephyr-tools-only branch July 2, 2024 19:25
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.

4 participants