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

Fix sdk install phase #13

Merged
merged 1 commit into from
Aug 14, 2024
Merged

Conversation

urob
Copy link
Contributor

@urob urob commented Aug 14, 2024

zephyr-sdk doesn't contain hosttools-standalone scripts for non-linux platforms, resulting in a build failure on Darwin without the force flag.

The `zephyr-sdk` does not contain a hosttools setup script, failing the
derivation without the force flag.
@urob
Copy link
Contributor Author

urob commented Aug 14, 2024

There is one other issue on Darwin builds, which I am not sure how to best resolve: qemu_full requires ceph, which is only available for linux platforms. I am not sure how important ceph is, maybe the dependency can be overridden to be dropped?

@adisbladis
Copy link
Owner

qemu_full requires ceph, which is only available for linux platforms. I am not sure how important ceph is, maybe the dependency can be overridden to be dropped?

Could we use qemu instead of qemu_full on darwin?

@adisbladis adisbladis merged commit 46eb860 into adisbladis:master Aug 14, 2024
2 checks passed
@urob
Copy link
Contributor Author

urob commented Aug 14, 2024

Could we use qemu instead of qemu_full on darwin?

This should fix the dependency, but I am not sure how it restricts functionality. But perhaps the best course of action in lieu of alternatives?

@adisbladis
Copy link
Owner

We could of course simply override and disable ceph support, but that would mean building qemu from source.

Actually the best fix would be to fix qemu_full in nixpkgs.
A quick fix would be to disable cephSupport on darwin.

If you make a PR cc me please :)

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