hadrian: use new-exec to make sure alex & happy are in PATH (#16120)
authorAdam Sandberg Eriksson <adam@sandbergericsson.se>
Wed, 2 Jan 2019 21:08:49 +0000 (22:08 +0100)
committerBen Gamari <ben@smart-cactus.org>
Sun, 27 Jan 2019 13:32:12 +0000 (08:32 -0500)
hadrian/build.cabal.bat
hadrian/build.cabal.sh
hadrian/src/Environment.hs

index 96f2725..3344791 100644 (file)
@@ -31,7 +31,8 @@ if %CABMAJOR% equ 2 (
 )
 if %_cabal_ok% equ 1 (
     "%CABAL%" --project-file=%PROJ% new-build %CABFLAGS% -j exe:hadrian
-    "%CABAL%" --project-file=%PROJ% new-run   %CABFLAGS%    exe:hadrian -- ^
+    rem use new-exec instead of new-run to make sure that the build-tools (alex & happy) are in PATH
+    "%CABAL%" --project-file=%PROJ% new-exec  %CABFLAGS%    hadrian -- ^
         --directory "%CD%" ^
         %*
 ) else (
index 8c7b594..f28c3e0 100755 (executable)
@@ -24,7 +24,8 @@ CABVER=( ${CABVERSTR//./ } )
 if [ "${CABVER[0]}" -gt 2 -o "${CABVER[0]}" -eq 2 -a "${CABVER[1]}" -ge 2 ];
 then
     "$CABAL" --project-file="$PROJ" new-build $CABFLAGS -j exe:hadrian
-    "$CABAL" --project-file="$PROJ" new-run   $CABFLAGS    exe:hadrian -- \
+    # use new-exec instead of new-run to make sure that the build-tools (alex & happy) are in PATH
+    "$CABAL" --project-file="$PROJ" new-exec  $CABFLAGS    hadrian -- \
         --directory "$PWD" \
         "$@"
 else
index 1666c68..ef98bb1 100644 (file)
@@ -10,6 +10,10 @@ setupEnvironment = do
     -- Cabal refuses to work when GHC_PACKAGE_PATH is set (e.g. by Stack)
     unsetEnv "GHC_PACKAGE_PATH"
 
+    -- cabal new-exec sets GHC_ENVIRONMENT, it needs to be unset for GHC
+    -- invocations to work properly
+    unsetEnv "GHC_ENVIRONMENT"
+
     -- in MinGW if PWD is set to a Windows "C:\\" style path then configure
     -- `pwd` will return the Windows path, and then modifying $PATH will fail.
     -- See https://github.com/snowleopard/hadrian/issues/189 for details.