Merge pull request #3685 from kpcyrd/dockerfile_prepare_cmd

Dockerfile: Deprecate implicit daemon argument

Dockerfile

@@ -74,3 +74,6 @@ VOLUME $IPFS_PATH
 # 1. There's an fs-repo, and initializes one if there isn't.
 # 2. The API and Gateway are accessible from outside the container.
 ENTRYPOINT ["/usr/local/bin/start_ipfs"]
+
+# Execute the daemon subcommand by default
+CMD ["daemon", "--migrate=true"]

Dockerfile.fast

@@ -53,3 +53,4 @@ RUN cd $SRC_PATH \
 USER ipfs
 VOLUME $IPFS_PATH
 ENTRYPOINT ["/usr/local/bin/start_ipfs"]
+CMD ["daemon", "--migrate=true"]

bin/container_daemon

@@ -19,4 +19,22 @@ else
   ipfs config Addresses.Gateway /ip4/0.0.0.0/tcp/8080
 fi
 
+# if the first argument is daemon
+if [ "$1" = "daemon" ]; then
+  # filter the first argument until
+  # https://github.com/ipfs/go-ipfs/pull/3573
+  # has been resolved
+  shift
+else
+  # print deprecation warning
+  # go-ipfs used to hardcode "ipfs daemon" in it's entrypoint
+  # this workaround supports the new syntax so people start setting daemon explicitly
+  # when overwriting CMD, making this PR safe to merge
+  echo "DEPRECATED: arguments have been set but the first argument isn't 'daemon'" >&2
+  echo "DEPRECATED: run 'docker run ipfs/go-ipfs daemon $@' instead" >&2
+  echo "DEPRECATED: see the following PRs for more information:" >&2
+  echo "DEPRECATED: * https://github.com/ipfs/go-ipfs/pull/3573" >&2
+  echo "DEPRECATED: * https://github.com/ipfs/go-ipfs/pull/3685" >&2
+fi
+
 exec ipfs daemon "$@"