[Git][ghc/ghc][wip/bindist-install] 3 commits: hadrian: Fix bindist installation on Darwin

[Ben Gamari (@bgamari)]

Ben Gamari pushed to branch wip/bindist-install at Glasgow Haskell Compiler / GHC


Commits:
4e8be0b4 by Ben Gamari at 2022-08-08T10:26:09-04:00
hadrian: Fix bindist installation on Darwin

It turns out that `cp -P` on Darwin does not always copy a symlink as
a symlink. In order to get these semantics one must pass `-RP`. It's not
entirely clear whether this is valid under POSIX, but it is nevertheless
what Apple does.

- - - - -
24e14ff8 by Ben Gamari at 2022-08-08T10:26:09-04:00
hadrian: Drop diagnostics output from bindist installation

- - - - -
db6501e9 by Ben Gamari at 2022-08-08T10:26:09-04:00
gitlab-ci: Don't use coreutils on Darwin

In general we want to ensure that the tested environment is as similar
as possible to the environment the user will use. In the case of Darwin,
this means we want to use the system's BSD command-line utilities, not
coreutils.

This would have caught #21974.

- - - - -


2 changed files:

- .gitlab/darwin/toolchain.nix
- hadrian/bindist/Makefile


Changes:

=====================================
.gitlab/darwin/toolchain.nix
=====================================
@@ -85,7 +85,6 @@ pkgs.writeTextFile {
     export PATH
     PATH="${pkgs.autoconf}/bin:$PATH"
     PATH="${pkgs.automake}/bin:$PATH"
-    PATH="${pkgs.coreutils}/bin:$PATH"
     export FONTCONFIG_FILE=${fonts}
     export XELATEX="${ourtexlive}/bin/xelatex"
     export MAKEINDEX="${ourtexlive}/bin/makeindex"


=====================================
hadrian/bindist/Makefile
=====================================
@@ -39,11 +39,10 @@ endif
 # of program names. For hadrian build this will work as programs have a
 # consistent naming procedure.
 define installscript
-	echo "installscript $1 -> $2"
 	@if [ -L 'wrappers/$1' ]; then                \
-		$(CP) -P 'wrappers/$1' '$2' ;             \
-	else								          \
-		rm -f '$2' && 		                      \
+		$(CP) -RP 'wrappers/$1' '$2' ;            \
+    else                                          \
+        rm -f '$2' &&                             \
 		$(CREATE_SCRIPT) '$2' &&                  \
 		echo "#!$(SHELL)" >>  '$2'  &&            \
 		echo "exedir=\"$4\"" >> '$2'  &&          \
@@ -57,7 +56,7 @@ define installscript
 		cat 'wrappers/$1' >> '$2'  &&             \
 		$(EXECUTABLE_FILE) '$2' ;                 \
 	fi
-	@echo "$1 installed to $2"
+	@echo "$1 installed to $2";
 endef
 
 # patchpackageconf



View it on GitLab: https://gitlab.haskell.org/ghc/ghc/-/compare/e47fb9b0932848b5cb986b6274756e762db165a1...db6501e91553278ff80889fedf17dff1f2b6c957

-- 
View it on GitLab: https://gitlab.haskell.org/ghc/ghc/-/compare/e47fb9b0932848b5cb986b6274756e762db165a1...db6501e91553278ff80889fedf17dff1f2b6c957
You're receiving this email because of your account on gitlab.haskell.org.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.haskell.org/pipermail/ghc-commits/attachments/20220808/da05d9ee/attachment-0001.html>