Add a better Docker build script + update Dockerfile

Dockerfile

@@ -1,28 +1,29 @@
-FROM debian:jessie
-MAINTAINER Erik Dasque <[email protected]>
+FROM debian
 
-RUN apt-get update && apt-get install --no-install-recommends -y build-essential \
-    gcc \
-    unzip \
-    wget \
-    zip \
-    gcc-avr \
-    binutils-avr \
+RUN apt-get update && apt-get install --no-install-recommends -y \
     avr-libc \
+    avrdude \
+    binutils-arm-none-eabi \
+    binutils-avr \
+    build-essential \
     dfu-programmer \
     dfu-util \
+    gcc \
     gcc-arm-none-eabi \
-    binutils-arm-none-eabi \
-    libnewlib-arm-none-eabi \
+    gcc-avr \
     git \
+    libnewlib-arm-none-eabi \
     software-properties-common \
-    avrdude \
+    unzip \
+    wget \
+    zip \
     && rm -rf /var/lib/apt/lists/*
 
-ENV keyboard=ergodox
-ENV subproject=ez
-ENV keymap=default
+ENV KEYBOARD=ergodox_ez
+ENV KEYMAP=default
+
+VOLUME /qmk_firmware
+WORKDIR /qmk_firmware
+COPY . .
 
-VOLUME /qmk
-WORKDIR /qmk
-CMD make clean ; make keyboard=${keyboard} subproject=${subproject} keymap=${keymap}
+CMD make $KEYBOARD:$KEYMAP

docs/getting_started_build_tools.md

@@ -127,16 +127,15 @@ If this is a bit complex for you, Docker might be the turn-key solution you need
 ```bash
 # You'll run this every time you want to build a keymap
 # modify the keymap and keyboard assignment to compile what you want
-# defaults are ergodox/default
+# defaults are ergodox_ez:default
 
-docker run -e keymap=gwen -e keyboard=ergodox_ez --rm -v $('pwd'):/qmk:rw edasque/qmk_firmware
+docker run --rm -e KEYBOARD=planck/rev6 -e KEYMAP=default -v $('pwd'):/qmk:rw qmkfm/qmk_firmware
 ```
 
 On Windows Docker seems to have issues with the VOLUME tag in Dockerfile, and `$('pwd')` won't print a Windows compliant path; use full path instead, like this:
 
 ```bash
-docker run -e keymap=default -e keyboard=ergodox_ez --rm -v D:/Users/Sacapuces/Documents/Repositories/qmk:/qmk:rw edasque/qmk_firmware
-
+docker run --rm -e KEYBOARD=planck/rev6 -e KEYMAP=default -v C:/Users/QMK/Projects/qmk:/qmk:rw qmkfm/qmk_firmware
 ```
 
 This will compile the targeted keyboard/keymap and leave it in your QMK directory for you to flash.

util/docker_build.sh

@@ -0,0 +1,43 @@
+#!/bin/bash
+
+USAGE="Usage: $0 [keyboard[:keymap[:target]]]"
+
+# Check preconditions
+if [[ $* =~ "--help" ]]; then
+    echo "$USAGE"
+    exit 0
+elif [[ $# -gt 1 ]]; then
+    echo "$USAGE" >&2
+    exit 1
+elif [[ ! $(command -v docker) ]]; then
+    echo "Error: docker not found" >&2
+    echo "See https://docs.docker.com/install/#supported-platforms for installation instructions" >&2
+    exit 2
+fi
+
+# Determine arguments
+if [[ $# -eq 0 ]]; then
+    read -rp "keyboard=" keyboard
+    [[ -n $keyboard ]] && read -rp "keymap=" keymap
+    [[ -n $keymap   ]] && read -rp "target=" target
+else
+    IFS=':' read -r keyboard keymap target x <<< "$1"
+    if [[ -n $x ]]; then
+        echo "$USAGE" >&2
+        exit 1
+    fi
+fi
+if [[ -n $target ]]; then
+    if [[ $(uname) == "Linux" || $(docker-machine active 2>/dev/null) ]]; then
+        usb_args="--privileged -v /dev/bus/usb:/dev/bus/usb"
+    else
+        echo "target requires docker-machine to run on your platform" >&2
+        echo "See http://gw.tnode.com/docker/docker-machine-with-usb-support-on-windows-macos" >&2
+        exit 3
+    fi
+fi
+dir=$(pwd -W 2>/dev/null) || dir=$PWD  # Use Windows path if on Windows
+
+# Run container and build firmware
+docker run --rm $usb_args -v "$dir":/qmk_firmware qmkfm/qmk_firmware \
+    make "$keyboard${keymap:+:$keymap}${target:+:$target}"