@@ -14,25 +14,24 @@ module Obelisk.Command.Run where
14import Control.Arrow ((&&&))
14import Control.Arrow ((&&&))
15import Control.Exception (Exception, bracket)
15import Control.Exception (Exception, bracket)
16import Control.Lens (ifor, (.~), (&))
16import Control.Lens (ifor, (.~), (&))
17−import Control.Monad (filterM)
17+import Control.Monad (filterM, unless, void)
18import Control.Monad.Except (runExceptT, throwError)
18import Control.Monad.Except (runExceptT, throwError)
19import Control.Monad.IO.Class (liftIO)
19import Control.Monad.IO.Class (liftIO)
20import Control.Monad.Reader (MonadIO)
20import Control.Monad.Reader (MonadIO)
21import Data.Bifoldable (bifoldr1)
21import Data.Bifoldable (bifoldr1)
22import Data.Bifunctor (bimap)
22import Data.Bifunctor (bimap)
23+import qualified Data.ByteString.UTF8 as BSU
23import Data.Coerce (coerce)
24import Data.Coerce (coerce)
24import Data.Default (def)
25import Data.Default (def)
25import Data.Either (partitionEithers)
26import Data.Either (partitionEithers)
26import Data.Foldable (fold, for_, toList)
27import Data.Foldable (fold, for_, toList)
27import Data.Functor.Identity (runIdentity)
28import Data.Functor.Identity (runIdentity)
28−import Data.List (intercalate)
29import Data.List.NonEmpty (NonEmpty)
29import Data.List.NonEmpty (NonEmpty)
30import qualified Data.List.NonEmpty as NE
30import qualified Data.List.NonEmpty as NE
31import Data.Map (Map)
31import Data.Map (Map)
32import qualified Data.Map as Map
32import qualified Data.Map as Map
33import qualified Data.Map.Monoidal as MMap
33import qualified Data.Map.Monoidal as MMap
34−import Data.Maybe (catMaybes, fromMaybe, mapMaybe)
34+import Data.Maybe
35−import Data.Ord (comparing)
36import Data.Set (Set)
35import Data.Set (Set)
37import qualified Data.Set as Set
36import qualified Data.Set as Set
38import Data.String.Here.Interpolated (i)
37import Data.String.Here.Interpolated (i)
@@ -46,38 +45,40 @@ import Distribution.Compiler (CompilerFlavor(..))
46import Distribution.PackageDescription.Parsec (parseGenericPackageDescription)
45import Distribution.PackageDescription.Parsec (parseGenericPackageDescription)
47import Distribution.Parsec.ParseResult (runParseResult)
46import Distribution.Parsec.ParseResult (runParseResult)
48import qualified Distribution.System as Dist
47import qualified Distribution.System as Dist
49−import Distribution.Types.BuildInfo (buildable, cppOptions, defaultExtensions, defaultLanguage, hsSourceDirs, options)
48+import Distribution.Types.BuildInfo
50−import Distribution.Types.CondTree (simplifyCondTree)
49+import Distribution.Types.CondTree
51−import Distribution.Types.GenericPackageDescription (ConfVar (Arch, Impl, OS), condLibrary)
50+import Distribution.Types.GenericPackageDescription
52−import Distribution.Types.Library (libBuildInfo)
51+import Distribution.Types.Library
53−import Distribution.Utils.Generic (toUTF8BS, readUTF8File)
52+import Distribution.Utils.Generic
54import qualified Distribution.Parsec.Common as Dist
53import qualified Distribution.Parsec.Common as Dist
55import qualified Hpack.Config as Hpack
54import qualified Hpack.Config as Hpack
56import qualified Hpack.Render as Hpack
55import qualified Hpack.Render as Hpack
57import qualified Hpack.Yaml as Hpack
56import qualified Hpack.Yaml as Hpack
58−import Language.Haskell.Extension (Extension, Language)
57+import Language.Haskell.Extension
59−import qualified Network.Socket as Socket
58+import Network.Socket hiding (Debug)
60import System.Directory
59import System.Directory
61import System.Environment (getExecutablePath)
60import System.Environment (getExecutablePath)
62import System.FilePath
61import System.FilePath
63import qualified System.Info
62import qualified System.Info
64import System.IO.Temp (withSystemTempDirectory)
63import System.IO.Temp (withSystemTempDirectory)
64+import Text.ShellEscape (bash, bytes)
66import Obelisk.App (MonadObelisk)
66import Obelisk.App (MonadObelisk)
67import Obelisk.CliApp (
67import Obelisk.CliApp (
68 Severity (..),
68 Severity (..),
72 readCreateProcessWithExitCode,
73 readCreateProcessWithExitCode,
73 readProcessAndLogStderr,
74 readProcessAndLogStderr,
76 setDelegateCtlc,
76 setDelegateCtlc,
77 withSpinner,
78 withSpinner,
79import Obelisk.Command.Nix
80import Obelisk.Command.Nix
80−import Obelisk.Command.Project (nixShellWithoutPkgs, withProjectRoot, findProjectAssets, bashEscape)
81+import Obelisk.Command.Project (nixShellWithoutPkgs, withProjectRoot, findProjectAssets)
81import Obelisk.Command.Thunk (attrCacheFileName)
82import Obelisk.Command.Thunk (attrCacheFileName)
82import Obelisk.Command.Utils (findExePath, ghcidExePath)
83import Obelisk.Command.Utils (findExePath, ghcidExePath)
@@ -94,11 +95,9 @@ data CabalPackageInfo = CabalPackageInfo
94 -- ^ List of globally set languages of the library component
95 -- ^ List of globally set languages of the library component
95 , _cabalPackageInfo_compilerOptions :: [(CompilerFlavor, [String])]
96 , _cabalPackageInfo_compilerOptions :: [(CompilerFlavor, [String])]
96 -- ^ List of compiler-specific options (e.g., the "ghc-options" field of the cabal file)
97 -- ^ List of compiler-specific options (e.g., the "ghc-options" field of the cabal file)
97− , _cabalPackageInfo_cppOptions :: [String]
98− -- ^ List of CPP (C Preprocessor) options (e.g. the "cpp-options" field of the cabal file)
101−-- | 'Bool' with a better name for its purpose.
100+-- | 'Bool' with a better name for it's purpose.
102data Interpret = Interpret_Interpret | Interpret_NoInterpret deriving (Eq, Ord, Show)
101data Interpret = Interpret_Interpret | Interpret_NoInterpret deriving (Eq, Ord, Show)
104textInterpret :: Interpret -> Text
103textInterpret :: Interpret -> Text
@@ -151,7 +150,7 @@ profile profileBasePattern rtsFlags = withProjectRoot "." $ \root -> do
151 liftIO $ createDirectoryIfMissing True $ takeDirectory $ root </> profileBaseName
150 liftIO $ createDirectoryIfMissing True $ takeDirectory $ root </> profileBaseName
152 putLog Debug $ "Storing profiled data under base name of " <> T.pack (root </> profileBaseName)
151 putLog Debug $ "Storing profiled data under base name of " <> T.pack (root </> profileBaseName)
153 freePort <- getFreePort
152 freePort <- getFreePort
154− runProcess_ $ setCwd (Just root) $ setDelegateCtlc True $ proc (outPath </> "bin" </> "ob-run") $
153+ (_, _, _, ph) <- createProcess_ "runProfExe" $ setCwd (Just root) $ setDelegateCtlc True $ proc (outPath </> "bin" </> "ob-run") $
155 [ show freePort
154 [ show freePort
156 , T.unpack assets
155 , T.unpack assets
157 , profileBaseName
156 , profileBaseName
@@ -159,16 +158,17 @@ profile profileBasePattern rtsFlags = withProjectRoot "." $ \root -> do
159 , "-po" <> profileBaseName
158 , "-po" <> profileBaseName
160 ] <> rtsFlags
159 ] <> rtsFlags
161 <> [ "-RTS" ]
160 <> [ "-RTS" ]
161+ void $ waitForProcess ph
163run :: MonadObelisk m => FilePath -> PathTree Interpret -> m ()
163run :: MonadObelisk m => FilePath -> PathTree Interpret -> m ()
164run root interpretPaths = do
164run root interpretPaths = do
165 pkgs <- getParsedLocalPkgs root interpretPaths
165 pkgs <- getParsedLocalPkgs root interpretPaths
166+ withGhciScript pkgs $ \dotGhciPath -> do
166− assets <- findProjectAssets root
167+ assets <- findProjectAssets root
167− putLog Debug $ "Assets impurely loaded from: " <> assets
168+ putLog Debug $ "Assets impurely loaded from: " <> assets
168− ghciArgs <- getGhciSessionSettings pkgs root True
169+ ghciArgs <- getGhciSessionSettings pkgs root True
169− freePort <- getFreePort
170+ freePort <- getFreePort
170− withGhciScriptArgs pkgs $ \dotGhciArgs -> do
171− runGhcid root True (ghciArgs <> dotGhciArgs) pkgs $ Just $ unwords
171+ runGhcid root True (ghciArgs <> mkGhciScriptArg dotGhciPath) pkgs $ Just $ unwords
172 [ "Obelisk.Run.run"
172 [ "Obelisk.Run.run"
173 , show freePort
173 , show freePort
174 , "(Obelisk.Run.runServeAsset " ++ show assets ++ ")"
174 , "(Obelisk.Run.runServeAsset " ++ show assets ++ ")"
@@ -180,15 +180,15 @@ runRepl :: MonadObelisk m => FilePath -> PathTree Interpret -> m ()
180runRepl root interpretPaths = do
180runRepl root interpretPaths = do
181 pkgs <- getParsedLocalPkgs root interpretPaths
181 pkgs <- getParsedLocalPkgs root interpretPaths
182 ghciArgs <- getGhciSessionSettings pkgs "." True
182 ghciArgs <- getGhciSessionSettings pkgs "." True
183− withGhciScriptArgs pkgs $ \dotGhciArgs ->
183+ withGhciScript pkgs $ \dotGhciPath ->
184− runGhciRepl root pkgs (ghciArgs <> dotGhciArgs)
184+ runGhciRepl root pkgs (ghciArgs <> mkGhciScriptArg dotGhciPath)
186runWatch :: MonadObelisk m => FilePath -> PathTree Interpret -> m ()
186runWatch :: MonadObelisk m => FilePath -> PathTree Interpret -> m ()
187runWatch root interpretPaths = do
187runWatch root interpretPaths = do
188 pkgs <- getParsedLocalPkgs root interpretPaths
188 pkgs <- getParsedLocalPkgs root interpretPaths
189 ghciArgs <- getGhciSessionSettings pkgs root True
189 ghciArgs <- getGhciSessionSettings pkgs root True
190− withGhciScriptArgs pkgs $ \dotGhciArgs ->
190+ withGhciScript pkgs $ \dotGhciPath ->
191− runGhcid root True (ghciArgs <> dotGhciArgs) pkgs Nothing
191+ runGhcid root True (ghciArgs <> mkGhciScriptArg dotGhciPath) pkgs Nothing
193exportGhciConfig :: MonadObelisk m => Bool -> FilePath -> PathTree Interpret -> m [String]
193exportGhciConfig :: MonadObelisk m => Bool -> FilePath -> PathTree Interpret -> m [String]
194exportGhciConfig useRelativePaths root interpretPaths = do
194exportGhciConfig useRelativePaths root interpretPaths = do
@@ -304,16 +304,15 @@ parseCabalPackage
304 -> m (Maybe CabalPackageInfo)
304 -> m (Maybe CabalPackageInfo)
305parseCabalPackage dir = parseCabalPackage' dir >>= \case
305parseCabalPackage dir = parseCabalPackage' dir >>= \case
306 Left err -> Nothing <$ putLog Error err
306 Left err -> Nothing <$ putLog Error err
307− Right (Just (warnings, pkgInfo)) -> do
307+ Right (warnings, pkgInfo) -> do
308 for_ warnings $ putLog Warning . T.pack . show
308 for_ warnings $ putLog Warning . T.pack . show
309 pure $ Just pkgInfo
309 pure $ Just pkgInfo
310− Right Nothing -> pure Nothing
312-- | Like 'parseCabalPackage' but returns errors and warnings directly so as to avoid 'MonadObelisk'.
311-- | Like 'parseCabalPackage' but returns errors and warnings directly so as to avoid 'MonadObelisk'.
313parseCabalPackage'
312parseCabalPackage'
314 :: (MonadIO m)
313 :: (MonadIO m)
315 => FilePath -- ^ Package directory
314 => FilePath -- ^ Package directory
316− -> m (Either T.Text (Maybe ([Dist.PWarning], CabalPackageInfo)))
315+ -> m (Either T.Text ([Dist.PWarning], CabalPackageInfo))
317parseCabalPackage' pkg = runExceptT $ do
316parseCabalPackage' pkg = runExceptT $ do
318 (cabalContents, packageFile, packageName) <- guessCabalPackageFile pkg >>= \case
317 (cabalContents, packageFile, packageName) <- guessCabalPackageFile pkg >>= \case
319 Left GuessPackageFileError_NotFound -> throwError $ "No .cabal or package.yaml file found in " <> T.pack pkg
318 Left GuessPackageFileError_NotFound -> throwError $ "No .cabal or package.yaml file found in " <> T.pack pkg
@@ -341,7 +340,7 @@ parseCabalPackage' pkg = runExceptT $ do
341 case condLibrary <$> result of
340 case condLibrary <$> result of
342 Right (Just condLib) -> do
341 Right (Just condLib) -> do
343 let (_, lib) = simplifyCondTree evalConfVar condLib
342 let (_, lib) = simplifyCondTree evalConfVar condLib
344− pure $ Just $ (warnings,) $ CabalPackageInfo
343+ pure $ (warnings,) $ CabalPackageInfo
345 { _cabalPackageInfo_packageName = T.pack packageName
344 { _cabalPackageInfo_packageName = T.pack packageName
346 , _cabalPackageInfo_packageFile = packageFile
345 , _cabalPackageInfo_packageFile = packageFile
347 , _cabalPackageInfo_packageRoot = takeDirectory packageFile
346 , _cabalPackageInfo_packageRoot = takeDirectory packageFile
@@ -353,19 +352,18 @@ parseCabalPackage' pkg = runExceptT $ do
353 , _cabalPackageInfo_defaultLanguage =
352 , _cabalPackageInfo_defaultLanguage =
354 defaultLanguage $ libBuildInfo lib
353 defaultLanguage $ libBuildInfo lib
355 , _cabalPackageInfo_compilerOptions = options $ libBuildInfo lib
354 , _cabalPackageInfo_compilerOptions = options $ libBuildInfo lib
356− , _cabalPackageInfo_cppOptions = cppOptions $ libBuildInfo lib
358− Right Nothing -> pure Nothing
356+ Right Nothing -> throwError "Haskell package has no library component"
359 Left (_, errors) ->
357 Left (_, errors) ->
360 throwError $ T.pack $ "Failed to parse " <> packageFile <> ":\n" <> unlines (map show errors)
358 throwError $ T.pack $ "Failed to parse " <> packageFile <> ":\n" <> unlines (map show errors)
362parsePackagesOrFail :: (MonadObelisk m, Foldable f) => f FilePath -> m (NE.NonEmpty CabalPackageInfo)
360parsePackagesOrFail :: (MonadObelisk m, Foldable f) => f FilePath -> m (NE.NonEmpty CabalPackageInfo)
363parsePackagesOrFail dirs' = do
361parsePackagesOrFail dirs' = do
364− packageInfos' <- fmap catMaybes $ for dirs $ \dir -> do
362+ (pkgDirErrs, packageInfos') <- fmap partitionEithers $ for dirs $ \dir -> do
365 flip fmap (parseCabalPackage dir) $ \case
363 flip fmap (parseCabalPackage dir) $ \case
366 Just packageInfo
364 Just packageInfo
367− | _cabalPackageInfo_buildable packageInfo -> Just packageInfo
365+ | _cabalPackageInfo_buildable packageInfo -> Right packageInfo
368− _ -> Nothing
366+ _ -> Left dir
370 -- Sort duplicate packages such that we prefer shorter paths, but fall back to alphabetical ordering.
368 -- Sort duplicate packages such that we prefer shorter paths, but fall back to alphabetical ordering.
371 let packagesByName = Map.map (NE.sortBy $ comparing $ \p -> let n = _cabalPackageInfo_packageFile p in (length n, n))
369 let packagesByName = Map.map (NE.sortBy $ comparing $ \p -> let n = _cabalPackageInfo_packageFile p in (length n, n))
@@ -387,6 +385,9 @@ parsePackagesOrFail dirs' = do
387 "No valid, buildable packages found" <> (if null dirs then "" else " in " <> intercalate ", " dirs)
385 "No valid, buildable packages found" <> (if null dirs then "" else " in " <> intercalate ", " dirs)
388 Just xs -> pure xs
386 Just xs -> pure xs
388+ unless (null pkgDirErrs) $
389+ putLog Warning $ T.pack $ "Failed to find buildable packages in " <> intercalate ", " pkgDirErrs
390 pure packageInfos
391 pure packageInfos
392 dirs = toList dirs'
393 dirs = toList dirs'
@@ -394,29 +395,13 @@ parsePackagesOrFail dirs' = do
394packageInfoToNamePathMap :: Foldable f => f CabalPackageInfo -> Map Text FilePath
395packageInfoToNamePathMap :: Foldable f => f CabalPackageInfo -> Map Text FilePath
395packageInfoToNamePathMap = Map.fromList . map (_cabalPackageInfo_packageName &&& _cabalPackageInfo_packageRoot) . toList
396packageInfoToNamePathMap = Map.fromList . map (_cabalPackageInfo_packageName &&& _cabalPackageInfo_packageRoot) . toList
397−-- Like 'withGhciScript' but provides the precise ghci arguments to add to a ghci session
399− :: (MonadObelisk m, Foldable f)
400− => f CabalPackageInfo -- ^ List of packages to load into ghci
401− -> ([String] -> m ()) -- ^ Action to run with the extra ghci arguments
403−withGhciScriptArgs packageInfos f = withGhciScript loadPreludeManually packageInfos $ \fp -> f ["-XNoImplicitPrelude", "-ghci-script", fp]
405− -- These lines must be first and allow the session to support a custom Prelude when @-XNoImplicitPrelude@
406− -- is passed to the ghci session.
407− loadPreludeManually =
408− [ ":add Prelude" -- @:add@ is used because it's less noisy when there is no custom Prelude
409− , ":set -XImplicitPrelude" -- Turn the default setting on
412-- | Create ghci configuration to load the given packages
398-- | Create ghci configuration to load the given packages
413withGhciScript
399withGhciScript
414 :: (MonadObelisk m, Foldable f)
400 :: (MonadObelisk m, Foldable f)
415− => [String] -- ^ Commands to prefix to file
416− -> f CabalPackageInfo -- ^ List of packages to load into ghci
401+ => f CabalPackageInfo -- ^ List of packages to load into ghci
417 -> (FilePath -> m ()) -- ^ Action to run with the path to generated temporary .ghci
402 -> (FilePath -> m ()) -- ^ Action to run with the path to generated temporary .ghci
419−withGhciScript preCommands (toList -> packageInfos) f =
404+withGhciScript (toList -> packageInfos) f =
420 withSystemTempDirectory "ob-ghci" $ \fp -> do
405 withSystemTempDirectory "ob-ghci" $ \fp -> do
421 let dotGhciPath = fp </> ".ghci"
406 let dotGhciPath = fp </> ".ghci"
422 liftIO $ writeFile dotGhciPath dotGhci
407 liftIO $ writeFile dotGhciPath dotGhci
@@ -428,8 +413,7 @@ withGhciScript preCommands (toList -> packageInfos) f =
428 , [ "Backend" | "backend" `Set.member` packageNames ]
413 , [ "Backend" | "backend" `Set.member` packageNames ]
429 , [ "Frontend" | "frontend" `Set.member` packageNames ]
414 , [ "Frontend" | "frontend" `Set.member` packageNames ]
431− dotGhci = unlines $
416+ dotGhci = unlines
433 [ if null modulesToLoad then "" else ":load " <> unwords modulesToLoad
417 [ if null modulesToLoad then "" else ":load " <> unwords modulesToLoad
434 , "import qualified Obelisk.Run"
418 , "import qualified Obelisk.Run"
435 , "import qualified Frontend"
419 , "import qualified Frontend"
@@ -482,7 +466,7 @@ runGhciRepl root (toList -> packages) ghciArgs =
482 -- NOTE: We do *not* want to use $(staticWhich "ghci") here because we need the
466 -- NOTE: We do *not* want to use $(staticWhich "ghci") here because we need the
483 -- ghc that is provided by the shell in the user's project.
467 -- ghc that is provided by the shell in the user's project.
484 nixShellWithoutPkgs root True False (packageInfoToNamePathMap packages) "ghc" $
468 nixShellWithoutPkgs root True False (packageInfoToNamePathMap packages) "ghc" $
485− Just $ unwords $ fmap bashEscape $ "ghci" : ghciArgs
469+ Just $ unwords $ fmap (BSU.toString . bytes . bash . BSU.fromString) $ "ghci" : ghciArgs
487-- | Run ghcid
471-- | Run ghcid
@@ -495,7 +479,7 @@ runGhcid
496runGhcid root chdirToRoot ghciArgs (toList -> packages) mcmd =
480runGhcid root chdirToRoot ghciArgs (toList -> packages) mcmd =
497 nixShellWithoutPkgs root True chdirToRoot (packageInfoToNamePathMap packages) "ghc" $
481 nixShellWithoutPkgs root True chdirToRoot (packageInfoToNamePathMap packages) "ghc" $
498− Just $ unwords $ fmap bashEscape $ ghcidExePath : opts
482+ Just $ unwords $ fmap (BSU.toString . bytes . bash . BSU.fromString) $ ghcidExePath : opts
500 opts = concat
484 opts = concat
@@ -509,14 +493,17 @@ runGhcid root chdirToRoot ghciArgs (toList -> packages) mcmd =
509 reloadFiles = map adjustRoot [root </> "config"]
493 reloadFiles = map adjustRoot [root </> "config"]
510 restartFiles = map (adjustRoot . _cabalPackageInfo_packageFile) packages
494 restartFiles = map (adjustRoot . _cabalPackageInfo_packageFile) packages
496+mkGhciScriptArg :: FilePath -> [String]
497+mkGhciScriptArg dotGhci = ["-ghci-script", dotGhci]
512−getFreePort :: MonadIO m => m Socket.PortNumber
499+getFreePort :: MonadIO m => m PortNumber
513−getFreePort = liftIO $ Socket.withSocketsDo $ do
500+getFreePort = liftIO $ withSocketsDo $ do
514− addr:_ <- Socket.getAddrInfo (Just Socket.defaultHints) (Just "127.0.0.1") (Just "0")
501+ addr:_ <- getAddrInfo (Just defaultHints) (Just "127.0.0.1") (Just "0")
515− bracket (open addr) Socket.close Socket.socketPort
502+ bracket (open addr) close socketPort
517 open addr = do
504 open addr = do
518− sock <- Socket.socket (Socket.addrFamily addr) (Socket.addrSocketType addr) (Socket.addrProtocol addr)
505+ sock <- socket (addrFamily addr) (addrSocketType addr) (addrProtocol addr)
519− Socket.bind sock (Socket.addrAddress addr)
506+ bind sock (addrAddress addr)
520 return sock
507 return sock