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.

Sign up for GitHub

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

Jump to bottom

Fix docker build #329

Merged

conorpp merged 1 commit into solokeys:master from ccinelli:fix-docker

Oct 27, 2019

ccinelli commented Oct 24, 2019 •

edited

Loading

Currently the Makefile script expects arm-none-eabi-size to be in the PATH.
The docker container does not have it.

$(SZ) is alias that already exist with proper path.

So $(SZ) should be used instead of arm-none-eabi-size.


          Fix docker build

9990b03

ccinelli force-pushed the fix-docker branch from cfd4045 to 9990b03 Compare

October 24, 2019 08:15

ccinelli mentioned this pull request

Split docker makefile in toolchain build and firmware build #330

Merged

Member

conorpp commented Oct 27, 2019

Thanks!

conorpp merged commit 765dc27 into solokeys:master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet