Merge pull request #26 from blueyed/remove-bash

Remove bash

Author: blueyed

Dockerfile

@@ -1,9 +1,6 @@
 FROM alpine:3.5
 
-RUN apk --update add bash \
-  && rm -rf /var/cache/apk/* /tmp/* /var/tmp/*
-
-RUN adduser -h /home -s /bin/bash -D -u 8465 vimtest
+RUN adduser -h /home -s /bin/sh -D -u 8465 vimtest
 
 RUN mkdir -p /vim /vim-build/bin /plugins
 RUN chown vimtest:vimtest /home /plugins

example/Makefile

@@ -15,5 +15,17 @@ test: test-setup
 	for vim in $$vims; do \
 	  $(DOCKER) $$vim '+Vader! test/*'; \
 	done
+	out=$$(docker run --rm "$(IMAGE)" /vim-build/bin/argecho "arg1" "arg 2"); \
+	for line in "PWD=/" "Running as: root" "Arguments:" "arg1" "arg 2"; do \
+	  if ! echo "$$out" | grep -qFx -e "$$line"; then \
+	    echo "Line not found: $$line"; echo "$$out"; exit 1; \
+	  fi \
+	done
+	out=$$(docker run --rm "$(IMAGE)" argecho "arg1" "arg 2"); \
+	for line in "Running as: vimtest" "PWD=/testplugin" "Arguments:" "-u" "/home/vimrc" "-i" "NONE" "arg1" "arg 2"; do \
+	  if ! echo "$$out" | grep -qFx -e "$$line"; then \
+	    echo "Line not found: $$line"; echo "$$out"; exit 1; \
+	  fi \
+	done
 
 .PHONY: test-setup test

scripts/argecho.sh

@@ -1,4 +1,4 @@
-#!/bin/bash
+#!/bin/sh
 
 echo "Running as: $(whoami)"
 
@@ -9,6 +9,6 @@ env
 echo
 echo "Arguments:"
 while [ $# -gt 0 ]; do
-  echo $1
+  echo "$1"
   shift
 done

scripts/install_vim.sh

@@ -1,4 +1,4 @@
-#!/bin/bash
+#!/bin/sh
 
 set -e
 
@@ -20,6 +20,8 @@ build() {
   [ -z $NAME ] && NAME="vim-${TAG}"
   [ -z $TAG ] && bail "-tag is required"
 
+  apk add ncurses
+
   VIM_NAME="vim_${TAG}_py${PYTHON}_rb${RUBY}_lua${LUA}"
   VIM_PATH="/vim-build/$VIM_NAME"
   VIM_BIN="$VIM_PATH/bin/vim"

scripts/run_vim.sh

@@ -1,10 +1,10 @@
-#!/bin/bash
+#!/bin/sh
 
 BIN=$1
 shift
 
-if [ "$BIN" == "bash" ] || [ -z "$BIN" ]; then
-  exec /bin/bash
+if [ "$BIN" = "sh" ] || [ -z "$BIN" ]; then
+  exec /bin/sh
 fi
 if ! [ -x "/vim-build/bin/$BIN" ]; then
   exec "$BIN" "$@"