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

Use CMake to build rimage #3

Merged
merged 3 commits into from
Mar 26, 2020
Merged

Use CMake to build rimage #3

merged 3 commits into from
Mar 26, 2020

Conversation

dcpleung
Copy link
Contributor

This adds the necessary bits to allow building rimage with CMake with similar configure options as the just removed autotools files.

Also update the rimage source from SOF master (@ 4e87899f25b0c859169c1dfd4dd9f9fd65d48ad2).

Update source files from main SOF repository
@ commit 4e87899f25b0c859169c1dfd4dd9f9fd65d48ad2.

Signed-off-by: Daniel Leung <[email protected]>
CMake is to be used instead of autotools. So remove the autotools
files.

This revert commit f6bf4b0 and
commit 0d33e87

Signed-off-by: Daniel Leung <[email protected]>
This adds the necessary bits to allow building rimage
with CMake with similar configure options as the just
removed autotools files.

Also update README.md for new build instructions.

Signed-off-by: Daniel Leung <[email protected]>
Copy link
Member

@lgirdwood lgirdwood left a comment

Choose a reason for hiding this comment

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

@dcpleung thank you for doing this.

@lgirdwood
Copy link
Member

@jajanusz good for you ?

Copy link
Contributor

@jajanusz jajanusz left a comment

Choose a reason for hiding this comment

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

Looks ok, thanks

@lgirdwood lgirdwood merged commit a264697 into thesofproject:master Mar 26, 2020
@dcpleung dcpleung deleted the standalone_rimage branch March 26, 2020 12:37
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.

3 participants