Compiling with Docker

[marwonline]

Today I wanted to compile the Kernel on a fresh Archlinux and realized that installing all those cross-compiler packages may take a time. So I hacked a small Dockerfile for me, which I'll share with you:

Dockerfile:

FROM ubuntu:xenial

RUN useradd -ms /bin/bash builder

RUN apt-get update               \
 && apt-get -y -q upgrade        \
 && apt-get -y -q install        \
    bc                           \
    binutils-arm-linux-gnueabihf \
    bison                        \
    build-essential              \
    ccache                       \
    flex                         \
    gcc-arm-linux-gnueabihf      \
    gccgo-4.7-arm-linux-gnueabi  \
    gcc-aarch64-linux-gnu        \
    git                          \
    libc6-armhf-cross            \
    libncurses-dev               \
    libssl-dev                   \
    make                         \
    u-boot-tools                 \
    wget                         \
    xz-utils                     \
 && apt-get clean

USER builder
WORKDIR /srv/

Create the local image with: docker build . --tag bpi-compile:1

Run the compile environment (adjust the path before the colon):
docker run -it -v $(pwd)/SD:/SD -v $(pwd)/BPI-R2-4.14:/srv bpi-compile:1 /bin/bash

Is it worth to create a pull request for this?

[frank-w]

nice idea to create a docker container...isn't there a bionic base?

it seems you create the docker-file 1 level above the git repo (and mounting this to /srv), right?

need to setup a docker environment to test it...

[marwonline]

Yes it one level outside of the git repo, but you can adjust the paths easily which are behind the -v flag.

Upgrading to bionic should be rather easy and also worth to try 👍

[frank-w]

imho the docker-file should be inside utils-directory (folder "docker" or similar) with a script to initialize it (your additional commands).

if you make a patch (git format-patch) i can apply it directly...should be easier as a pull request

imho gccgo-4.7-arm-linux-gnueabi and binutils-arm-linux-gnueabihf are not needed (or automaticly installed)

[marwonline]

I changed the stuff you mentioned and create the patch:

• bionic
• moved to /docker folder
• removed unused packages from apt

Download the patch

[marwonline]

I just noticed that the docker daemon will create the SD folder if not present with root rights. I changed this in this patch:
adding-docker-env.patch.txt

[frank-w]

can you move docker folder to utils/ (utils/docker/...)? and squash the 3 Patches?

you can include the fixes/upgrade in single Patch instead of creating old value and than change it

default should be running docker with user-rights (user in group docker) also inside container build.sh should not be run as root

[frank-w]

merged your patch, but please make a patch to move docker folder into utils/

and some usage-instructions will be nice ;) as me and some other are not experienced with docker

[marwonline]

As wished, I ...

• moved the Docker stuff into ./utils and
• documented the usage in the README.md

moved_Docker_tools_to___utils_and_added_some_documentation_for_it.patch.txt

[frank-w]

seems patchfile is broken

[16:08:29]$ patch -p1 --dry-run < ~/Downloads/moved_Docker_tools_to___utils_and_added_some_documentation_for_it.patch
checking file docker/build_container.sh
patch: **** malformed patch at line 5: --- 2,1 ----

can you please make a patch with git-format-patch?

[frank-w]

i have added the changes manually to 4.14-main, can you please test it?

[marwonline]

I tried the version from 4.14-main and everything works fine. 👍

[frank-w]

Ok,added it also to 4.19 and set you as author