Skip to content

scripts: pre update sof docker for cmake covert - #852

Merged
lgirdwood merged 1 commit into
thesofproject:masterfrom
xiulipan:pr/cmakedockerupdate
Jan 22, 2019
Merged

lgirdwood merged 1 commit into
thesofproject:masterfrom
xiulipan:pr/cmakedockerupdate

Conversation

@xiulipan

Copy link
Copy Markdown
Contributor

Update the sof docker to support cmake to pre test cmake build system
change.

Signed-off-by: Pan Xiuli xiuli.pan@linux.intel.com

pre test for #845

Update the sof docker to support cmake to pre test cmake build system
change.

Signed-off-by: Pan Xiuli <xiuli.pan@linux.intel.com>

@jajanusz jajanusz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good, we can merge it after #845 is merged.

@jajanusz jajanusz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I guess it's needed for this CI: http://otc-sof-ci.sh.intel.com:8080/job/sof_prs/536/consoleText

If so we need to merge it asap so it has CMake.

@xiulipan Am I right?

@lgirdwood

Copy link
Copy Markdown
Member

@xiulipan @jajanusz lets merge this and build PR tomorrow, unless there are any blockers now ?

@jajanusz

Copy link
Copy Markdown
Contributor

@xiulipan @jajanusz lets merge this and build PR tomorrow, unless there are any blockers now ?

There are no blockers for this one. For #845 just need CI team to fix something.
Anyway this PR won't break anything cos it just installs cmake and python.

@xiulipan

Copy link
Copy Markdown
Contributor Author

@jajanusz @lgirdwood
let me check with sof-ci build log and give some feedback later.

@xiulipan

Copy link
Copy Markdown
Contributor Author

@lgirdwood Please merge this PR as I have already update the docker image with the scripts in this PR. And it works fine with both old build system and new system

@xiulipan xiulipan changed the title [DO NOT MERGE]scripts: pre update sof docker for cmake covert scripts: pre update sof docker for cmake covert Jan 22, 2019
@lgirdwood
lgirdwood merged commit 36e1139 into thesofproject:master Jan 22, 2019
@lgirdwood

Copy link
Copy Markdown
Member

@jajanusz @xiulipan applied with false positive CI warning

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