@@ -5,72 +5,59 @@
55{-# LANGUAGE MultiWayIf #-}
66{-# LANGUAGE OverloadedStrings #-}
77{-# LANGUAGE PatternGuards #-}
8+{-# LANGUAGE QuasiQuotes #-}
89{-# LANGUAGE ScopedTypeVariables #-}
910{-# LANGUAGE TypeApplications #-}
10−module Obelisk.Command.Thunk
11− ( ThunkPtr (..)
12− , ThunkRev (..)
13− , ThunkSource (..)
14− , ThunkData (..)
15− , ReadThunkError (..)
16− , GitHubSource (..)
17− , getThunkGitBranch
18− , getLatestRev
19− , updateThunkToLatest
20− , createThunk
21− , createThunkWithLatest
22− , nixBuildAttrWithCache
23− , nixBuildThunkAttrWithCache
24− , unpackThunk
25− , packThunk
26− , readThunk
27− , updateThunk
28− , getThunkPtr
29− , getThunkPtr'
30− , parseGitUri
31− , uriThunkPtr
32− ) where
11+module Obelisk.Command.Thunk where
3312
3413import Control.Applicative
3514import Control.Exception (displayException, try)
36−import qualified Control.Lens as Lens
37−import Control.Lens.Indexed hiding ((<.>))
15+import Control.Lens (ifor, ifor_, (.~))
3816import Control.Monad
39−import Control.Monad.Catch (handle)
17+import Control.Monad.Extra (findM)
18+import Control.Monad.Catch (MonadCatch, handle)
4019import Control.Monad.Except
4120import Data.Aeson ((.=))
4221import qualified Data.Aeson as Aeson
4322import Data.Aeson.Encode.Pretty
4423import qualified Data.Aeson.Types as Aeson
24+import Data.Bifunctor (first)
4525import qualified Data.ByteString.Lazy as LBS
26+import Data.Containers.ListUtils (nubOrd)
4627import Data.Default
4728import Data.Either.Combinators (fromRight', rightToMaybe)
29+import Data.Foldable (toList)
30+import Data.Function ((&))
31+import Data.Functor ((<&>))
4832import Data.Git.Ref (Ref)
4933import qualified Data.Git.Ref as Ref
5034import qualified Data.List as L
51−import Data.List.NonEmpty (NonEmpty (..))
35+import Data.List.NonEmpty (NonEmpty (..), nonEmpty)
5236import qualified Data.List.NonEmpty as NonEmpty
5337import Data.Map (Map)
5438import qualified Data.Map as Map
5539import Data.Maybe
5640import Data.Set (Set)
5741import qualified Data.Set as Set
42+import Data.String.Here.Interpolated (i)
43+import Data.String.Here.Uninterpolated (here)
5844import Data.Text (Text)
5945import qualified Data.Text as T
6046import Data.Text.Encoding
6147import qualified Data.Text.IO as T
6248import Data.Time.Clock.POSIX (posixSecondsToUTCTime)
49+import Data.Traversable (for)
6350import Data.Yaml (parseMaybe)
6451import GitHub
6552import GitHub.Data.Name
6653import Obelisk.Command.Nix
6754import System.Directory
55+import System.Exit
6856import System.FilePath
6957import System.IO.Error
7058import System.IO.Temp
71−import System.Posix (getSymbolicLinkStatus, modificationTime)
72−import System.Process (proc)
73−import Text.URI
59+import System.PosixCompat.Files (getSymbolicLinkStatus, modificationTime)
60+import qualified Text.URI as URI
7461
7562import Obelisk.App (MonadObelisk)
7663import Obelisk.CliApp
@@ -78,11 +65,10 @@ import Obelisk.Command.Utils
7865
7966--TODO: Support symlinked thunk data
8067data ThunkData
81− = ThunkData_Packed ThunkPtr
68+ = ThunkData_Packed ThunkSpec ThunkPtr
8269 -- ^ Packed thunk
83− | ThunkData_Checkout (Maybe ThunkPtr)
70+ | ThunkData_Checkout
8471 -- ^ Checked out thunk that was unpacked from this pointer
85− deriving (Show, Eq, Ord)
8672
8773-- | A reference to the exact data that a thunk should translate into
8874data ThunkPtr = ThunkPtr
@@ -95,7 +81,7 @@ type NixSha256 = Text --TODO: Use a smart constructor and make this actually ver
9581
9682-- | A specific revision of data; it may be available from multiple sources
9783data ThunkRev = ThunkRev
98− { _thunkRev_commit :: Ref
84+ { _thunkRev_commit :: Ref Ref.SHA1
9985 , _thunkRev_nixSha256 :: NixSha256
10086 }
10187 deriving (Show, Eq, Ord)
@@ -108,41 +94,72 @@ data ThunkSource
10894 | ThunkSource_Git GitSource
10995 deriving (Show, Eq, Ord)
11096
97+thunkSourceToGitSource :: ThunkSource -> GitSource
98+thunkSourceToGitSource = \case
99+ ThunkSource_GitHub s -> forgetGithub False s
100+ ThunkSource_Git s -> s
101+
111102data GitHubSource = GitHubSource
112103 { _gitHubSource_owner :: Name Owner
113104 , _gitHubSource_repo :: Name Repo
114105 , _gitHubSource_branch :: Maybe (Name Branch)
106+ , _gitHubSource_private :: Bool
115107 }
116108 deriving (Show, Eq, Ord)
117109
110+newtype GitUri = GitUri { unGitUri :: URI.URI } deriving (Eq, Ord, Show)
111+
112+gitUriToText :: GitUri -> Text
113+gitUriToText (GitUri uri)
114+ | (T.toLower . URI.unRText <$> URI.uriScheme uri) == Just "file"
115+ , Just (_, path) <- URI.uriPath uri
116+ = "/" <> T.intercalate "/" (map URI.unRText $ NonEmpty.toList path)
117+ | otherwise = URI.render uri
118+
118119data GitSource = GitSource
119− { _gitSource_url :: URI
120+ { _gitSource_url :: GitUri
120121 , _gitSource_branch :: Maybe (Name Branch)
121122 , _gitSource_fetchSubmodules :: Bool
123+ , _gitSource_private :: Bool
122124 }
123125 deriving (Show, Eq, Ord)
124126
127+newtype ThunkConfig = ThunkConfig
128+ { _thunkConfig_private :: Maybe Bool
129+ } deriving Show
130+
131+data ThunkUpdateConfig = ThunkUpdateConfig
132+ { _thunkUpdateConfig_branch :: Maybe String
133+ , _thunkUpdateConfig_config :: ThunkConfig
134+ } deriving Show
135+
136+data ThunkPackConfig = ThunkPackConfig
137+ { _thunkPackConfig_force :: Bool
138+ , _thunkPackConfig_config :: ThunkConfig
139+ } deriving Show
140+
125141-- | Convert a GitHub source to a regular Git source. Assumes no submodules.
126142forgetGithub :: Bool -> GitHubSource -> GitSource
127143forgetGithub useSsh s = GitSource
128− { _gitSource_url = URI
129− { uriScheme = Just $ fromRight' $ mkScheme $ if useSsh then "ssh" else "https"
130− , uriAuthority = Right $ Authority
131− { authUserInfo = UserInfo (fromRight' $ mkUsername "git") Nothing
144+ { _gitSource_url = GitUri $ URI.URI
145+ { URI.uriScheme = Just $ fromRight' $ URI.mkScheme $ if useSsh then "ssh" else "https"
146+ , URI.uriAuthority = Right $ URI.Authority
147+ { URI.authUserInfo = URI.UserInfo (fromRight' $ URI.mkUsername "git") Nothing
132148 <$ guard useSsh
133− , authHost = fromRight' $ mkHost "github.com"
134− , authPort = Nothing
149+ , URI.authHost = fromRight' $ URI.mkHost "github.com"
150+ , URI.authPort = Nothing
135151 }
136− , uriPath = Just ( False
137− , fromRight' . mkPathPiece <$>
152+ , URI.uriPath = Just ( False
153+ , fromRight' . URI.mkPathPiece <$>
138154 untagName (_gitHubSource_owner s)
139155 :| [ untagName (_gitHubSource_repo s) <> ".git" ]
140156 )
141− , uriQuery = []
142− , uriFragment = Nothing
157+ , URI.uriQuery = []
158+ , URI.uriFragment = Nothing
143159 }
144160 , _gitSource_branch = _gitHubSource_branch s
145161 , _gitSource_fetchSubmodules = False
162+ , _gitSource_private = _gitHubSource_private s
146163 }
147164
148165getThunkGitBranch :: ThunkPtr -> Maybe Text
@@ -150,26 +167,26 @@ getThunkGitBranch (ThunkPtr _ src) = fmap untagName $ case src of
150167 ThunkSource_GitHub s -> _gitHubSource_branch s
151168 ThunkSource_Git s -> _gitSource_branch s
152169
153−commitNameToRef :: Name Commit -> Ref
170+commitNameToRef :: Name Commit -> Ref Ref.SHA1
154171commitNameToRef (N c) = Ref.fromHex $ encodeUtf8 c
155172
156173-- TODO: Use spinner here.
157174getNixSha256ForUriUnpacked
158175 :: MonadObelisk m
159− => URI
176+ => GitUri
160177 -> m NixSha256
161178getNixSha256ForUriUnpacked uri =
162− withExitFailMessage ("nix-prefetch-url: Failed to determine sha256 hash of URL " <> render uri) $ do
179+ withExitFailMessage ("nix-prefetch-url: Failed to determine sha256 hash of URL " <> gitUriToText uri) $ do
163180 [hash] <- fmap T.lines $ readProcessAndLogOutput (Debug, Debug) $
164− proc "nix-prefetch-url" ["--unpack" , "--type" , "sha256" , T.unpack $ render uri]
181+ proc nixPrefetchUrlPath ["--unpack", "--type", "sha256", T.unpack $ gitUriToText uri]
165182 pure hash
166183
167−nixPrefetchGit :: MonadObelisk m => URI -> Text -> Bool -> m NixSha256
184+nixPrefetchGit :: MonadObelisk m => GitUri -> Text -> Bool -> m NixSha256
168185nixPrefetchGit uri rev fetchSubmodules =
169− withExitFailMessage ("nix-prefetch-git: Failed to determine sha256 hash of Git repo " <> render uri <> " at " <> rev) $ do
186+ withExitFailMessage ("nix-prefetch-git: Failed to determine sha256 hash of Git repo " <> gitUriToText uri <> " at " <> rev) $ do
170187 out <- readProcessAndLogStderr Debug $
171− proc "nix-prefetch-git" $ filter (/="")
172− [ "--url", T.unpack $ render uri
188+ proc nixPrefetchGitPath $ filter (/="")
189+ [ "--url", T.unpack $ gitUriToText uri
173190 , "--rev", T.unpack rev
174191 , if fetchSubmodules then "--fetch-submodules" else ""
175192 , "--quiet"
@@ -181,93 +198,117 @@ nixPrefetchGit uri rev fetchSubmodules =
181198
182199--TODO: Pretty print these
183200data ReadThunkError
184− = ReadThunkError_UnrecognizedFiles
185− | ReadThunkError_AmbiguousFiles
186− | ReadThunkError_UnrecognizedLoader Text
187− | ReadThunkError_UnparseablePtr LBS.ByteString
188− deriving (Show)
189−
190−-- | Read a thunk and validate that it is only a thunk, either packed or unpacked.
191−-- If the thunk is packed and additional data is present, fail.
192−readThunk :: MonadObelisk m => FilePath -> m (Either ReadThunkError ThunkData)
193−readThunk thunkDir = do
194− files <- liftIO $ listDirectory thunkDir
195− case ".git" `elem` files of
196− True -> fmap ThunkData_Checkout <$> do
197− let origThunkPath = thunkDir </> ".git" </> "obelisk" </> "orig-thunk"
198− liftIO $ doesDirectoryExist origThunkPath >>= \case
199− True -> fmap Just <$> readPackedThunk origThunkPath
200− False -> return $ return Nothing
201− False -> liftIO $ fmap ThunkData_Packed <$> readPackedThunk thunkDir
202−
203−data ThunkType = ThunkType
204− { _thunkType_loader :: FilePath
205− , _thunkType_json :: FilePath
206− , _thunkType_optional :: Set FilePath
207− , _thunkType_loaderVersions :: NonEmpty Text
208− , _thunkType_parser :: Aeson.Object -> Aeson.Parser ThunkPtr
201+ = ReadThunkError_UnrecognizedThunk
202+ | ReadThunkError_UnrecognizedPaths (NonEmpty FilePath)
203+ | ReadThunkError_MissingPaths (NonEmpty FilePath)
204+ | ReadThunkError_UnparseablePtr FilePath String
205+ | ReadThunkError_FileError IOError
206+ | ReadThunkError_FileDoesNotMatch FilePath Text
207+ | ReadThunkError_UnrecognizedState String
208+ | ReadThunkError_AmbiguousPackedState ThunkPtr ThunkPtr
209+ deriving (Show)
210+
211+unpackedDirName :: FilePath
212+unpackedDirName = "."
213+
214+attrCacheFileName :: FilePath
215+attrCacheFileName = ".attr-cache"
216+
217+-- | Specification for how a file in a thunk version works.
218+data ThunkFileSpec
219+ = ThunkFileSpec_Ptr (LBS.ByteString -> Either String ThunkPtr) -- ^ This file specifies 'ThunkPtr' data
220+ | ThunkFileSpec_FileMatches Text -- ^ This file must match the given content exactly
221+ | ThunkFileSpec_CheckoutIndicator -- ^ Existence of this directory indicates that the thunk is unpacked
222+ | ThunkFileSpec_AttrCache -- ^ This directory is an attribute cache
223+
224+-- | Specification for how a set of files in a thunk version work.
225+data ThunkSpec = ThunkSpec
226+ { _thunkSpec_name :: !Text
227+ , _thunkSpec_files :: !(Map FilePath ThunkFileSpec)
209228 }
210229
211−gitHubThunkType :: ThunkType
212−gitHubThunkType = ThunkType
213− { _thunkType_loader = "default.nix"
214− , _thunkType_json = "github.json"
215− , _thunkType_optional = Set.fromList [".attr-cache"]
216− , _thunkType_loaderVersions = gitHubStandaloneLoaders
217− , _thunkType_parser = parseThunkPtr $ \v ->
218− ThunkSource_GitHub <$> parseGitHubSource v <|> ThunkSource_Git <$> parseGitSource v
219− }
220−
221−gitThunkType :: ThunkType
222−gitThunkType = ThunkType
223− { _thunkType_loader = "default.nix"
224− , _thunkType_json = "git.json"
225− , _thunkType_optional = Set.fromList [".attr-cache"]
226− , _thunkType_loaderVersions = plainGitStandaloneLoaders
227− , _thunkType_parser = parseThunkPtr $ fmap ThunkSource_Git . parseGitSource
228− }
229−
230−thunkTypes :: [ThunkType]
231−thunkTypes = [gitThunkType, gitHubThunkType]
232−
233−findThunkType :: [ThunkType] -> FilePath -> IO (Either ReadThunkError ThunkType)
234−findThunkType types thunkDir = do
235− matches <- fmap catMaybes $ forM types $ \thunkType -> do
236− let
237− expectedContents = Set.fromList $ (thunkDir </>) <$>
238− [ _thunkType_loader thunkType
239− , _thunkType_json thunkType
240− ]
241− optionalContents = Set.map (thunkDir </>) (_thunkType_optional thunkType)
242−
243− -- Ensure that there aren't any other files in the thunk
244− -- NB: System.Directory.listDirectory returns the contents without the directory path
245− files <- liftIO $ Set.fromList . fmap (thunkDir </>) <$> listDirectory thunkDir
246− let unexpectedContents = files `Set.difference` (expectedContents `Set.union` optionalContents)
247− missingContents = expectedContents `Set.difference` files
248− pure $ if Set.null unexpectedContents && Set.null missingContents
249− then Just thunkType
250− else Nothing
251−
252− pure $ case matches of
253− [singleMatch] -> Right singleMatch
254− [] -> Left ReadThunkError_UnrecognizedFiles
255− _ -> Left ReadThunkError_AmbiguousFiles
230+thunkSpecTypes :: NonEmpty (NonEmpty ThunkSpec)
231+thunkSpecTypes = gitThunkSpecs :| [gitHubThunkSpecs]
232+
233+-- | Attempts to match a 'ThunkSpec' to a given directory.
234+matchThunkSpecToDir
235+ :: (MonadError ReadThunkError m, MonadIO m, MonadCatch m)
236+ => ThunkSpec -- ^ 'ThunkSpec' to match against the given files/directory
237+ -> FilePath -- ^ Path to directory
238+ -> Set FilePath -- ^ Set of file paths relative to the given directory
239+ -> m ThunkData
240+matchThunkSpecToDir thunkSpec dir dirFiles = do
241+ case nonEmpty (toList $ dirFiles `Set.difference` expectedPaths) of
242+ Just fs -> throwError $ ReadThunkError_UnrecognizedPaths $ (dir </>) <$> fs
243+ Nothing -> pure ()
244+ case nonEmpty (toList $ requiredPaths `Set.difference` dirFiles) of
245+ Just fs -> throwError $ ReadThunkError_MissingPaths $ (dir </>) <$> fs
246+ Nothing -> pure ()
247+ datas <- fmap toList $ flip Map.traverseMaybeWithKey (_thunkSpec_files thunkSpec) $ \expectedPath -> \case
248+ ThunkFileSpec_AttrCache -> Nothing <$ dirMayExist expectedPath
249+ ThunkFileSpec_CheckoutIndicator -> liftIO (doesDirectoryExist (dir </> expectedPath)) <&> \case
250+ False -> Nothing
251+ True -> Just ThunkData_Checkout
252+ ThunkFileSpec_FileMatches expectedContents -> handle (\(e :: IOError) -> throwError $ ReadThunkError_FileError e) $ do
253+ actualContents <- liftIO (T.readFile $ dir </> expectedPath)
254+ case T.strip expectedContents == T.strip actualContents of
255+ True -> pure Nothing
256+ False -> throwError $ ReadThunkError_FileDoesNotMatch (dir </> expectedPath) expectedContents
257+ ThunkFileSpec_Ptr parser -> handle (\(e :: IOError) -> throwError $ ReadThunkError_FileError e) $ do
258+ let path = dir </> expectedPath
259+ liftIO (doesFileExist path) >>= \case
260+ False -> pure Nothing
261+ True -> do
262+ actualContents <- liftIO $ LBS.readFile path
263+ case parser actualContents of
264+ Right v -> pure $ Just (ThunkData_Packed thunkSpec v)
265+ Left e -> throwError $ ReadThunkError_UnparseablePtr (dir </> expectedPath) e
266+
267+ case nonEmpty datas of
268+ Nothing -> throwError ReadThunkError_UnrecognizedThunk
269+ Just xs -> fold1WithM xs $ \a b -> either throwError pure (mergeThunkData a b)
270+ where
271+ rootPathsOnly = Set.fromList . mapMaybe takeRootDir . Map.keys
272+ takeRootDir = fmap NonEmpty.head . nonEmpty . splitPath
273+
274+ expectedPaths = rootPathsOnly $ _thunkSpec_files thunkSpec
275+
276+ requiredPaths = rootPathsOnly $ Map.filter isRequiredFileSpec $ _thunkSpec_files thunkSpec
277+ isRequiredFileSpec = \case
278+ ThunkFileSpec_FileMatches _ -> True
279+ _ -> False
280+
281+ dirMayExist expectedPath = liftIO (doesFileExist (dir </> expectedPath)) >>= \case
282+ True -> throwError $ ReadThunkError_UnrecognizedPaths $ expectedPath :| []
283+ False -> pure ()
284+
285+ -- Combine 'ThunkData' from different files, preferring "Checkout" over "Packed"
286+ mergeThunkData ThunkData_Checkout ThunkData_Checkout = Right ThunkData_Checkout
287+ mergeThunkData ThunkData_Checkout ThunkData_Packed{} = Left bothPackedAndUnpacked
288+ mergeThunkData ThunkData_Packed{} ThunkData_Checkout = Left bothPackedAndUnpacked
289+ mergeThunkData a@(ThunkData_Packed _ ptrA) (ThunkData_Packed _ ptrB) =
290+ if ptrA == ptrB then Right a else Left $ ReadThunkError_AmbiguousPackedState ptrA ptrB
291+
292+ bothPackedAndUnpacked = ReadThunkError_UnrecognizedState "Both packed data and checkout present"
293+
294+ fold1WithM (x :| xs) f = foldM f x xs
295+
296+readThunkWith
297+ :: (MonadObelisk m)
298+ => NonEmpty (NonEmpty ThunkSpec) -> FilePath -> m (Either ReadThunkError ThunkData)
299+readThunkWith specTypes dir = do
300+ dirFiles <- Set.fromList <$> liftIO (listDirectory dir)
301+ let specs = concatMap toList $ toList $ NonEmpty.transpose specTypes -- Interleave spec types so we try each one in a "fair" ordering
302+ flip fix specs $ \loop -> \case
303+ [] -> pure $ Left ReadThunkError_UnrecognizedThunk
304+ spec:rest -> runExceptT (matchThunkSpecToDir spec dir dirFiles) >>= \case
305+ Left e -> putLog Debug [i|Thunk specification ${_thunkSpec_name spec} did not match ${dir}: ${e}|] *> loop rest
306+ x@(Right _) -> x <$ putLog Debug [i|Thunk specification ${_thunkSpec_name spec} matched ${dir}|]
256307
257308-- | Read a thunk and validate that it is exactly a packed thunk.
258309-- If additional data is present, fail.
259−readPackedThunk :: FilePath -> IO (Either ReadThunkError ThunkPtr)
260−readPackedThunk thunkDir = runExceptT $ do
261− thunkType <- ExceptT $ findThunkType thunkTypes thunkDir
262− -- Ensure that we recognize the thunk loader
263− loader <- liftIO $ T.readFile $ thunkDir </> _thunkType_loader thunkType
264− unless (loader `elem` _thunkType_loaderVersions thunkType) $ do
265− throwError $ ReadThunkError_UnrecognizedLoader loader
266−
267− txt <- liftIO $ LBS.readFile $ thunkDir </> _thunkType_json thunkType
268− case parseMaybe (_thunkType_parser thunkType) =<< Aeson.decode txt of
269− Nothing -> throwError $ ReadThunkError_UnparseablePtr txt
270− Just ptr -> return ptr
310+readThunk :: (MonadObelisk m) => FilePath -> m (Either ReadThunkError ThunkData)
311+readThunk = readThunkWith thunkSpecTypes
271312
272313parseThunkPtr :: (Aeson.Object -> Aeson.Parser ThunkSource) -> Aeson.Object -> Aeson.Parser ThunkPtr
273314parseThunkPtr parseSrc v = do
@@ -287,10 +328,12 @@ parseGitHubSource v = do
287328 owner <- v Aeson..: "owner"
288329 repo <- v Aeson..: "repo"
289330 branch <- v Aeson..:! "branch"
331+ private <- v Aeson..:? "private"
290332 pure $ GitHubSource
291333 { _gitHubSource_owner = owner
292334 , _gitHubSource_repo = repo
293335 , _gitHubSource_branch = branch
336+ , _gitHubSource_private = fromMaybe False private
294337 }
295338
296339parseGitSource :: Aeson.Object -> Aeson.Parser GitSource
@@ -298,25 +341,29 @@ parseGitSource v = do
298341 Just url <- parseGitUri <$> v Aeson..: "url"
299342 branch <- v Aeson..:! "branch"
300343 fetchSubmodules <- v Aeson..:! "fetchSubmodules"
344+ private <- v Aeson..:? "private"
301345 pure $ GitSource
302346 { _gitSource_url = url
303347 , _gitSource_branch = branch
304348 , _gitSource_fetchSubmodules = fromMaybe False fetchSubmodules
349+ , _gitSource_private = fromMaybe False private
305350 }
306351
307352overwriteThunk :: MonadObelisk m => FilePath -> ThunkPtr -> m ()
308353overwriteThunk target thunk = do
309354 -- Ensure that this directory is a valid thunk (i.e. so we aren't losing any data)
310− Right _ <- readThunk target
355+ readThunk target >>= \case
356+ Left e -> failWith [i|Invalid thunk at ${target}: ${e}|]
357+ Right _ -> pure ()
311358
312359 --TODO: Is there a safer way to do this overwriting?
313− liftIO $ removeDirectoryRecursive target
314− liftIO $ createThunk target thunk
360+ liftIO $ removePathForcibly target
361+ createThunk target $ Right thunk
315362
316−thunkPtrLoader :: ThunkPtr -> Text
317−thunkPtrLoader thunk = case _thunkPtr_source thunk of
318− ThunkSource_GitHub _ -> NonEmpty.head gitHubStandaloneLoaders
319− ThunkSource_Git _ -> NonEmpty.head plainGitStandaloneLoaders
363+thunkPtrToSpec :: ThunkPtr -> ThunkSpec
364+thunkPtrToSpec thunk = case _thunkPtr_source thunk of
365+ ThunkSource_GitHub _ -> NonEmpty.head gitHubThunkSpecs
366+ ThunkSource_Git _ -> NonEmpty.head gitThunkSpecs
320367
321368-- It's important that formatting be very consistent here, because
322369-- otherwise when people update thunks, their patches will be messy
@@ -328,13 +375,15 @@ encodeThunkPtrData (ThunkPtr rev src) = case src of
328375 , ("branch" .=) <$> _gitHubSource_branch s
329376 , Just $ "rev" .= Ref.toHexString (_thunkRev_commit rev)
330377 , Just $ "sha256" .= _thunkRev_nixSha256 rev
378+ , Just $ "private" .= _gitHubSource_private s
331379 ]
332380 ThunkSource_Git s -> encodePretty' plainGitCfg $ Aeson.object $ catMaybes
333− [ Just $ "url" .= render (_gitSource_url s)
381+ [ Just $ "url" .= gitUriToText (_gitSource_url s)
334382 , Just $ "rev" .= Ref.toHexString (_thunkRev_commit rev)
335383 , ("branch" .=) <$> _gitSource_branch s
336384 , Just $ "sha256" .= _thunkRev_nixSha256 rev
337385 , Just $ "fetchSubmodules" .= _gitSource_fetchSubmodules s
386+ , Just $ "private" .= _gitSource_private s
338387 ]
339388 where
340389 githubCfg = defConfig
@@ -355,62 +404,84 @@ encodeThunkPtrData (ThunkPtr rev src) = case src of
355404 [ "url"
356405 , "rev"
357406 , "sha256"
407+ , "private"
358408 , "fetchSubmodules"
359409 ] <> compare
360410 , confTrailingNewline = True
361411 }
362412
363−createThunk :: MonadIO m => FilePath -> ThunkPtr -> m ()
364−createThunk target thunk = liftIO $ do
365− createDirectoryIfMissing True (target </> ".attr-cache")
366− T.writeFile (target </> "default.nix") (thunkPtrLoader thunk)
367− let
368− jsonFileName = case _thunkPtr_source thunk of
369− ThunkSource_GitHub _ -> "github"
370− ThunkSource_Git _ -> "git"
371− LBS.writeFile (target </> jsonFileName <.> "json") (encodeThunkPtrData thunk)
413+createThunk :: MonadObelisk m => FilePath -> Either ThunkSpec ThunkPtr -> m ()
414+createThunk target ptrInfo =
415+ ifor_ (_thunkSpec_files spec) $ \path -> \case
416+ ThunkFileSpec_FileMatches content -> withReadyPath path $ \p -> liftIO $ T.writeFile p content
417+ ThunkFileSpec_Ptr _ -> case ptrInfo of
418+ Left _ -> pure () -- We can't write the ptr without it
419+ Right ptr -> withReadyPath path $ \p -> liftIO $ LBS.writeFile p (encodeThunkPtrData ptr)
420+ _ -> pure ()
421+ where
422+ spec = either id thunkPtrToSpec ptrInfo
423+ withReadyPath path f = do
424+ let fullPath = target </> path
425+ putLog Debug $ "Writing thunk file " <> T.pack fullPath
426+ liftIO $ createDirectoryIfMissing True $ takeDirectory fullPath
427+ f fullPath
372428
373429createThunkWithLatest :: MonadObelisk m => FilePath -> ThunkSource -> m ()
374430createThunkWithLatest target s = do
375431 rev <- getLatestRev s
376− createThunk target $ ThunkPtr
432+ createThunk target $ Right $ ThunkPtr
377433 { _thunkPtr_source = s
378434 , _thunkPtr_rev = rev
379435 }
380436
381−updateThunkToLatest :: MonadObelisk m => FilePath -> m ()
382−updateThunkToLatest target = withSpinner' ("Updating thunk " <> T.pack target <> " to latest") (pure $ const $ "Thunk " <> T.pack target <> " updated to latest") $
383− case target of
384− "." -> failWith "ob thunk update directory cannot be '.'"
385− _ -> do
437+updateThunkToLatest :: MonadObelisk m => ThunkUpdateConfig -> FilePath -> m ()
438+updateThunkToLatest (ThunkUpdateConfig mBranch thunkConfig) target = spinner $ do
439+ checkThunkDirectory target
440+ -- check to see if thunk should be updated to a specific branch or just update it's current branch
441+ case mBranch of
442+ Nothing -> do
386443 (overwrite, ptr) <- readThunk target >>= \case
387− Left err -> failWith $ T.pack $ "thunk update: " <> show err
444+ Left err -> failWith [i|Thunk update: ${err}|]
388445 Right c -> case c of
389− ThunkData_Packed t -> return (target, t)
390− ThunkData_Checkout _ -> failWith "cannot update an unpacked thunk"
446+ ThunkData_Packed _ t -> return (target, t)
447+ ThunkData_Checkout -> failWith "cannot update an unpacked thunk"
391448 let src = _thunkPtr_source ptr
392449 rev <- getLatestRev src
393− overwriteThunk overwrite $ ThunkPtr
450+ overwriteThunk overwrite $ modifyThunkPtrByConfig thunkConfig $ ThunkPtr
394451 { _thunkPtr_source = src
395452 , _thunkPtr_rev = rev
396453 }
454+ Just branch -> readThunk target >>= \case
455+ Left err -> failWith [i|Thunk update: ${err}|]
456+ Right c -> case c of
457+ ThunkData_Packed _ t -> setThunk thunkConfig target (thunkSourceToGitSource $ _thunkPtr_source t) branch
458+ ThunkData_Checkout -> failWith [i|Thunk located at ${target} is unpacked. Use 'ob thunk pack' on the desired directory and then try 'ob thunk update' again.|]
459+ where
460+ spinner = withSpinner' ("Updating thunk " <> T.pack target <> " to latest") (pure $ const $ "Thunk " <> T.pack target <> " updated to latest")
461+
462+setThunk :: MonadObelisk m => ThunkConfig -> FilePath -> GitSource -> String -> m ()
463+setThunk thunkConfig target gs branch = do
464+ newThunkPtr <- uriThunkPtr (_gitSource_url gs) (_thunkConfig_private thunkConfig) (Just $ T.pack branch) Nothing
465+ overwriteThunk target newThunkPtr
466+ updateThunkToLatest (ThunkUpdateConfig Nothing thunkConfig) target
397467
398468-- | All recognized github standalone loaders, ordered from newest to oldest.
399469-- This tool will only ever produce the newest one when it writes a thunk.
400−gitHubStandaloneLoaders :: NonEmpty Text
401−gitHubStandaloneLoaders =
402− gitHubStandaloneLoaderV2 :|
403− [ gitHubStandaloneLoaderV3
404− , gitHubStandaloneLoaderV1
470+gitHubThunkSpecs :: NonEmpty ThunkSpec
471+gitHubThunkSpecs =
472+ gitHubThunkSpecV5 :|
473+ [ gitHubThunkSpecV4
474+ , gitHubThunkSpecV3
475+ , gitHubThunkSpecV2
476+ , gitHubThunkSpecV1
405477 ]
406478
407−gitHubStandaloneLoaderV1 :: Text
408−gitHubStandaloneLoaderV1 = T.unlines
409− [ "import ((import <nixpkgs> {}).fetchFromGitHub (builtins.fromJSON (builtins.readFile ./github.json)))"
410− ]
479+gitHubThunkSpecV1 :: ThunkSpec
480+gitHubThunkSpecV1 = legacyGitHubThunkSpec "github-v1"
481+ "import ((import <nixpkgs> {}).fetchFromGitHub (builtins.fromJSON (builtins.readFile ./github.json)))"
411482
412−gitHubStandaloneLoaderV2 :: Text
413−gitHubStandaloneLoaderV2 = T.unlines
483+gitHubThunkSpecV2 :: ThunkSpec
484+gitHubThunkSpecV2 = legacyGitHubThunkSpec "github-v2" $ T.unlines
414485 [ "# DO NOT HAND-EDIT THIS FILE" --TODO: Add something about how to get more info on Obelisk, etc.
415486 , "import ((import <nixpkgs> {}).fetchFromGitHub ("
416487 , " let json = builtins.fromJSON (builtins.readFile ./github.json);"
@@ -420,8 +491,8 @@ gitHubStandaloneLoaderV2 = T.unlines
420491 , "))"
421492 ]
422493
423−gitHubStandaloneLoaderV3 :: Text
424−gitHubStandaloneLoaderV3 = T.unlines
494+gitHubThunkSpecV3 :: ThunkSpec
495+gitHubThunkSpecV3 = legacyGitHubThunkSpec "github-v3" $ T.unlines
425496 [ "# DO NOT HAND-EDIT THIS FILE"
426497 , "let"
427498 , " fetch = { private ? false, ... }@args: if private && builtins.hasAttr \"fetchGit\" builtins"
@@ -440,22 +511,62 @@ gitHubStandaloneLoaderV3 = T.unlines
440511 , "in import (fetch (builtins.fromJSON (builtins.readFile ./github.json)))"
441512 ]
442513
443−plainGitStandaloneLoaders :: NonEmpty Text
444−plainGitStandaloneLoaders =
445− plainGitStandaloneLoaderV1 :|
446− [ plainGitStandaloneLoaderV2
514+gitHubThunkSpecV4 :: ThunkSpec
515+gitHubThunkSpecV4 = legacyGitHubThunkSpec "github-v4" $ T.unlines
516+ [ "# DO NOT HAND-EDIT THIS FILE"
517+ , "let fetch = { private ? false, fetchSubmodules ? false, owner, repo, rev, sha256, ... }:"
518+ , " if !fetchSubmodules && !private then builtins.fetchTarball {"
519+ , " url = \"https://github.com/${owner}/${repo}/archive/${rev}.tar.gz\"; inherit sha256;"
520+ , " } else (import <nixpkgs> {}).fetchFromGitHub {"
521+ , " inherit owner repo rev sha256 fetchSubmodules private;"
522+ , " };"
523+ , "in import (fetch (builtins.fromJSON (builtins.readFile ./github.json)))"
524+ ]
525+
526+legacyGitHubThunkSpec :: Text -> Text -> ThunkSpec
527+legacyGitHubThunkSpec name loader = ThunkSpec name $ Map.fromList
528+ [ ("default.nix", ThunkFileSpec_FileMatches $ T.strip loader)
529+ , ("github.json" , ThunkFileSpec_Ptr parseGitHubJsonBytes)
530+ , (attrCacheFileName, ThunkFileSpec_AttrCache)
531+ , (".git", ThunkFileSpec_CheckoutIndicator)
532+ ]
533+
534+gitHubThunkSpecV5 :: ThunkSpec
535+gitHubThunkSpecV5 = mkThunkSpec "github-v5" "github.json" parseGitHubJsonBytes [here|
536+# DO NOT HAND-EDIT THIS FILE
537+let fetch = { private ? false, fetchSubmodules ? false, owner, repo, rev, sha256, ... }:
538+ if !fetchSubmodules && !private then builtins.fetchTarball {
539+ url = "https://github.com/${owner}/${repo}/archive/${rev}.tar.gz"; inherit sha256;
540+ } else (import <nixpkgs> {}).fetchFromGitHub {
541+ inherit owner repo rev sha256 fetchSubmodules private;
542+ };
543+ json = builtins.fromJSON (builtins.readFile ./github.json);
544+in fetch json
545+|]
546+
547+parseGitHubJsonBytes :: LBS.ByteString -> Either String ThunkPtr
548+parseGitHubJsonBytes = parseJsonObject $ parseThunkPtr $ \v ->
549+ ThunkSource_GitHub <$> parseGitHubSource v <|> ThunkSource_Git <$> parseGitSource v
550+
551+gitThunkSpecs :: NonEmpty ThunkSpec
552+gitThunkSpecs =
553+ gitThunkSpecV5 :|
554+ [ gitThunkSpecV4
555+ , gitThunkSpecV3
556+ , gitThunkSpecV2
557+ , gitThunkSpecV1
447558 ]
448559
449−plainGitStandaloneLoaderV1 :: Text
450−plainGitStandaloneLoaderV1 = T.unlines
560+gitThunkSpecV1 :: ThunkSpec
561+gitThunkSpecV1 = legacyGitThunkSpec "git-v1" $ T.unlines
451562 [ "# DO NOT HAND-EDIT THIS FILE"
452563 , "let fetchGit = {url, rev, ref ? null, branch ? null, sha256 ? null, fetchSubmodules ? null}:"
453564 , " assert !fetchSubmodules; (import <nixpkgs> {}).fetchgit { inherit url rev sha256; };"
454565 , "in import (fetchGit (builtins.fromJSON (builtins.readFile ./git.json)))"
455566 ]
456567
457−plainGitStandaloneLoaderV2 :: Text
458−plainGitStandaloneLoaderV2 = T.unlines
568+gitThunkSpecV2 :: ThunkSpec
569+gitThunkSpecV2 = legacyGitThunkSpec "git-v2" $ T.unlines
459570 [ "# DO NOT HAND-EDIT THIS FILE"
460571 , "let fetchGit = {url, rev, ref ? null, branch ? null, sha256 ? null, fetchSubmodules ? null}:"
461572 , " if builtins.hasAttr \"fetchGit\" builtins"
@@ -464,10 +575,87 @@ plainGitStandaloneLoaderV2 = T.unlines
464575 , "in import (fetchGit (builtins.fromJSON (builtins.readFile ./git.json)))"
465576 ]
466577
467−thunkFileNames :: [FilePath]
468−thunkFileNames = ["default.nix", "github.json", "git.json"]
578+-- This loader has a bug because @builtins.fetchGit@ is not given a @ref@
579+-- and will fail to find commits without this because it does shallow clones.
580+gitThunkSpecV3 :: ThunkSpec
581+gitThunkSpecV3 = legacyGitThunkSpec "git-v3" $ T.unlines
582+ [ "# DO NOT HAND-EDIT THIS FILE"
583+ , "let fetch = {url, rev, ref ? null, sha256 ? null, fetchSubmodules ? false, private ? false, ...}:"
584+ , " let realUrl = let firstChar = builtins.substring 0 1 url; in"
585+ , " if firstChar == \"/\" then /. + url"
586+ , " else if firstChar == \".\" then ./. + url"
587+ , " else url;"
588+ , " in if !fetchSubmodules && private then builtins.fetchGit {"
589+ , " url = realUrl; inherit rev;"
590+ , " } else (import <nixpkgs> {}).fetchgit {"
591+ , " url = realUrl; inherit rev sha256;"
592+ , " };"
593+ , "in import (fetch (builtins.fromJSON (builtins.readFile ./git.json)))"
594+ ]
595+
596+gitThunkSpecV4 :: ThunkSpec
597+gitThunkSpecV4 = legacyGitThunkSpec "git-v4" $ T.unlines
598+ [ "# DO NOT HAND-EDIT THIS FILE"
599+ , "let fetch = {url, rev, branch ? null, sha256 ? null, fetchSubmodules ? false, private ? false, ...}:"
600+ , " let realUrl = let firstChar = builtins.substring 0 1 url; in"
601+ , " if firstChar == \"/\" then /. + url"
602+ , " else if firstChar == \".\" then ./. + url"
603+ , " else url;"
604+ , " in if !fetchSubmodules && private then builtins.fetchGit {"
605+ , " url = realUrl; inherit rev;"
606+ , " ${if branch == null then null else \"ref\"} = branch;"
607+ , " } else (import <nixpkgs> {}).fetchgit {"
608+ , " url = realUrl; inherit rev sha256;"
609+ , " };"
610+ , "in import (fetch (builtins.fromJSON (builtins.readFile ./git.json)))"
611+ ]
612+
613+legacyGitThunkSpec :: Text -> Text -> ThunkSpec
614+legacyGitThunkSpec name loader = ThunkSpec name $ Map.fromList
615+ [ ("default.nix", ThunkFileSpec_FileMatches $ T.strip loader)
616+ , ("git.json" , ThunkFileSpec_Ptr parseGitJsonBytes)
617+ , (attrCacheFileName, ThunkFileSpec_AttrCache)
618+ , (".git", ThunkFileSpec_CheckoutIndicator)
619+ ]
620+
621+gitThunkSpecV5 :: ThunkSpec
622+gitThunkSpecV5 = mkThunkSpec "git-v5" "git.json" parseGitJsonBytes [here|
623+# DO NOT HAND-EDIT THIS FILE
624+let fetch = {url, rev, branch ? null, sha256 ? null, fetchSubmodules ? false, private ? false, ...}:
625+ let realUrl = let firstChar = builtins.substring 0 1 url; in
626+ if firstChar == "/" then /. + url
627+ else if firstChar == "." then ./. + url
628+ else url;
629+ in if !fetchSubmodules && private then builtins.fetchGit {
630+ url = realUrl; inherit rev;
631+ ${if branch == null then null else "ref"} = branch;
632+ } else (import <nixpkgs> {}).fetchgit {
633+ url = realUrl; inherit rev sha256;
634+ };
635+ json = builtins.fromJSON (builtins.readFile ./git.json);
636+in fetch json
637+|]
638+
639+parseGitJsonBytes :: LBS.ByteString -> Either String ThunkPtr
640+parseGitJsonBytes = parseJsonObject $ parseThunkPtr $ fmap ThunkSource_Git . parseGitSource
641+
642+mkThunkSpec :: Text -> FilePath -> (LBS.ByteString -> Either String ThunkPtr) -> Text -> ThunkSpec
643+mkThunkSpec name jsonFileName parser srcNix = ThunkSpec name $ Map.fromList
644+ [ ("default.nix", ThunkFileSpec_FileMatches defaultNixViaSrc)
645+ , ("thunk.nix", ThunkFileSpec_FileMatches srcNix)
646+ , (jsonFileName, ThunkFileSpec_Ptr parser)
647+ , (attrCacheFileName, ThunkFileSpec_AttrCache)
648+ , (normalise $ unpackedDirName </> ".git", ThunkFileSpec_CheckoutIndicator)
649+ ]
650+ where
651+ defaultNixViaSrc = [here|
652+# DO NOT HAND-EDIT THIS FILE
653+import (import ./thunk.nix)
654+|]
655+
469656
470−--TODO: when checking something out, make a shallow clone
657+parseJsonObject :: (Aeson.Object -> Aeson.Parser a) -> LBS.ByteString -> Either String a
658+parseJsonObject p bytes = Aeson.parseEither p =<< Aeson.eitherDecode bytes
471659
472660-- | Checks a cache directory to see if there is a fresh symlink
473661-- to the result of building an attribute of a thunk.
@@ -475,44 +663,49 @@ thunkFileNames = ["default.nix", "github.json", "git.json"]
475663-- and the result is symlinked into the cache.
476664nixBuildThunkAttrWithCache
477665 :: MonadObelisk m
478− => FilePath
666+ => ThunkSpec
667+ -> FilePath
479668 -- ^ Path to directory containing Thunk
480669 -> String
481670 -- ^ Attribute to build
482− -> m FilePath
671+ -> m (Maybe FilePath)
483672 -- ^ Symlink to cached or built nix output
484673-- WARNING: If the thunk uses an impure reference such as '<nixpkgs>'
485674-- the caching mechanism will fail as it merely measures the modification
486675-- time of the cache link and the expression to build.
487−nixBuildThunkAttrWithCache thunkDir attr = do
488− --NB: Expects thunkDir to be normalised with no trailing path separator.
489− --This should be guaranteed by the command argument parser.
490− let cacheErrHandler e
491− | isDoesNotExistError e = return Nothing -- expected from a cache miss
492− | otherwise = putLog Error (T.pack $ displayException e) >> return Nothing
493− cacheDir = thunkDir </> ".attr-cache"
494− cachePath = cacheDir </> attr <.> "out"
676+nixBuildThunkAttrWithCache thunkSpec thunkDir attr = do
495677 latestChange <- liftIO $ do
496− createDirectoryIfMissing False cacheDir
497− let getModificationTimeMaybe = fmap rightToMaybe . try @IOError . getModificationTime
498− maximum . catMaybes <$> mapM (getModificationTimeMaybe . (thunkDir </>)) thunkFileNames
499− cacheHit <- handle cacheErrHandler $ do
500− cacheTime <- liftIO $ posixSecondsToUTCTime . realToFrac . modificationTime <$> getSymbolicLinkStatus cachePath
501− return $ if latestChange <= cacheTime
502− then Just cachePath
503− else Nothing
504− case cacheHit of
505− Just c -> return c
506− Nothing -> do
507− putLog Warning $ T.pack $ mconcat [thunkDir, ": ", attr, " not cached, building ..."]
508− _ <- nixCmd $ NixCmd_Build$ def
509− Lens.& nixBuildConfig_outLink Lens..~ OutLink_IndirectRoot cachePath
510− Lens.& nixCmdConfig_target Lens..~ Target
511− { _target_path = Just thunkDir
512− , _target_attr = Just attr
513− , _target_expr = Nothing
514− }
515− return cachePath
678+ let
679+ getModificationTimeMaybe = fmap rightToMaybe . try @IOError . getModificationTime
680+ thunkFileNames = Map.keys $ _thunkSpec_files thunkSpec
681+ maximum . catMaybes <$> traverse (getModificationTimeMaybe . (thunkDir </>)) thunkFileNames
682+
683+ let cachePaths' = nonEmpty $ Map.keys $ Map.filter (\case ThunkFileSpec_AttrCache -> True; _ -> False) $
684+ _thunkSpec_files thunkSpec
685+ for cachePaths' $ \cachePaths ->
686+ fmap NonEmpty.head $ for cachePaths $ \cacheDir -> do
687+ let
688+ cachePath = thunkDir </> cacheDir </> attr <.> "out"
689+ cacheErrHandler e
690+ | isDoesNotExistError e = pure Nothing -- expected from a cache miss
691+ | otherwise = Nothing <$ putLog Error (T.pack $ displayException e)
692+ cacheHit <- handle cacheErrHandler $ do
693+ cacheTime <- liftIO $ posixSecondsToUTCTime . realToFrac . modificationTime <$> getSymbolicLinkStatus cachePath
694+ pure $ if latestChange <= cacheTime
695+ then Just cachePath
696+ else Nothing
697+ case cacheHit of
698+ Just c -> pure c
699+ Nothing -> do
700+ putLog Warning $ T.pack $ mconcat [thunkDir, ": ", attr, " not cached, building ..."]
701+ liftIO $ createDirectoryIfMissing True (takeDirectory cachePath)
702+ (cachePath <$) $ nixCmd $ NixCmd_Build $ def
703+ & nixBuildConfig_outLink .~ OutLink_IndirectRoot cachePath
704+ & nixCmdConfig_target .~ Target
705+ { _target_path = Just thunkDir
706+ , _target_attr = Just attr
707+ , _target_expr = Nothing
708+ }
516709
517710-- | Build a nix attribute, and cache the result if possible
518711nixBuildAttrWithCache
@@ -523,13 +716,15 @@ nixBuildAttrWithCache
523716 -- ^ Attribute to build
524717 -> m FilePath
525718 -- ^ Symlink to cached or built nix output
526−nixBuildAttrWithCache exprPath attr = do
527− readThunk exprPath >>= \case
528− -- Only packed thunks are cached. in particular, checkouts are not
529− Right (ThunkData_Packed _) -> nixBuildThunkAttrWithCache exprPath attr
530− _ -> nixCmd $ NixCmd_Build $ def
531− Lens.& nixBuildConfig_outLink Lens..~ OutLink_None
532− Lens.& nixCmdConfig_target Lens..~ Target
719+nixBuildAttrWithCache exprPath attr = readThunk exprPath >>= \case
720+ -- Only packed thunks are cached. In particular, checkouts are not.
721+ Right (ThunkData_Packed spec _) ->
722+ maybe build pure =<< nixBuildThunkAttrWithCache spec exprPath attr
723+ _ -> build
724+ where
725+ build = nixCmd $ NixCmd_Build $ def
726+ & nixBuildConfig_outLink .~ OutLink_None
727+ & nixCmdConfig_target .~ Target
533728 { _target_path = Just exprPath
534729 , _target_attr = Just attr
535730 , _target_expr = Nothing
@@ -550,101 +745,153 @@ updateThunk p f = withSystemTempDirectory "obelisk-thunkptr-" $ \tmpDir -> do
550745 where
551746 copyThunkToTmp tmpDir thunkDir = readThunk thunkDir >>= \case
552747 Left err -> failWith $ "withThunkUnpacked: " <> T.pack (show err)
553− Right (ThunkData_Packed _) -> do
748+ Right ThunkData_Packed{} -> do
554749 let tmpThunk = tmpDir </> "thunk"
555750 callProcessAndLogOutput (Notice, Error) $
556− proc "cp" ["-r", "-T", thunkDir, tmpThunk]
751+ proc cp ["-r", "-T", thunkDir, tmpThunk]
557752 return tmpThunk
558− Right _ -> failWith $ "Thunk is not packed"
753+ Right _ -> failWith "Thunk is not packed"
559754 updateThunkFromTmp p' = do
560− _ <- packThunk' True p'
755+ _ <- packThunk' True (ThunkPackConfig False (ThunkConfig Nothing)) p'
561756 callProcessAndLogOutput (Notice, Error) $
562− proc "cp" ["-r", "-T", p', p]
757+ proc cp ["-r", "-T", p', p]
563758
564759finalMsg :: Bool -> (a -> Text) -> Maybe (a -> Text)
565760finalMsg noTrail s = if noTrail then Nothing else Just s
566761
567−unpackThunk :: MonadObelisk m => FilePath -> m ()
568−unpackThunk = unpackThunk' False
569−
570762-- | Check that we are not somewhere inside the thunk directory
571763checkThunkDirectory :: MonadObelisk m => FilePath -> m ()
572764checkThunkDirectory thunkDir = do
573765 currentDir <- liftIO getCurrentDirectory
574766 thunkDir' <- liftIO $ canonicalizePath thunkDir
575767 when (thunkDir' `L.isInfixOf` currentDir) $
576− failWith "Can't pack/unpack from within the thunk directory"
768+ failWith [i|Can't perform thunk operations from within the thunk directory: ${thunkDir}|]
769+
770+ -- Don't let thunk commands work when directly given an unpacked repo
771+ when (takeFileName thunkDir == unpackedDirName) $
772+ readThunk (takeDirectory thunkDir) >>= \case
773+ Right _ -> failWith [i|Refusing to perform thunk operation on ${thunkDir} because it is a thunk's unpacked source|]
774+ Left _ -> pure ()
775+
776+unpackThunk :: MonadObelisk m => FilePath -> m ()
777+unpackThunk = unpackThunk' False
577778
578779unpackThunk' :: MonadObelisk m => Bool -> FilePath -> m ()
579−unpackThunk' noTrail thunkDir = checkThunkDirectory thunkDir >> readThunk thunkDir >>= \case
580− Left err -> failWith $ "thunk unpack: " <> T.pack (show err)
780+unpackThunk' noTrail thunkDir = checkThunkDirectory thunkDir *> readThunk thunkDir >>= \case
781+ Left err -> failWith [i|Invalid thunk at ${thunkDir}: ${err}|]
581782 --TODO: Overwrite option that rechecks out thunk; force option to do so even if working directory is dirty
582− Right (ThunkData_Checkout _) -> failWith "thunk unpack: thunk is already unpacked"
583− Right (ThunkData_Packed tptr) -> do
783+ Right ThunkData_Checkout -> failWith [i|Thunk at ${thunkDir} is already unpacked|]
784+ Right (ThunkData_Packed _ tptr) -> do
584785 let (thunkParent, thunkName) = splitFileName thunkDir
585− withTempDirectory thunkParent thunkName $ \tmpRepo -> do
586− let obGitDir = tmpRepo </> ".git" </> "obelisk"
587− s = case _thunkPtr_source tptr of
588− ThunkSource_GitHub s' -> forgetGithub False s'
589− ThunkSource_Git s' -> s'
786+ withTempDirectory thunkParent thunkName $ \tmpThunk -> do
787+ let
788+ gitSrc = thunkSourceToGitSource $ _thunkPtr_source tptr
789+ newSpec = case _thunkPtr_source tptr of
790+ ThunkSource_GitHub _ -> NonEmpty.head gitHubThunkSpecs
791+ ThunkSource_Git _ -> NonEmpty.head gitThunkSpecs
590792 withSpinner' ("Fetching thunk " <> T.pack thunkName)
591793 (finalMsg noTrail $ const $ "Fetched thunk " <> T.pack thunkName) $ do
592− let git = callProcessAndLogOutput (Notice, Notice) . gitProc tmpRepo
593− git $ [ "clone" ]
594− ++ ("--recursive" <$ guard (_gitSource_fetchSubmodules s))
595− ++ [ T.unpack $ render $ _gitSource_url s ]
596− ++ do branch <- maybeToList $ _gitSource_branch s
597− [ "--branch", T.unpack $ untagName branch ]
598− git ["reset", "--hard", Ref.toHexString $ _thunkRev_commit $ _thunkPtr_rev tptr]
599− when (_gitSource_fetchSubmodules s) $
600− git ["submodule", "update", "--recursive", "--init"]
601−
602− liftIO $ createDirectory obGitDir
603− callProcessAndLogOutput (Notice, Error) $
604− proc "cp" ["-r", "-T", thunkDir </> ".", obGitDir </> "orig-thunk"]
605− callProcessAndLogOutput (Notice, Error) $
606− proc "rm" ["-r", thunkDir]
607− callProcessAndLogOutput (Notice, Error) $
608− proc "mv" ["-T", tmpRepo, thunkDir]
794+ let unpackedPath = tmpThunk </> unpackedDirName
795+ gitCloneForThunkUnpack gitSrc (_thunkRev_commit $ _thunkPtr_rev tptr) unpackedPath
796+
797+ let normalizeMore = dropTrailingPathSeparator . normalise
798+ when (normalizeMore unpackedPath /= normalizeMore tmpThunk) $ -- Only write meta data if the checkout is not inplace
799+ createThunk tmpThunk $ Left newSpec
800+
801+ liftIO $ do
802+ removePathForcibly thunkDir
803+ renameDirectory tmpThunk thunkDir
804+
805+gitCloneForThunkUnpack
806+ :: MonadObelisk m
807+ => GitSource -- ^ Git source to use
808+ -> Ref hash -- ^ Commit hash to reset to
809+ -> FilePath -- ^ Directory to clone into
810+ -> m ()
811+gitCloneForThunkUnpack gitSrc commit dir = do
812+ let git = callProcessAndLogOutput (Notice, Notice) . gitProc dir
813+ git $ [ "clone" ]
814+ ++ ["--recursive" | _gitSource_fetchSubmodules gitSrc]
815+ ++ [ T.unpack $ gitUriToText $ _gitSource_url gitSrc ]
816+ ++ do branch <- maybeToList $ _gitSource_branch gitSrc
817+ [ "--branch", T.unpack $ untagName branch ]
818+ git ["reset", "--hard", Ref.toHexString commit]
819+ when (_gitSource_fetchSubmodules gitSrc) $
820+ git ["submodule", "update", "--recursive", "--init"]
609821
610−--TODO: add force mode to pack even if changes are present
611822--TODO: add a rollback mode to pack to the original thunk
612−packThunk :: MonadObelisk m => FilePath -> m ThunkPtr
823+packThunk :: MonadObelisk m => ThunkPackConfig -> FilePath -> m ThunkPtr
613824packThunk = packThunk' False
614825
615−packThunk' :: MonadObelisk m => Bool -> FilePath -> m ThunkPtr
616−packThunk' noTrail thunkDir = checkThunkDirectory thunkDir >> readThunk thunkDir >>= \case
617− Left err -> failWith $ T.pack $ "thunk pack: " <> show err
618− Right (ThunkData_Packed _) -> failWith "pack: thunk is already packed"
619− Right (ThunkData_Checkout _) -> do
620− withSpinner' ("Packing thunk " <> T.pack thunkDir)
621− (finalMsg noTrail $ const $ "Packed thunk " <> T.pack thunkDir) $ do
622− thunkPtr <- getThunkPtr thunkDir
623− callProcessAndLogOutput (Debug, Error) $ proc "rm" ["-rf", thunkDir]
624− liftIO $ createThunk thunkDir thunkPtr
826+packThunk' :: MonadObelisk m => Bool -> ThunkPackConfig -> FilePath -> m ThunkPtr
827+packThunk' noTrail (ThunkPackConfig force thunkConfig) thunkDir = checkThunkDirectory thunkDir *> readThunk thunkDir >>= \case
828+ Right ThunkData_Packed{} -> failWith [i|Thunk at ${thunkDir} is is already packed|]
829+ _ -> withSpinner'
830+ ("Packing thunk " <> T.pack thunkDir)
831+ (finalMsg noTrail $ const $ "Packed thunk " <> T.pack thunkDir) $
832+ do
833+ let checkClean = if force then CheckClean_NoCheck else CheckClean_FullCheck
834+ thunkPtr <- modifyThunkPtrByConfig thunkConfig <$> getThunkPtr checkClean thunkDir (_thunkConfig_private thunkConfig)
835+ liftIO $ removePathForcibly thunkDir
836+ createThunk thunkDir $ Right thunkPtr
625837 pure thunkPtr
626838
627−getThunkPtr :: MonadObelisk m => FilePath -> m ThunkPtr
628−getThunkPtr = getThunkPtr' True
839+modifyThunkPtrByConfig :: ThunkConfig -> ThunkPtr -> ThunkPtr
840+modifyThunkPtrByConfig (ThunkConfig markPrivate') ptr = case markPrivate' of
841+ Nothing -> ptr
842+ Just markPrivate -> ptr { _thunkPtr_source = case _thunkPtr_source ptr of
843+ ThunkSource_Git s -> ThunkSource_Git $ s { _gitSource_private = markPrivate }
844+ ThunkSource_GitHub s -> ThunkSource_GitHub $ s { _gitHubSource_private = markPrivate }
845+ }
629846
630−getThunkPtr' :: forall m. MonadObelisk m => Bool -> FilePath -> m ThunkPtr
631−getThunkPtr' checkClean thunkDir = do
632− when checkClean $ ensureCleanGitRepo thunkDir True $
847+data CheckClean
848+ = CheckClean_FullCheck
849+ -- ^ Check that the repo is clean, including .gitignored files
850+ | CheckClean_NotIgnored
851+ -- ^ Check that the repo is clean, not including .gitignored files
852+ | CheckClean_NoCheck
853+ -- ^ Don't check that the repo is clean
854+
855+getThunkPtr :: forall m. MonadObelisk m => CheckClean -> FilePath -> Maybe Bool -> m ThunkPtr
856+getThunkPtr gitCheckClean dir mPrivate = do
857+ let repoLocations = nubOrd $ map (first normalise)
858+ [(".git", "."), (unpackedDirName </> ".git", unpackedDirName)]
859+ repoLocation' <- liftIO $ flip findM repoLocations $ doesDirectoryExist . (dir </>) . fst
860+ thunkDir <- case repoLocation' of
861+ Nothing -> failWith [i|Can't find an unpacked thunk in ${dir}|]
862+ Just (_, path) -> pure $ normalise $ dir </> path
863+
864+ let (checkClean, checkIgnored) = case gitCheckClean of
865+ CheckClean_FullCheck -> (True, True)
866+ CheckClean_NotIgnored -> (True, False)
867+ CheckClean_NoCheck -> (False, False)
868+ when checkClean $ ensureCleanGitRepo thunkDir checkIgnored
633869 "thunk pack: thunk checkout contains unsaved modifications"
634870
635871 -- Check whether there are any stashes
636− stashOutput <- readGitProcess thunkDir ["stash", "list"]
637− when checkClean $ case T.null stashOutput of
638− False -> do
872+ when checkClean $ do
873+ stashOutput <- readGitProcess thunkDir ["stash", "list"]
874+ unless (T.null stashOutput) $
639875 failWith $ T.unlines $
640876 [ "thunk pack: thunk checkout has stashes"
641877 , "git stash list:"
642878 ] ++ T.lines stashOutput
643− True -> return ()
879+
880+ -- Get current branch
881+ (mCurrentBranch, mCurrentCommit) <- do
882+ b <- listToMaybe . T.lines <$> readGitProcess thunkDir ["rev-parse", "--abbrev-ref", "HEAD"]
883+ c <- listToMaybe . T.lines <$> readGitProcess thunkDir ["rev-parse", "HEAD"]
884+ case b of
885+ (Just "HEAD") -> failWith $ T.unlines
886+ [ "thunk pack: You are in 'detached HEAD' state."
887+ , "If you want to pack at the current ref \
888+ \then please create a new branch with 'git checkout -b <new-branch-name>' and push this upstream."
889+ ]
890+ _ -> return (b, c)
644891
645892 -- Get information on all branches and their (optional) designated upstream
646893 -- correspondents
647− (headDump :: [Text]) <- T.lines <$> readGitProcess thunkDir
894+ headDump :: [Text] <- T.lines <$> readGitProcess thunkDir
648895 [ "for-each-ref"
649896 , "--format=%(refname:short) %(upstream:short) %(upstream:remotename)"
650897 , "refs/heads/"
@@ -690,7 +937,7 @@ getThunkPtr' checkClean thunkDir = do
690937 ]
691938
692939 -- loosely by https://stackoverflow.com/questions/7773939/show-git-ahead-and-behind-info-for-all-branches-including-remotes
693− stats <- iforM headUpstream $ \branch (upstream, _remote) -> do
940+ stats <- ifor headUpstream $ \branch (upstream, _remote) -> do
694941 (stat :: [Text]) <- T.lines <$> readGitProcess thunkDir
695942 [ "rev-list", "--left-right"
696943 , T.unpack branch <> "..." <> T.unpack upstream
@@ -707,20 +954,16 @@ getThunkPtr' checkClean thunkDir = do
707954 [ "thunk pack: Certain branches in the thunk have commits not yet pushed upstream:"
708955 , ""
709956 ] ++
710− (flip map (Map.toList nonGood) $ \(branch, (upstream, (ahead, behind))) -> mconcat
957+ flip map (Map.toList nonGood) (\(branch, (upstream, (ahead, behind))) -> mconcat
711958 [" ", branch, " ahead: ", T.pack (show ahead), " behind: ", T.pack (show behind), " remote branch ", upstream]) ++
712959 [ ""
713960 , "Please push these upstream and try again. (Or just fetch, if they are somehow \
714961 \pushed but this repo's remote tracking branches don't know it.)"
715962 ]
716963
717− -- We assume it's safe to pack the thunk at this point
718− putLog Informational $ "All changes safe in git remotes. OK to pack thunk."
719−
720− -- Get current branch ``
721− mCurrentBranch <- listToMaybe
722− <$> T.lines
723− <$> readGitProcess thunkDir ["rev-parse", "--abbrev-ref", "HEAD"]
964+ when checkClean $ do
965+ -- We assume it's safe to pack the thunk at this point
966+ putLog Informational "All changes safe in git remotes. OK to pack thunk."
724967
725968 let remote = maybe "origin" snd $ flip Map.lookup headUpstream =<< mCurrentBranch
726969
@@ -733,14 +976,12 @@ getThunkPtr' checkClean thunkDir = do
733976 remoteUri <- case parseGitUri remoteUri' of
734977 Nothing -> failWith $ "Could not identify git remote: " <> remoteUri'
735978 Just uri -> pure uri
736− uriThunkPtr remoteUri mCurrentBranch
979+ uriThunkPtr remoteUri mPrivate mCurrentBranch mCurrentCommit
737980
738981-- | Get the latest revision available from the given source
739982getLatestRev :: MonadObelisk m => ThunkSource -> m ThunkRev
740983getLatestRev os = do
741− let gitS = case os of
742− ThunkSource_GitHub s -> forgetGithub False s
743− ThunkSource_Git s -> s
984+ let gitS = thunkSourceToGitSource os
744985 (_, commit) <- gitGetCommitBranch (_gitSource_url gitS) (untagName <$> _gitSource_branch gitS)
745986 case os of
746987 ThunkSource_GitHub s -> githubThunkRev s commit
@@ -752,16 +993,18 @@ getLatestRev os = do
752993-- performance. If that doesn't work (e.g. authentication issue), we fall back
753994-- on just doing things the normal way for git repos in general, and save it as
754995-- a regular git thunk.
755−uriThunkPtr :: MonadObelisk m => URI -> Maybe Text -> m ThunkPtr
756−uriThunkPtr uri mbranch = do
757− (_, commit) <- gitGetCommitBranch uri mbranch
758− (src, rev) <- case uriToThunkSource uri mbranch of
996+uriThunkPtr :: MonadObelisk m => GitUri -> Maybe Bool -> Maybe Text -> Maybe Text -> m ThunkPtr
997+uriThunkPtr uri mPrivate mbranch mcommit = do
998+ commit <- case mcommit of
999+ Nothing -> gitGetCommitBranch uri mbranch >>= return . snd
1000+ (Just c) -> return c
1001+ (src, rev) <- uriToThunkSource uri mPrivate mbranch >>= \case
7591002 ThunkSource_GitHub s -> do
7601003 rev <- runExceptT $ githubThunkRev s commit
7611004 case rev of
7621005 Right r -> pure (ThunkSource_GitHub s, r)
7631006 Left e -> do
764− putLog Warning $ "\
1007+ putLog Warning "\
7651008\Failed to fetch archive from GitHub. This is probably a private repo. \
7661009\Falling back on normal fetchgit. Original failure:"
7671010 errorToWarning e
@@ -778,34 +1021,66 @@ uriThunkPtr uri mbranch = do
7781021-- If the thunk is a GitHub thunk and fails, we do *not* fall back like with
7791022-- `uriThunkPtr`. Unlike a plain URL, a thunk src explicitly states which method
7801023-- should be employed, and so we respect that.
781−uriToThunkSource :: URI -> Maybe Text -> ThunkSource
782−uriToThunkSource u
783− | Right uriAuth <- uriAuthority u
784− , Just scheme <- unRText <$> uriScheme u
1024+uriToThunkSource :: MonadObelisk m => GitUri -> Maybe Bool -> Maybe Text -> m ThunkSource
1025+uriToThunkSource (GitUri u) mPrivate
1026+ | Right uriAuth <- URI.uriAuthority u
1027+ , Just scheme <- URI.unRText <$> URI.uriScheme u
7851028 , case scheme of
786− "ssh" -> uriAuth == Authority
787− { authUserInfo = Just $ UserInfo (fromRight' $ mkUsername "git") Nothing
788− , authHost = fromRight' $ mkHost "github.com"
789− , authPort = Nothing
1029+ "ssh" -> uriAuth == URI.Authority
1030+ { URI.authUserInfo = Just $ URI.UserInfo (fromRight' $ URI.mkUsername "git") Nothing
1031+ , URI.authHost = fromRight' $ URI.mkHost "github.com"
1032+ , URI.authPort = Nothing
7901033 }
7911034 s -> s `L.elem` [ "git", "https", "http" ] -- "http:" just redirects to "https:"
792− && unRText (authHost uriAuth) == "github.com"
793− , Just (_, owner :| [repoish]) <- uriPath u
794− = \mbranch -> ThunkSource_GitHub $ GitHubSource
795− { _gitHubSource_owner = N $ unRText owner
796− , _gitHubSource_repo = N $ let
797− repoish' = unRText repoish
798− in case T.stripSuffix ".git" repoish' of
799− Just repo -> repo
800− Nothing -> repoish'
801− , _gitHubSource_branch = N <$> mbranch
802− }
1035+ && URI.unRText (URI.authHost uriAuth) == "github.com"
1036+ , Just (_, owner :| [repoish]) <- URI.uriPath u
1037+ = \mbranch -> do
1038+ isPrivate <- getIsPrivate
1039+ pure $ ThunkSource_GitHub $ GitHubSource
1040+ { _gitHubSource_owner = N $ URI.unRText owner
1041+ , _gitHubSource_repo = N $ let
1042+ repoish' = URI.unRText repoish
1043+ in fromMaybe repoish' $ T.stripSuffix ".git" repoish'
1044+ , _gitHubSource_branch = N <$> mbranch
1045+ , _gitHubSource_private = isPrivate
1046+ }
8031047
804− | otherwise = \mbranch -> ThunkSource_Git $ GitSource
805− { _gitSource_url = u
806− , _gitSource_branch = N <$> mbranch
807− , _gitSource_fetchSubmodules = False -- TODO: How do we determine if this should be true?
808− }
1048+ | otherwise = \mbranch -> do
1049+ isPrivate <- getIsPrivate
1050+ pure $ ThunkSource_Git $ GitSource
1051+ { _gitSource_url = GitUri u
1052+ , _gitSource_branch = N <$> mbranch
1053+ , _gitSource_fetchSubmodules = False -- TODO: How do we determine if this should be true?
1054+ , _gitSource_private = isPrivate
1055+ }
1056+ where
1057+ getIsPrivate = maybe (guessGitRepoIsPrivate $ GitUri u) pure mPrivate
1058+
1059+guessGitRepoIsPrivate :: MonadObelisk m => GitUri -> m Bool
1060+guessGitRepoIsPrivate uri = flip fix urisToTry $ \loop -> \case
1061+ [] -> pure True
1062+ uriAttempt:xs -> do
1063+ result <- readCreateProcessWithExitCode $
1064+ isolateGitProc $
1065+ gitProcNoRepo
1066+ [ "ls-remote"
1067+ , "--quiet"
1068+ , "--exit-code"
1069+ , "--symref"
1070+ , T.unpack $ gitUriToText uriAttempt
1071+ ]
1072+ case result of
1073+ (ExitSuccess, _, _) -> pure False -- Must be a public repo
1074+ _ -> loop xs
1075+ where
1076+ urisToTry = nubOrd $
1077+ -- Include the original URI if it isn't using SSH because SSH will certainly fail.
1078+ [uri | fmap URI.unRText (URI.uriScheme (unGitUri uri)) /= Just "ssh"] <>
1079+ [changeScheme "https" uri, changeScheme "http" uri]
1080+ changeScheme scheme (GitUri u) = GitUri $ u
1081+ { URI.uriScheme = URI.mkScheme scheme
1082+ , URI.uriAuthority = (\x -> x { URI.authUserInfo = Nothing }) <$> URI.uriAuthority u
1083+ }
8091084
8101085-- Funny signature indicates no effects depend on the optional branch name.
8111086githubThunkRev
@@ -817,19 +1092,19 @@ githubThunkRev
8171092githubThunkRev s commit = do
8181093 owner <- forcePP $ _gitHubSource_owner s
8191094 repo <- forcePP $ _gitHubSource_repo s
820− revTarball <- mkPathPiece $ commit <> ".tar.gz"
821− let archiveUri = URI
822− { uriScheme = Just $ fromRight' $ mkScheme "https"
823− , uriAuthority = Right $ Authority
824− { authUserInfo = Nothing
825− , authHost = fromRight' $ mkHost "github.com"
826− , authPort = Nothing
1095+ revTarball <- URI.mkPathPiece $ commit <> ".tar.gz"
1096+ let archiveUri = GitUri $ URI.URI
1097+ { URI.uriScheme = Just $ fromRight' $ URI.mkScheme "https"
1098+ , URI.uriAuthority = Right $ URI.Authority
1099+ { URI.authUserInfo = Nothing
1100+ , URI.authHost = fromRight' $ URI.mkHost "github.com"
1101+ , URI.authPort = Nothing
8271102 }
828− , uriPath = Just ( False
829− , owner :| [ repo, fromRight' $ mkPathPiece "archive", revTarball ]
1103+ , URI.uriPath = Just ( False
1104+ , owner :| [ repo, fromRight' $ URI.mkPathPiece "archive", revTarball ]
8301105 )
831− , uriQuery = []
832− , uriFragment = Nothing
1106+ , URI.uriQuery = []
1107+ , URI.uriFragment = Nothing
8331108 }
8341109 hash <- getNixSha256ForUriUnpacked archiveUri
8351110 putLog Debug $ "Nix sha256 is " <> hash
@@ -838,8 +1113,8 @@ githubThunkRev s commit = do
8381113 , _thunkRev_nixSha256 = hash
8391114 }
8401115 where
841− forcePP :: Name entity -> m (RText 'PathPiece)
842− forcePP = mkPathPiece . untagName
1116+ forcePP :: Name entity -> m (URI.RText 'URI.PathPiece)
1117+ forcePP = URI.mkPathPiece . untagName
8431118
8441119gitThunkRev
8451120 :: MonadObelisk m
@@ -848,11 +1123,11 @@ gitThunkRev
8481123 -> m ThunkRev
8491124gitThunkRev s commit = do
8501125 let u = _gitSource_url s
851− protocols = ["https", "ssh", "git"]
852− Just scheme <- pure $ unRText <$> uriScheme u
1126+ protocols = ["file", "https", "ssh", "git"]
1127+ scheme = maybe "file" URI.unRText $ URI.uriScheme $ (\(GitUri x) -> x) u
8531128 unless (T.toLower scheme `elem` protocols) $
8541129 failWith $ "obelisk currently only supports "
855− <> T.intercalate ", " protocols <> " protocols for non-GitHub remotes"
1130+ <> T.intercalate ", " protocols <> " protocols for plain Git remotes"
8561131 hash <- nixPrefetchGit u commit $ _gitSource_fetchSubmodules s
8571132 putLog Informational $ "Nix sha256 is " <> hash
8581133 pure $ ThunkRev
@@ -865,12 +1140,14 @@ gitThunkRev s commit = do
8651140--
8661141-- If the branch name is passed in, it is returned exactly as-is. If it is not
8671142-- passed it, the default branch of the repo is used instead.
1143+
8681144gitGetCommitBranch
869− :: MonadObelisk m => URI -> Maybe Text -> m (Text, CommitId)
1145+ :: MonadObelisk m => GitUri -> Maybe Text -> m (Text, CommitId)
8701146gitGetCommitBranch uri mbranch = withExitFailMessage ("Failure for git remote " <> uriMsg) $ do
871− bothMaps <- gitLsRemote
872− (T.unpack $ render uri)
1147+ (_, bothMaps) <- gitLsRemote
1148+ (T.unpack $ gitUriToText uri)
8731149 (GitRef_Branch <$> mbranch)
1150+ Nothing
8741151 branch <- case mbranch of
8751152 Nothing -> withExitFailMessage "Failed to find default branch" $ do
8761153 b <- rethrowE $ gitLookupDefaultBranch bothMaps
@@ -884,18 +1161,21 @@ gitGetCommitBranch uri mbranch = withExitFailMessage ("Failure for git remote "
8841161 pure (branch, commit)
8851162 where
8861163 rethrowE = either failWith pure
887− uriMsg = render uri
1164+ uriMsg = gitUriToText uri
1165+
1166+parseGitUri :: Text -> Maybe GitUri
1167+parseGitUri x = GitUri <$> (parseFileURI x <|> parseAbsoluteURI x <|> parseSshShorthand x)
8881168
889−parseGitUri :: Text -> Maybe URI
890−parseGitUri x = parseAbsoluteURI x <|> parseSshShorthand x
1169+parseFileURI :: Text -> Maybe URI.URI
1170+parseFileURI uri = if "/" `T.isPrefixOf` uri then parseAbsoluteURI ("file://" <> uri) else Nothing
8911171
892−parseAbsoluteURI :: Text -> Maybe URI
1172+parseAbsoluteURI :: Text -> Maybe URI.URI
8931173parseAbsoluteURI uri = do
894− parsedUri <- mkURI uri
895− guard $ isPathAbsolute parsedUri
896− pure $ parsedUri
1174+ parsedUri <- URI.mkURI uri
1175+ guard $ URI.isPathAbsolute parsedUri
1176+ pure parsedUri
8971177
898−parseSshShorthand :: Text -> Maybe URI
1178+parseSshShorthand :: Text -> Maybe URI.URI
8991179parseSshShorthand uri = do
9001180 -- This is what git does to check that the remote
9011181 -- is not a local file path when parsing shorthand.
@@ -908,4 +1188,4 @@ parseSshShorthand uri = do
9081188 -- This check is used to disambiguate a filepath containing a colon from shorthand
9091189 guard $ isNothing (T.findIndex (=='/') authAndHostname)
9101190 && not (T.null colonAndPath)
911− mkURI properUri
1191+ URI.mkURI properUri