From e1e6dba31b34d2094d5b9fd8cc12781e5145e110 Mon Sep 17 00:00:00 2001 From: antonc42 Date: Fri, 4 Jan 2019 09:56:16 -0600 Subject: [PATCH] changed env var in makefile to builtin var makefile was using the environment variable PWD. this can be problematic if using 'sudo' as the PWD variable is not passed through. however GNU make has a builtin CURDIR variable for the current working directory that does work with 'sudo'. --- Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Makefile b/Makefile index ff84404..ff5000e 100644 --- a/Makefile +++ b/Makefile @@ -26,7 +26,7 @@ databases := GeoLite2-City GeoLite2-Country $(databases): mkdir -p data - curl -fsSL -m 30 https://geolite.maxmind.com/download/geoip/database/$@.tar.gz | tar $(TAR_OPTS) --strip-components=1 -C $(PWD)/data -xzf - '*.mmdb' + curl -fsSL -m 30 https://geolite.maxmind.com/download/geoip/database/$@.tar.gz | tar $(TAR_OPTS) --strip-components=1 -C $(CURDIR)/data -xzf - '*.mmdb' test ! -f data/GeoLite2-City.mmdb || mv data/GeoLite2-City.mmdb data/city.mmdb test ! -f data/GeoLite2-Country.mmdb || mv data/GeoLite2-Country.mmdb data/country.mmdb