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

wip: build-env: add a simplified build container #31

Closed

Conversation

egernst
Copy link

@egernst egernst commented May 7, 2020

Adding a simplified container for just building Zephyr. To facilitate this,
created subdirectories for a test-env (original container) and build-env (new
container).

As a next step, the original container should be use the build-env container as
its base image. Putting this up for initial feedback before investing more time on this second part.

Signed-off-by: Eric Ernst [email protected]

Adding a simplified container for *just* building Zephyr. To facilitate this,
created subdirectories for a test-env (original container) and build-env (new
container).

As a next step, the original container should be use the build-env container as
its base image.

Signed-off-by: Eric Ernst <[email protected]>
@egernst
Copy link
Author

egernst commented May 7, 2020

This PR is addresses #30

I'm concerned with the size of the resulting image (4.7 GB); will look into this.

@egernst
Copy link
Author

egernst commented May 7, 2020

/cc @nashif

DEBIAN_FRONTEND=noniteractive apt-get install -y --no-install-recommends \
ca-certificates \
ccache \
cmake \
Copy link

Choose a reason for hiding this comment

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

why install cmake here when you later on get it directly from kitware?

Copy link
Author

Choose a reason for hiding this comment

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

IIRC, I was scaling back the already existing tooling which may have had it?

In any event, I wasn't aware what's installed from kitware.

@nashif nashif force-pushed the master branch 13 times, most recently from b370af9 to 298a865 Compare December 15, 2020 18:39
@nashif
Copy link
Member

nashif commented Feb 28, 2021

stale

@nashif nashif closed this Feb 28, 2021
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