@@ -5,59 +5,72 @@
55{-# LANGUAGE MultiWayIf #-}
66{-# LANGUAGE OverloadedStrings #-}
77{-# LANGUAGE PatternGuards #-}
8−{-# LANGUAGE QuasiQuotes #-}
98{-# LANGUAGE ScopedTypeVariables #-}
109{-# LANGUAGE TypeApplications #-}
11−module Obelisk.Command.Thunk where
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
1233
1334import Control.Applicative
1435import Control.Exception (displayException, try)
15−import Control.Lens (ifor, ifor_, (.~))
36+import qualified Control.Lens as Lens
37+import Control.Lens.Indexed hiding ((<.>))
1638import Control.Monad
17−import Control.Monad.Extra (findM)
18−import Control.Monad.Catch (MonadCatch, handle)
39+import Control.Monad.Catch (handle)
1940import Control.Monad.Except
2041import Data.Aeson ((.=))
2142import qualified Data.Aeson as Aeson
2243import Data.Aeson.Encode.Pretty
2344import qualified Data.Aeson.Types as Aeson
24−import Data.Bifunctor (first)
2545import qualified Data.ByteString.Lazy as LBS
26−import Data.Containers.ListUtils (nubOrd)
2746import Data.Default
2847import Data.Either.Combinators (fromRight', rightToMaybe)
29−import Data.Foldable (toList)
30−import Data.Function ((&))
31−import Data.Functor ((<&>))
3248import Data.Git.Ref (Ref)
3349import qualified Data.Git.Ref as Ref
3450import qualified Data.List as L
35−import Data.List.NonEmpty (NonEmpty (..), nonEmpty)
51+import Data.List.NonEmpty (NonEmpty (..))
3652import qualified Data.List.NonEmpty as NonEmpty
3753import Data.Map (Map)
3854import qualified Data.Map as Map
3955import Data.Maybe
4056import Data.Set (Set)
4157import qualified Data.Set as Set
42−import Data.String.Here.Interpolated (i)
43−import Data.String.Here.Uninterpolated (here)
4458import Data.Text (Text)
4559import qualified Data.Text as T
4660import Data.Text.Encoding
4761import qualified Data.Text.IO as T
4862import Data.Time.Clock.POSIX (posixSecondsToUTCTime)
49−import Data.Traversable (for)
5063import Data.Yaml (parseMaybe)
5164import GitHub
5265import GitHub.Data.Name
5366import Obelisk.Command.Nix
5467import System.Directory
55−import System.Exit
5668import System.FilePath
5769import System.IO.Error
5870import System.IO.Temp
59−import System.PosixCompat.Files (getSymbolicLinkStatus, modificationTime)
60−import qualified Text.URI as URI
71+import System.Posix (getSymbolicLinkStatus, modificationTime)
72+import System.Process (proc)
73+import Text.URI
6174
6275import Obelisk.App (MonadObelisk)
6376import Obelisk.CliApp
@@ -65,10 +78,11 @@ import Obelisk.Command.Utils
6578
6679--TODO: Support symlinked thunk data
6780data ThunkData
68− = ThunkData_Packed ThunkSpec ThunkPtr
81+ = ThunkData_Packed ThunkPtr
6982 -- ^ Packed thunk
70− | ThunkData_Checkout
83+ | ThunkData_Checkout (Maybe ThunkPtr)
7184 -- ^ Checked out thunk that was unpacked from this pointer
85+ deriving (Show, Eq, Ord)
7286
7387-- | A reference to the exact data that a thunk should translate into
7488data ThunkPtr = ThunkPtr
@@ -81,7 +95,7 @@ type NixSha256 = Text --TODO: Use a smart constructor and make this actually ver
8195
8296-- | A specific revision of data; it may be available from multiple sources
8397data ThunkRev = ThunkRev
84− { _thunkRev_commit :: Ref Ref.SHA1
98+ { _thunkRev_commit :: Ref
8599 , _thunkRev_nixSha256 :: NixSha256
86100 }
87101 deriving (Show, Eq, Ord)
@@ -94,72 +108,41 @@ data ThunkSource
94108 | ThunkSource_Git GitSource
95109 deriving (Show, Eq, Ord)
96110
97−thunkSourceToGitSource :: ThunkSource -> GitSource
98−thunkSourceToGitSource = \case
99− ThunkSource_GitHub s -> forgetGithub False s
100− ThunkSource_Git s -> s
101−
102111data GitHubSource = GitHubSource
103112 { _gitHubSource_owner :: Name Owner
104113 , _gitHubSource_repo :: Name Repo
105114 , _gitHubSource_branch :: Maybe (Name Branch)
106− , _gitHubSource_private :: Bool
107115 }
108116 deriving (Show, Eq, Ord)
109117
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−
119118data GitSource = GitSource
120− { _gitSource_url :: GitUri
119+ { _gitSource_url :: URI
121120 , _gitSource_branch :: Maybe (Name Branch)
122121 , _gitSource_fetchSubmodules :: Bool
123− , _gitSource_private :: Bool
124122 }
125123 deriving (Show, Eq, Ord)
126124
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−
141125-- | Convert a GitHub source to a regular Git source. Assumes no submodules.
142126forgetGithub :: Bool -> GitHubSource -> GitSource
143127forgetGithub useSsh s = GitSource
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
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
148132 <$ guard useSsh
149− , URI.authHost = fromRight' $ URI.mkHost "github.com"
150− , URI.authPort = Nothing
133+ , authHost = fromRight' $ mkHost "github.com"
134+ , authPort = Nothing
151135 }
152− , URI.uriPath = Just ( False
153− , fromRight' . URI.mkPathPiece <$>
136+ , uriPath = Just ( False
137+ , fromRight' . mkPathPiece <$>
154138 untagName (_gitHubSource_owner s)
155139 :| [ untagName (_gitHubSource_repo s) <> ".git" ]
156140 )
157− , URI.uriQuery = []
158− , URI.uriFragment = Nothing
141+ , uriQuery = []
142+ , uriFragment = Nothing
159143 }
160144 , _gitSource_branch = _gitHubSource_branch s
161145 , _gitSource_fetchSubmodules = False
162− , _gitSource_private = _gitHubSource_private s
163146 }
164147
165148getThunkGitBranch :: ThunkPtr -> Maybe Text
@@ -167,26 +150,26 @@ getThunkGitBranch (ThunkPtr _ src) = fmap untagName $ case src of
167150 ThunkSource_GitHub s -> _gitHubSource_branch s
168151 ThunkSource_Git s -> _gitSource_branch s
169152
170−commitNameToRef :: Name Commit -> Ref Ref.SHA1
153+commitNameToRef :: Name Commit -> Ref
171154commitNameToRef (N c) = Ref.fromHex $ encodeUtf8 c
172155
173156-- TODO: Use spinner here.
174157getNixSha256ForUriUnpacked
175158 :: MonadObelisk m
176− => GitUri
159+ => URI
177160 -> m NixSha256
178161getNixSha256ForUriUnpacked uri =
179− withExitFailMessage ("nix-prefetch-url: Failed to determine sha256 hash of URL " <> gitUriToText uri) $ do
162+ withExitFailMessage ("nix-prefetch-url: Failed to determine sha256 hash of URL " <> render uri) $ do
180163 [hash] <- fmap T.lines $ readProcessAndLogOutput (Debug, Debug) $
181− proc nixPrefetchUrlPath ["--unpack", "--type", "sha256", T.unpack $ gitUriToText uri]
164+ proc "nix-prefetch-url" ["--unpack" , "--type" , "sha256" , T.unpack $ render uri]
182165 pure hash
183166
184−nixPrefetchGit :: MonadObelisk m => GitUri -> Text -> Bool -> m NixSha256
167+nixPrefetchGit :: MonadObelisk m => URI -> Text -> Bool -> m NixSha256
185168nixPrefetchGit uri rev fetchSubmodules =
186− withExitFailMessage ("nix-prefetch-git: Failed to determine sha256 hash of Git repo " <> gitUriToText uri <> " at " <> rev) $ do
169+ withExitFailMessage ("nix-prefetch-git: Failed to determine sha256 hash of Git repo " <> render uri <> " at " <> rev) $ do
187170 out <- readProcessAndLogStderr Debug $
188− proc nixPrefetchGitPath $ filter (/="")
189− [ "--url", T.unpack $ gitUriToText uri
171+ proc "nix-prefetch-git" $ filter (/="")
172+ [ "--url", T.unpack $ render uri
190173 , "--rev", T.unpack rev
191174 , if fetchSubmodules then "--fetch-submodules" else ""
192175 , "--quiet"
@@ -198,117 +181,93 @@ nixPrefetchGit uri rev fetchSubmodules =
198181
199182--TODO: Pretty print these
200183data ReadThunkError
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)
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
228209 }
229210
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}|]
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
307256
308257-- | Read a thunk and validate that it is exactly a packed thunk.
309258-- If additional data is present, fail.
310−readThunk :: (MonadObelisk m) => FilePath -> m (Either ReadThunkError ThunkData)
311−readThunk = readThunkWith thunkSpecTypes
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
312271
313272parseThunkPtr :: (Aeson.Object -> Aeson.Parser ThunkSource) -> Aeson.Object -> Aeson.Parser ThunkPtr
314273parseThunkPtr parseSrc v = do
@@ -328,12 +287,10 @@ parseGitHubSource v = do
328287 owner <- v Aeson..: "owner"
329288 repo <- v Aeson..: "repo"
330289 branch <- v Aeson..:! "branch"
331− private <- v Aeson..:? "private"
332290 pure $ GitHubSource
333291 { _gitHubSource_owner = owner
334292 , _gitHubSource_repo = repo
335293 , _gitHubSource_branch = branch
336− , _gitHubSource_private = fromMaybe False private
337294 }
338295
339296parseGitSource :: Aeson.Object -> Aeson.Parser GitSource
@@ -341,29 +298,25 @@ parseGitSource v = do
341298 Just url <- parseGitUri <$> v Aeson..: "url"
342299 branch <- v Aeson..:! "branch"
343300 fetchSubmodules <- v Aeson..:! "fetchSubmodules"
344− private <- v Aeson..:? "private"
345301 pure $ GitSource
346302 { _gitSource_url = url
347303 , _gitSource_branch = branch
348304 , _gitSource_fetchSubmodules = fromMaybe False fetchSubmodules
349− , _gitSource_private = fromMaybe False private
350305 }
351306
352307overwriteThunk :: MonadObelisk m => FilePath -> ThunkPtr -> m ()
353308overwriteThunk target thunk = do
354309 -- Ensure that this directory is a valid thunk (i.e. so we aren't losing any data)
355− readThunk target >>= \case
356− Left e -> failWith [i|Invalid thunk at ${target}: ${e}|]
357− Right _ -> pure ()
310+ Right _ <- readThunk target
358311
359312 --TODO: Is there a safer way to do this overwriting?
360− liftIO $ removePathForcibly target
361− createThunk target $ Right thunk
313+ liftIO $ removeDirectoryRecursive target
314+ liftIO $ createThunk target thunk
362315
363−thunkPtrToSpec :: ThunkPtr -> ThunkSpec
364−thunkPtrToSpec thunk = case _thunkPtr_source thunk of
365− ThunkSource_GitHub _ -> NonEmpty.head gitHubThunkSpecs
366− ThunkSource_Git _ -> NonEmpty.head gitThunkSpecs
316+thunkPtrLoader :: ThunkPtr -> Text
317+thunkPtrLoader thunk = case _thunkPtr_source thunk of
318+ ThunkSource_GitHub _ -> NonEmpty.head gitHubStandaloneLoaders
319+ ThunkSource_Git _ -> NonEmpty.head plainGitStandaloneLoaders
367320
368321-- It's important that formatting be very consistent here, because
369322-- otherwise when people update thunks, their patches will be messy
@@ -375,15 +328,13 @@ encodeThunkPtrData (ThunkPtr rev src) = case src of
375328 , ("branch" .=) <$> _gitHubSource_branch s
376329 , Just $ "rev" .= Ref.toHexString (_thunkRev_commit rev)
377330 , Just $ "sha256" .= _thunkRev_nixSha256 rev
378− , Just $ "private" .= _gitHubSource_private s
379331 ]
380332 ThunkSource_Git s -> encodePretty' plainGitCfg $ Aeson.object $ catMaybes
381− [ Just $ "url" .= gitUriToText (_gitSource_url s)
333+ [ Just $ "url" .= render (_gitSource_url s)
382334 , Just $ "rev" .= Ref.toHexString (_thunkRev_commit rev)
383335 , ("branch" .=) <$> _gitSource_branch s
384336 , Just $ "sha256" .= _thunkRev_nixSha256 rev
385337 , Just $ "fetchSubmodules" .= _gitSource_fetchSubmodules s
386− , Just $ "private" .= _gitSource_private s
387338 ]
388339 where
389340 githubCfg = defConfig
@@ -404,84 +355,78 @@ encodeThunkPtrData (ThunkPtr rev src) = case src of
404355 [ "url"
405356 , "rev"
406357 , "sha256"
407− , "private"
408358 , "fetchSubmodules"
409359 ] <> compare
410360 , confTrailingNewline = True
411361 }
412362
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
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)
428372
429373createThunkWithLatest :: MonadObelisk m => FilePath -> ThunkSource -> m ()
430374createThunkWithLatest target s = do
431375 rev <- getLatestRev s
432− createThunk target $ Right $ ThunkPtr
376+ createThunk target $ ThunkPtr
433377 { _thunkPtr_source = s
434378 , _thunkPtr_rev = rev
435379 }
436380
437−updateThunkToLatest :: MonadObelisk m => ThunkUpdateConfig -> FilePath -> m ()
438−updateThunkToLatest (ThunkUpdateConfig mBranch thunkConfig) target = spinner $ do
439− checkThunkDirectory target
381+updateThunkToLatest :: MonadObelisk m => FilePath -> Maybe String -> m ()
382+updateThunkToLatest target mBranch = withSpinner' ("Updating thunk " <> T.pack target <> " to latest") (pure $ const $ "Thunk " <> T.pack target <> " updated to latest") $ do
383+ checkThunkDirectory "ob thunk update directory cannot be '.'" target
440384 -- check to see if thunk should be updated to a specific branch or just update it's current branch
441385 case mBranch of
442386 Nothing -> do
443387 (overwrite, ptr) <- readThunk target >>= \case
444− Left err -> failWith [i|Thunk update: ${err}|]
388+ Left err -> failWith $ T.pack $ "thunk update: " <> show err
445389 Right c -> case c of
446− ThunkData_Packed _ t -> return (target, t)
447− ThunkData_Checkout -> failWith "cannot update an unpacked thunk"
390+ ThunkData_Packed t -> return (target, t)
391+ ThunkData_Checkout _ -> failWith "cannot update an unpacked thunk"
448392 let src = _thunkPtr_source ptr
449393 rev <- getLatestRev src
450− overwriteThunk overwrite $ modifyThunkPtrByConfig thunkConfig $ ThunkPtr
394+ overwriteThunk overwrite $ ThunkPtr
451395 { _thunkPtr_source = src
452396 , _thunkPtr_rev = rev
453397 }
454398 Just branch -> readThunk target >>= \case
455− Left err -> failWith [i|Thunk update: ${err}|]
399+ Left err -> failWith $ T.pack $ "thunk update: " <> show err
456400 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
401+ ThunkData_Packed t -> case _thunkPtr_source t of
402+ ThunkSource_Git tsg -> setThunk target tsg branch
403+ ThunkSource_GitHub tsgh -> do
404+ let tsg = forgetGithub False tsgh
405+ setThunk target tsg branch
406+ ThunkData_Checkout _ -> failWith $ T.pack $ "thunk located at " <> (show target) <> " is unpacked. Use ob thunk pack on the desired directory and then try ob thunk update again."
407+
408+setThunk :: MonadObelisk m => FilePath -> GitSource -> String -> m ()
409+setThunk target gs branch = do
410+ newThunkPtr <- uriThunkPtr (_gitSource_url gs) (Just $ T.pack branch) Nothing
465411 overwriteThunk target newThunkPtr
466− updateThunkToLatest (ThunkUpdateConfig Nothing thunkConfig) target
412+ updateThunkToLatest target Nothing
467413
468414-- | All recognized github standalone loaders, ordered from newest to oldest.
469415-- This tool will only ever produce the newest one when it writes a thunk.
470−gitHubThunkSpecs :: NonEmpty ThunkSpec
471−gitHubThunkSpecs =
472− gitHubThunkSpecV5 :|
473− [ gitHubThunkSpecV4
474− , gitHubThunkSpecV3
475− , gitHubThunkSpecV2
476− , gitHubThunkSpecV1
416+gitHubStandaloneLoaders :: NonEmpty Text
417+gitHubStandaloneLoaders =
418+ gitHubStandaloneLoaderV2 :|
419+ [ gitHubStandaloneLoaderV3
420+ , gitHubStandaloneLoaderV1
477421 ]
478422
479−gitHubThunkSpecV1 :: ThunkSpec
480−gitHubThunkSpecV1 = legacyGitHubThunkSpec "github-v1"
481− "import ((import <nixpkgs> {}).fetchFromGitHub (builtins.fromJSON (builtins.readFile ./github.json)))"
423+gitHubStandaloneLoaderV1 :: Text
424+gitHubStandaloneLoaderV1 = T.unlines
425+ [ "import ((import <nixpkgs> {}).fetchFromGitHub (builtins.fromJSON (builtins.readFile ./github.json)))"
426+ ]
482427
483−gitHubThunkSpecV2 :: ThunkSpec
484−gitHubThunkSpecV2 = legacyGitHubThunkSpec "github-v2" $ T.unlines
428+gitHubStandaloneLoaderV2 :: Text
429+gitHubStandaloneLoaderV2 = T.unlines
485430 [ "# DO NOT HAND-EDIT THIS FILE" --TODO: Add something about how to get more info on Obelisk, etc.
486431 , "import ((import <nixpkgs> {}).fetchFromGitHub ("
487432 , " let json = builtins.fromJSON (builtins.readFile ./github.json);"
@@ -491,8 +436,8 @@ gitHubThunkSpecV2 = legacyGitHubThunkSpec "github-v2" $ T.unlines
491436 , "))"
492437 ]
493438
494−gitHubThunkSpecV3 :: ThunkSpec
495−gitHubThunkSpecV3 = legacyGitHubThunkSpec "github-v3" $ T.unlines
439+gitHubStandaloneLoaderV3 :: Text
440+gitHubStandaloneLoaderV3 = T.unlines
496441 [ "# DO NOT HAND-EDIT THIS FILE"
497442 , "let"
498443 , " fetch = { private ? false, ... }@args: if private && builtins.hasAttr \"fetchGit\" builtins"
@@ -511,62 +456,22 @@ gitHubThunkSpecV3 = legacyGitHubThunkSpec "github-v3" $ T.unlines
511456 , "in import (fetch (builtins.fromJSON (builtins.readFile ./github.json)))"
512457 ]
513458
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
459+plainGitStandaloneLoaders :: NonEmpty Text
460+plainGitStandaloneLoaders =
461+ plainGitStandaloneLoaderV1 :|
462+ [ plainGitStandaloneLoaderV2
558463 ]
559464
560−gitThunkSpecV1 :: ThunkSpec
561−gitThunkSpecV1 = legacyGitThunkSpec "git-v1" $ T.unlines
465+plainGitStandaloneLoaderV1 :: Text
466+plainGitStandaloneLoaderV1 = T.unlines
562467 [ "# DO NOT HAND-EDIT THIS FILE"
563468 , "let fetchGit = {url, rev, ref ? null, branch ? null, sha256 ? null, fetchSubmodules ? null}:"
564469 , " assert !fetchSubmodules; (import <nixpkgs> {}).fetchgit { inherit url rev sha256; };"
565470 , "in import (fetchGit (builtins.fromJSON (builtins.readFile ./git.json)))"
566471 ]
567472
568−gitThunkSpecV2 :: ThunkSpec
569−gitThunkSpecV2 = legacyGitThunkSpec "git-v2" $ T.unlines
473+plainGitStandaloneLoaderV2 :: Text
474+plainGitStandaloneLoaderV2 = T.unlines
570475 [ "# DO NOT HAND-EDIT THIS FILE"
571476 , "let fetchGit = {url, rev, ref ? null, branch ? null, sha256 ? null, fetchSubmodules ? null}:"
572477 , " if builtins.hasAttr \"fetchGit\" builtins"
@@ -575,87 +480,10 @@ gitThunkSpecV2 = legacyGitThunkSpec "git-v2" $ T.unlines
575480 , "in import (fetchGit (builtins.fromJSON (builtins.readFile ./git.json)))"
576481 ]
577482
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−
483+thunkFileNames :: [FilePath]
484+thunkFileNames = ["default.nix", "github.json", "git.json"]
656485
657−parseJsonObject :: (Aeson.Object -> Aeson.Parser a) -> LBS.ByteString -> Either String a
658−parseJsonObject p bytes = Aeson.parseEither p =<< Aeson.eitherDecode bytes
486+--TODO: when checking something out, make a shallow clone
659487
660488-- | Checks a cache directory to see if there is a fresh symlink
661489-- to the result of building an attribute of a thunk.
@@ -663,49 +491,44 @@ parseJsonObject p bytes = Aeson.parseEither p =<< Aeson.eitherDecode bytes
663491-- and the result is symlinked into the cache.
664492nixBuildThunkAttrWithCache
665493 :: MonadObelisk m
666− => ThunkSpec
667− -> FilePath
494+ => FilePath
668495 -- ^ Path to directory containing Thunk
669496 -> String
670497 -- ^ Attribute to build
671− -> m (Maybe FilePath)
498+ -> m FilePath
672499 -- ^ Symlink to cached or built nix output
673500-- WARNING: If the thunk uses an impure reference such as '<nixpkgs>'
674501-- the caching mechanism will fail as it merely measures the modification
675502-- time of the cache link and the expression to build.
676−nixBuildThunkAttrWithCache thunkSpec thunkDir attr = do
503+nixBuildThunkAttrWithCache thunkDir attr = do
504+ --NB: Expects thunkDir to be normalised with no trailing path separator.
505+ --This should be guaranteed by the command argument parser.
506+ let cacheErrHandler e
507+ | isDoesNotExistError e = return Nothing -- expected from a cache miss
508+ | otherwise = putLog Error (T.pack $ displayException e) >> return Nothing
509+ cacheDir = thunkDir </> ".attr-cache"
510+ cachePath = cacheDir </> attr <.> "out"
677511 latestChange <- liftIO $ do
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− }
512+ createDirectoryIfMissing False cacheDir
513+ let getModificationTimeMaybe = fmap rightToMaybe . try @IOError . getModificationTime
514+ maximum . catMaybes <$> mapM (getModificationTimeMaybe . (thunkDir </>)) thunkFileNames
515+ cacheHit <- handle cacheErrHandler $ do
516+ cacheTime <- liftIO $ posixSecondsToUTCTime . realToFrac . modificationTime <$> getSymbolicLinkStatus cachePath
517+ return $ if latestChange <= cacheTime
518+ then Just cachePath
519+ else Nothing
520+ case cacheHit of
521+ Just c -> return c
522+ Nothing -> do
523+ putLog Warning $ T.pack $ mconcat [thunkDir, ": ", attr, " not cached, building ..."]
524+ _ <- nixCmd $ NixCmd_Build$ def
525+ Lens.& nixBuildConfig_outLink Lens..~ OutLink_IndirectRoot cachePath
526+ Lens.& nixCmdConfig_target Lens..~ Target
527+ { _target_path = Just thunkDir
528+ , _target_attr = Just attr
529+ , _target_expr = Nothing
530+ }
531+ return cachePath
709532
710533-- | Build a nix attribute, and cache the result if possible
711534nixBuildAttrWithCache
@@ -716,15 +539,13 @@ nixBuildAttrWithCache
716539 -- ^ Attribute to build
717540 -> m FilePath
718541 -- ^ Symlink to cached or built nix output
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
542+nixBuildAttrWithCache exprPath attr = do
543+ readThunk exprPath >>= \case
544+ -- Only packed thunks are cached. in particular, checkouts are not
545+ Right (ThunkData_Packed _) -> nixBuildThunkAttrWithCache exprPath attr
546+ _ -> nixCmd $ NixCmd_Build $ def
547+ Lens.& nixBuildConfig_outLink Lens..~ OutLink_None
548+ Lens.& nixCmdConfig_target Lens..~ Target
728549 { _target_path = Just exprPath
729550 , _target_attr = Just attr
730551 , _target_expr = Nothing
@@ -745,144 +566,107 @@ updateThunk p f = withSystemTempDirectory "obelisk-thunkptr-" $ \tmpDir -> do
745566 where
746567 copyThunkToTmp tmpDir thunkDir = readThunk thunkDir >>= \case
747568 Left err -> failWith $ "withThunkUnpacked: " <> T.pack (show err)
748− Right ThunkData_Packed{} -> do
569+ Right (ThunkData_Packed _) -> do
749570 let tmpThunk = tmpDir </> "thunk"
750571 callProcessAndLogOutput (Notice, Error) $
751− proc cp ["-r", "-T", thunkDir, tmpThunk]
572+ proc "cp" ["-r", "-T", thunkDir, tmpThunk]
752573 return tmpThunk
753− Right _ -> failWith "Thunk is not packed"
574+ Right _ -> failWith $ "Thunk is not packed"
754575 updateThunkFromTmp p' = do
755− _ <- packThunk' True (ThunkPackConfig False (ThunkConfig Nothing)) p'
576+ _ <- packThunk' True p'
756577 callProcessAndLogOutput (Notice, Error) $
757− proc cp ["-r", "-T", p', p]
578+ proc "cp" ["-r", "-T", p', p]
758579
759580finalMsg :: Bool -> (a -> Text) -> Maybe (a -> Text)
760581finalMsg noTrail s = if noTrail then Nothing else Just s
761582
583+unpackThunk :: MonadObelisk m => FilePath -> m ()
584+unpackThunk = unpackThunk' False
585+
762586-- | Check that we are not somewhere inside the thunk directory
763−checkThunkDirectory :: MonadObelisk m => FilePath -> m ()
764−checkThunkDirectory thunkDir = do
587+checkThunkDirectory :: MonadObelisk m => Text -> FilePath -> m ()
588+checkThunkDirectory msg thunkDir = do
765589 currentDir <- liftIO getCurrentDirectory
766590 thunkDir' <- liftIO $ canonicalizePath thunkDir
767− when (thunkDir' `L.isInfixOf` currentDir) $
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
591+ when (thunkDir' `L.isInfixOf` currentDir) $ failWith msg
778592
779593unpackThunk' :: MonadObelisk m => Bool -> FilePath -> m ()
780−unpackThunk' noTrail thunkDir = checkThunkDirectory thunkDir *> readThunk thunkDir >>= \case
781− Left err -> failWith [i|Invalid thunk at ${thunkDir}: ${err}|]
594+unpackThunk' noTrail thunkDir = checkThunkDirectory "Can't pack/unpack from within the thunk directory" thunkDir >> readThunk thunkDir >>= \case
595+ Left err -> failWith $ "thunk unpack: " <> T.pack (show err)
782596 --TODO: Overwrite option that rechecks out thunk; force option to do so even if working directory is dirty
783− Right ThunkData_Checkout -> failWith [i|Thunk at ${thunkDir} is already unpacked|]
784− Right (ThunkData_Packed _ tptr) -> do
597+ Right (ThunkData_Checkout _) -> failWith "thunk unpack: thunk is already unpacked"
598+ Right (ThunkData_Packed tptr) -> do
785599 let (thunkParent, thunkName) = splitFileName thunkDir
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
600+ withTempDirectory thunkParent thunkName $ \tmpRepo -> do
601+ let obGitDir = tmpRepo </> ".git" </> "obelisk"
602+ s = case _thunkPtr_source tptr of
603+ ThunkSource_GitHub s' -> forgetGithub False s'
604+ ThunkSource_Git s' -> s'
792605 withSpinner' ("Fetching thunk " <> T.pack thunkName)
793606 (finalMsg noTrail $ const $ "Fetched thunk " <> T.pack thunkName) $ do
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"]
607+ let git = callProcessAndLogOutput (Notice, Notice) . gitProc tmpRepo
608+ git $ [ "clone" ]
609+ ++ ("--recursive" <$ guard (_gitSource_fetchSubmodules s))
610+ ++ [ T.unpack $ render $ _gitSource_url s ]
611+ ++ do branch <- maybeToList $ _gitSource_branch s
612+ [ "--branch", T.unpack $ untagName branch ]
613+ git ["reset", "--hard", Ref.toHexString $ _thunkRev_commit $ _thunkPtr_rev tptr]
614+ when (_gitSource_fetchSubmodules s) $
615+ git ["submodule", "update", "--recursive", "--init"]
616+
617+ liftIO $ createDirectory obGitDir
618+ callProcessAndLogOutput (Notice, Error) $
619+ proc "cp" ["-r", "-T", thunkDir </> ".", obGitDir </> "orig-thunk"]
620+ callProcessAndLogOutput (Notice, Error) $
621+ proc "rm" ["-r", thunkDir]
622+ callProcessAndLogOutput (Notice, Error) $
623+ proc "mv" ["-T", tmpRepo, thunkDir]
821624
625+--TODO: add force mode to pack even if changes are present
822626--TODO: add a rollback mode to pack to the original thunk
823−packThunk :: MonadObelisk m => ThunkPackConfig -> FilePath -> m ThunkPtr
627+packThunk :: MonadObelisk m => FilePath -> m ThunkPtr
824628packThunk = packThunk' False
825629
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
630+packThunk' :: MonadObelisk m => Bool -> FilePath -> m ThunkPtr
631+packThunk' noTrail thunkDir = checkThunkDirectory "Can't pack/unpack from within the thunk directory" thunkDir >> readThunk thunkDir >>= \case
632+ Left err -> failWith $ T.pack $ "thunk pack: " <> show err
633+ Right (ThunkData_Packed _) -> failWith "pack: thunk is already packed"
634+ Right (ThunkData_Checkout _) -> do
635+ withSpinner' ("Packing thunk " <> T.pack thunkDir)
636+ (finalMsg noTrail $ const $ "Packed thunk " <> T.pack thunkDir) $ do
637+ thunkPtr <- getThunkPtr thunkDir
638+ callProcessAndLogOutput (Debug, Error) $ proc "rm" ["-rf", thunkDir]
639+ liftIO $ createThunk thunkDir thunkPtr
837640 pure thunkPtr
838641
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− }
642+getThunkPtr :: MonadObelisk m => FilePath -> m ThunkPtr
643+getThunkPtr = getThunkPtr' True
846644
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
645+getThunkPtr' :: forall m. MonadObelisk m => Bool -> FilePath -> m ThunkPtr
646+getThunkPtr' checkClean thunkDir = do
647+ when checkClean $ ensureCleanGitRepo thunkDir True $
869648 "thunk pack: thunk checkout contains unsaved modifications"
870649
871650 -- Check whether there are any stashes
872− when checkClean $ do
873− stashOutput <- readGitProcess thunkDir ["stash", "list"]
874− unless (T.null stashOutput) $
651+ stashOutput <- readGitProcess thunkDir ["stash", "list"]
652+ when checkClean $ case T.null stashOutput of
653+ False -> do
875654 failWith $ T.unlines $
876655 [ "thunk pack: thunk checkout has stashes"
877656 , "git stash list:"
878657 ] ++ T.lines stashOutput
658+ True -> return ()
879659
880− -- Get current branch
660+ -- Get current branch ``
881661 (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"]
662+ b <- listToMaybe
663+ <$> T.lines
664+ <$> readGitProcess thunkDir ["rev-parse", "--abbrev-ref", "HEAD"]
665+ c <- listToMaybe
666+ <$> T.lines
667+ <$> readGitProcess thunkDir ["rev-parse", "HEAD"]
884668 case b of
885− (Just "HEAD") -> failWith $ T.unlines
669+ (Just "HEAD") -> failWith $ T.unlines $
886670 [ "thunk pack: You are in 'detached HEAD' state."
887671 , "If you want to pack at the current ref \
888672 \then please create a new branch with 'git checkout -b <new-branch-name>' and push this upstream."
@@ -891,7 +675,7 @@ getThunkPtr gitCheckClean dir mPrivate = do
891675
892676 -- Get information on all branches and their (optional) designated upstream
893677 -- correspondents
894− headDump :: [Text] <- T.lines <$> readGitProcess thunkDir
678+ (headDump :: [Text]) <- T.lines <$> readGitProcess thunkDir
895679 [ "for-each-ref"
896680 , "--format=%(refname:short) %(upstream:short) %(upstream:remotename)"
897681 , "refs/heads/"
@@ -937,7 +721,7 @@ getThunkPtr gitCheckClean dir mPrivate = do
937721 ]
938722
939723 -- loosely by https://stackoverflow.com/questions/7773939/show-git-ahead-and-behind-info-for-all-branches-including-remotes
940− stats <- ifor headUpstream $ \branch (upstream, _remote) -> do
724+ stats <- iforM headUpstream $ \branch (upstream, _remote) -> do
941725 (stat :: [Text]) <- T.lines <$> readGitProcess thunkDir
942726 [ "rev-list", "--left-right"
943727 , T.unpack branch <> "..." <> T.unpack upstream
@@ -954,16 +738,15 @@ getThunkPtr gitCheckClean dir mPrivate = do
954738 [ "thunk pack: Certain branches in the thunk have commits not yet pushed upstream:"
955739 , ""
956740 ] ++
957− flip map (Map.toList nonGood) (\(branch, (upstream, (ahead, behind))) -> mconcat
741+ (flip map (Map.toList nonGood) $ \(branch, (upstream, (ahead, behind))) -> mconcat
958742 [" ", branch, " ahead: ", T.pack (show ahead), " behind: ", T.pack (show behind), " remote branch ", upstream]) ++
959743 [ ""
960744 , "Please push these upstream and try again. (Or just fetch, if they are somehow \
961745 \pushed but this repo's remote tracking branches don't know it.)"
962746 ]
963747
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."
748+ -- We assume it's safe to pack the thunk at this point
749+ putLog Informational $ "All changes safe in git remotes. OK to pack thunk."
967750
968751 let remote = maybe "origin" snd $ flip Map.lookup headUpstream =<< mCurrentBranch
969752
@@ -976,12 +759,14 @@ getThunkPtr gitCheckClean dir mPrivate = do
976759 remoteUri <- case parseGitUri remoteUri' of
977760 Nothing -> failWith $ "Could not identify git remote: " <> remoteUri'
978761 Just uri -> pure uri
979− uriThunkPtr remoteUri mPrivate mCurrentBranch mCurrentCommit
762+ uriThunkPtr remoteUri mCurrentBranch mCurrentCommit
980763
981764-- | Get the latest revision available from the given source
982765getLatestRev :: MonadObelisk m => ThunkSource -> m ThunkRev
983766getLatestRev os = do
984− let gitS = thunkSourceToGitSource os
767+ let gitS = case os of
768+ ThunkSource_GitHub s -> forgetGithub False s
769+ ThunkSource_Git s -> s
985770 (_, commit) <- gitGetCommitBranch (_gitSource_url gitS) (untagName <$> _gitSource_branch gitS)
986771 case os of
987772 ThunkSource_GitHub s -> githubThunkRev s commit
@@ -993,18 +778,18 @@ getLatestRev os = do
993778-- performance. If that doesn't work (e.g. authentication issue), we fall back
994779-- on just doing things the normal way for git repos in general, and save it as
995780-- a regular git thunk.
996−uriThunkPtr :: MonadObelisk m => GitUri -> Maybe Bool -> Maybe Text -> Maybe Text -> m ThunkPtr
997−uriThunkPtr uri mPrivate mbranch mcommit = do
781+uriThunkPtr :: MonadObelisk m => URI -> Maybe Text -> Maybe Text -> m ThunkPtr
782+uriThunkPtr uri mbranch mcommit = do
998783 commit <- case mcommit of
999784 Nothing -> gitGetCommitBranch uri mbranch >>= return . snd
1000785 (Just c) -> return c
1001− (src, rev) <- uriToThunkSource uri mPrivate mbranch >>= \case
786+ (src, rev) <- case uriToThunkSource uri mbranch of
1002787 ThunkSource_GitHub s -> do
1003788 rev <- runExceptT $ githubThunkRev s commit
1004789 case rev of
1005790 Right r -> pure (ThunkSource_GitHub s, r)
1006791 Left e -> do
1007− putLog Warning "\
792+ putLog Warning $ "\
1008793\Failed to fetch archive from GitHub. This is probably a private repo. \
1009794\Falling back on normal fetchgit. Original failure:"
1010795 errorToWarning e
@@ -1021,66 +806,34 @@ uriThunkPtr uri mPrivate mbranch mcommit = do
1021806-- If the thunk is a GitHub thunk and fails, we do *not* fall back like with
1022807-- `uriThunkPtr`. Unlike a plain URL, a thunk src explicitly states which method
1023808-- should be employed, and so we respect that.
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
809+uriToThunkSource :: URI -> Maybe Text -> ThunkSource
810+uriToThunkSource u
811+ | Right uriAuth <- uriAuthority u
812+ , Just scheme <- unRText <$> uriScheme u
1028813 , case scheme of
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
814+ "ssh" -> uriAuth == Authority
815+ { authUserInfo = Just $ UserInfo (fromRight' $ mkUsername "git") Nothing
816+ , authHost = fromRight' $ mkHost "github.com"
817+ , authPort = Nothing
1033818 }
1034819 s -> s `L.elem` [ "git", "https", "http" ] -- "http:" just redirects to "https:"
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− }
820+ && unRText (authHost uriAuth) == "github.com"
821+ , Just (_, owner :| [repoish]) <- uriPath u
822+ = \mbranch -> ThunkSource_GitHub $ GitHubSource
823+ { _gitHubSource_owner = N $ unRText owner
824+ , _gitHubSource_repo = N $ let
825+ repoish' = unRText repoish
826+ in case T.stripSuffix ".git" repoish' of
827+ Just repo -> repo
828+ Nothing -> repoish'
829+ , _gitHubSource_branch = N <$> mbranch
830+ }
1047831
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− }
832+ | otherwise = \mbranch -> ThunkSource_Git $ GitSource
833+ { _gitSource_url = u
834+ , _gitSource_branch = N <$> mbranch
835+ , _gitSource_fetchSubmodules = False -- TODO: How do we determine if this should be true?
836+ }
1084837
1085838-- Funny signature indicates no effects depend on the optional branch name.
1086839githubThunkRev
@@ -1092,19 +845,19 @@ githubThunkRev
1092845githubThunkRev s commit = do
1093846 owner <- forcePP $ _gitHubSource_owner s
1094847 repo <- forcePP $ _gitHubSource_repo s
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
848+ revTarball <- mkPathPiece $ commit <> ".tar.gz"
849+ let archiveUri = URI
850+ { uriScheme = Just $ fromRight' $ mkScheme "https"
851+ , uriAuthority = Right $ Authority
852+ { authUserInfo = Nothing
853+ , authHost = fromRight' $ mkHost "github.com"
854+ , authPort = Nothing
1102855 }
1103− , URI.uriPath = Just ( False
1104− , owner :| [ repo, fromRight' $ URI.mkPathPiece "archive", revTarball ]
856+ , uriPath = Just ( False
857+ , owner :| [ repo, fromRight' $ mkPathPiece "archive", revTarball ]
1105858 )
1106− , URI.uriQuery = []
1107− , URI.uriFragment = Nothing
859+ , uriQuery = []
860+ , uriFragment = Nothing
1108861 }
1109862 hash <- getNixSha256ForUriUnpacked archiveUri
1110863 putLog Debug $ "Nix sha256 is " <> hash
@@ -1113,8 +866,8 @@ githubThunkRev s commit = do
1113866 , _thunkRev_nixSha256 = hash
1114867 }
1115868 where
1116− forcePP :: Name entity -> m (URI.RText 'URI.PathPiece)
1117− forcePP = URI.mkPathPiece . untagName
869+ forcePP :: Name entity -> m (RText 'PathPiece)
870+ forcePP = mkPathPiece . untagName
1118871
1119872gitThunkRev
1120873 :: MonadObelisk m
@@ -1123,11 +876,11 @@ gitThunkRev
1123876 -> m ThunkRev
1124877gitThunkRev s commit = do
1125878 let u = _gitSource_url s
1126− protocols = ["file", "https", "ssh", "git"]
1127− scheme = maybe "file" URI.unRText $ URI.uriScheme $ (\(GitUri x) -> x) u
879+ protocols = ["https", "ssh", "git"]
880+ Just scheme <- pure $ unRText <$> uriScheme u
1128881 unless (T.toLower scheme `elem` protocols) $
1129882 failWith $ "obelisk currently only supports "
1130− <> T.intercalate ", " protocols <> " protocols for plain Git remotes"
883+ <> T.intercalate ", " protocols <> " protocols for non-GitHub remotes"
1131884 hash <- nixPrefetchGit u commit $ _gitSource_fetchSubmodules s
1132885 putLog Informational $ "Nix sha256 is " <> hash
1133886 pure $ ThunkRev
@@ -1140,12 +893,11 @@ gitThunkRev s commit = do
1140893--
1141894-- If the branch name is passed in, it is returned exactly as-is. If it is not
1142895-- passed it, the default branch of the repo is used instead.
1143−
1144896gitGetCommitBranch
1145− :: MonadObelisk m => GitUri -> Maybe Text -> m (Text, CommitId)
897+ :: MonadObelisk m => URI -> Maybe Text -> m (Text, CommitId)
1146898gitGetCommitBranch uri mbranch = withExitFailMessage ("Failure for git remote " <> uriMsg) $ do
1147899 (_, bothMaps) <- gitLsRemote
1148− (T.unpack $ gitUriToText uri)
900+ (T.unpack $ render uri)
1149901 (GitRef_Branch <$> mbranch)
1150902 Nothing
1151903 branch <- case mbranch of
@@ -1161,21 +913,18 @@ gitGetCommitBranch uri mbranch = withExitFailMessage ("Failure for git remote "
1161913 pure (branch, commit)
1162914 where
1163915 rethrowE = either failWith pure
1164− uriMsg = gitUriToText uri
1165−
1166−parseGitUri :: Text -> Maybe GitUri
1167−parseGitUri x = GitUri <$> (parseFileURI x <|> parseAbsoluteURI x <|> parseSshShorthand x)
916+ uriMsg = render uri
1168917
1169−parseFileURI :: Text -> Maybe URI.URI
1170−parseFileURI uri = if "/" `T.isPrefixOf` uri then parseAbsoluteURI ("file://" <> uri) else Nothing
918+parseGitUri :: Text -> Maybe URI
919+parseGitUri x = parseAbsoluteURI x <|> parseSshShorthand x
1171920
1172−parseAbsoluteURI :: Text -> Maybe URI.URI
921+parseAbsoluteURI :: Text -> Maybe URI
1173922parseAbsoluteURI uri = do
1174− parsedUri <- URI.mkURI uri
1175− guard $ URI.isPathAbsolute parsedUri
1176− pure parsedUri
923+ parsedUri <- mkURI uri
924+ guard $ isPathAbsolute parsedUri
925+ pure $ parsedUri
1177926
1178−parseSshShorthand :: Text -> Maybe URI.URI
927+parseSshShorthand :: Text -> Maybe URI
1179928parseSshShorthand uri = do
1180929 -- This is what git does to check that the remote
1181930 -- is not a local file path when parsing shorthand.
@@ -1188,4 +937,4 @@ parseSshShorthand uri = do
1188937 -- This check is used to disambiguate a filepath containing a colon from shorthand
1189938 guard $ isNothing (T.findIndex (=='/') authAndHostname)
1190939 && not (T.null colonAndPath)
1191− URI.mkURI properUri
940+ mkURI properUri