From 8aae45a455a5dfcc3e794ef6dc1154310c6d7e05 Mon Sep 17 00:00:00 2001 From: Barry Gordon Date: Fri, 10 Sep 2021 16:58:06 +0100 Subject: [PATCH] Prefer checking user over touching a file --- Dockerfile.development | 1 - bin/dry-run.rb | 3 ++- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/Dockerfile.development b/Dockerfile.development index f9a02f850a..a1ba2d0413 100644 --- a/Dockerfile.development +++ b/Dockerfile.development @@ -2,7 +2,6 @@ FROM dependabot/dependabot-core # Temporarily switch to root user in order to install packages USER root -RUN touch /etc/dependabot-core-dev RUN apt-get update \ && apt-get install -y vim strace ltrace gdb shellcheck \ && rm -rf /var/lib/apt/lists/* diff --git a/bin/dry-run.rb b/bin/dry-run.rb index 4783ab7c03..eb4efb8b2d 100755 --- a/bin/dry-run.rb +++ b/bin/dry-run.rb @@ -35,7 +35,8 @@ # rubocop:disable Style/GlobalVars -unless File.exist?("/etc/dependabot-core-dev") +require "etc" +unless Etc.getpwuid(Process.uid).name == "dependabot" puts <<~INFO bin/dry-run.rb is only supported in a developerment container.