[commit: ghc] master: Relocatable GHC (bb11a2d)
git at git.haskell.org
git at git.haskell.org
Sat Nov 18 03:13:00 UTC 2017
Repository : ssh://git@git.haskell.org/ghc
On branch : master
Link : http://ghc.haskell.org/trac/ghc/changeset/bb11a2d976b933ad870a2888d665d98385dc55fd/ghc
>---------------------------------------------------------------
commit bb11a2d976b933ad870a2888d665d98385dc55fd
Author: Moritz Angermann <moritz.angermann at gmail.com>
Date: Fri Nov 17 15:04:50 2017 +0800
Relocatable GHC
GHC and the binary distribution that's produced is
not relocatable outside of Windows. This diff tries to
address this for at least Linux and macOS.
Reviewers: austin, hvr, bgamari, erikd, goldfire, Phyx
Reviewed By: bgamari
Subscribers: duog, rwbarton, thomie, erikd
Differential Revision: https://phabricator.haskell.org/D4121
>---------------------------------------------------------------
bb11a2d976b933ad870a2888d665d98385dc55fd
compiler/main/SysTools.hs | 21 +++++++++++++++++++++
utils/ghc-pkg/Main.hs | 11 +++++++++++
2 files changed, 32 insertions(+)
diff --git a/compiler/main/SysTools.hs b/compiler/main/SysTools.hs
index 21ed03b..860b724 100644
--- a/compiler/main/SysTools.hs
+++ b/compiler/main/SysTools.hs
@@ -48,6 +48,7 @@ import Platform
import Util
import DynFlags
+import System.Environment (getExecutablePath)
import System.FilePath
import System.IO
import System.Directory
@@ -530,6 +531,26 @@ type GetFinalPath = HANDLE -> LPTSTR -> DWORD -> DWORD -> IO DWORD
foreign import WINDOWS_CCONV unsafe "dynamic"
makeGetFinalPathNameByHandle :: FunPtr GetFinalPath -> GetFinalPath
+#elif defined(darwin_HOST_OS) || defined(linux_HOST_OS)
+-- on unix, this is a bit more confusing.
+-- The layout right now is somehting like
+--
+-- /bin/ghc-X.Y.Z <- wrapper script (1)
+-- /bin/ghc <- symlink to wrapper script (2)
+-- /lib/ghc-X.Y.Z/bin/ghc <- ghc executable (3)
+-- /lib/ghc-X.Y.Z <- $topdir (4)
+--
+-- As such, we first need to find the absolute location to the
+-- binary.
+--
+-- getExecutablePath will return (3). One takeDirectory will
+-- give use /lib/ghc-X.Y.Z/bin, and another will give us (4).
+--
+-- This of course only works due to the current layout. If
+-- the layout is changed, such that we have ghc-X.Y.Z/{bin,lib}
+-- this would need to be changed accordingly.
+--
+getBaseDir = Just . (\p -> p </> "lib") . takeDirectory . takeDirectory <$> getExecutablePath
#else
getBaseDir = return Nothing
#endif
diff --git a/utils/ghc-pkg/Main.hs b/utils/ghc-pkg/Main.hs
index 6420dd4..3dafffd 100644
--- a/utils/ghc-pkg/Main.hs
+++ b/utils/ghc-pkg/Main.hs
@@ -2116,6 +2116,17 @@ getExecPath = try_size 2048 -- plenty, PATH_MAX is 512 under Win32.
foreign import WINDOWS_CCONV unsafe "windows.h GetModuleFileNameW"
c_GetModuleFileName :: Ptr () -> CWString -> Word32 -> IO Word32
+#elfi defined(darwin_HOST_OS) || defined(linux_HOST_OS)
+-- TODO: a) this is copy-pasta from SysTools.hs / getBaseDir. Why can't we reuse
+-- this here? and parameterise getBaseDir over the executable (for
+-- windows)?
+-- Answer: we can not, because if we share `getBaseDir` via `ghc-boot`,
+-- that would add `base` as a dependency for windows.
+-- b) why is the windows getBaseDir logic, not part of getExecutablePath?
+-- it would be much wider available then and we could drop all the
+-- custom logic?
+-- Answer: yes this should happen. No one has found the time just yet.
+getLibDir = Just . (\p -> p </> "lib") . takeDirectory . takeDirectory <$> getExecutablePath
#else
getLibDir :: IO (Maybe String)
getLibDir = return Nothing
More information about the ghc-commits
mailing list