summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGravatar Joey Hess <joey@kitenet.net>2011-10-30 16:38:48 -0400
committerGravatar Joey Hess <joey@kitenet.net>2011-10-30 16:38:48 -0400
commitee715647540a8b5d81254ed60cdf7709f63f42af (patch)
tree59065cdd9d1025fe7575456a98f50b0235241ad8
parent4e9be0d1f86893a469b33b763b55edfe75bdb3aa (diff)
add command name to some output
-rw-r--r--CmdLine.hs34
-rw-r--r--doc/bugs/minor_bug:_errors_are_not_verbose_enough.mdwn2
2 files changed, 21 insertions, 15 deletions
diff --git a/CmdLine.hs b/CmdLine.hs
index fffd343f0..658b38ab1 100644
--- a/CmdLine.hs
+++ b/CmdLine.hs
@@ -26,22 +26,25 @@ dispatch :: [String] -> [Command] -> [Option] -> String -> Git.Repo -> IO ()
dispatch args cmds options header gitrepo = do
setupConsole
state <- Annex.new gitrepo
- (actions, state') <- Annex.run state $ parseCmd args header cmds options
- tryRun state' $ [startup] ++ actions ++ [shutdown]
+ ((cmd, actions), state') <- Annex.run state $ parseCmd args header cmds options
+ tryRun state' cmd $ [startup] ++ actions ++ [shutdown]
{- Parses command line, stores configure flags, and returns a
- - list of actions to be run in the Annex monad. -}
-parseCmd :: [String] -> String -> [Command] -> [Option] -> Annex [Annex Bool]
+ - list of actions to be run in the Annex monad and the Command
+ - being run. -}
+parseCmd :: [String] -> String -> [Command] -> [Option] -> Annex (Command, [Annex Bool])
parseCmd argv header cmds options = do
(flags, params) <- liftIO getopt
when (null params) $ error $ "missing command" ++ usagemsg
- case lookupCmd (head params) of
- [] -> error $ "unknown command" ++ usagemsg
+ let (c:rest) = params
+ case lookupCmd c of
+ [] -> error $ "unknown command " ++ c ++ " " ++ usagemsg
[cmd] -> do
_ <- sequence flags
checkCommand cmd
- prepCommand cmd (drop 1 params)
- _ -> error "internal error: multiple matching commands"
+ as <- prepCommand cmd rest
+ return (cmd, as)
+ _ -> error $ "internal error: multiple matching commands for " ++ c
where
getopt = case getOpt Permute options argv of
(flags, params, []) ->
@@ -70,10 +73,10 @@ usage header cmds options =
{- Runs a list of Annex actions. Catches IO errors and continues
- (but explicitly thrown errors terminate the whole command).
-}
-tryRun :: Annex.AnnexState -> [Annex Bool] -> IO ()
+tryRun :: Annex.AnnexState -> Command -> [Annex Bool] -> IO ()
tryRun = tryRun' 0
-tryRun' :: Integer -> Annex.AnnexState -> [Annex Bool] -> IO ()
-tryRun' errnum state (a:as) = do
+tryRun' :: Integer -> Annex.AnnexState -> Command -> [Annex Bool] -> IO ()
+tryRun' errnum state cmd (a:as) = do
result <- try $ Annex.run state $ do
Annex.Queue.flushWhenFull
a
@@ -82,10 +85,11 @@ tryRun' errnum state (a:as) = do
Annex.eval state $ do
showErr err
showEndFail
- tryRun' (errnum + 1) state as
- Right (True,state') -> tryRun' errnum state' as
- Right (False,state') -> tryRun' (errnum + 1) state' as
-tryRun' errnum _ [] = when (errnum > 0) $ error $ show errnum ++ " failed"
+ tryRun' (errnum + 1) state cmd as
+ Right (True,state') -> tryRun' errnum state' cmd as
+ Right (False,state') -> tryRun' (errnum + 1) state' cmd as
+tryRun' errnum _ cmd [] = when (errnum > 0) $
+ error $ cmdname cmd ++ ": " ++ show errnum ++ " failed"
{- Actions to perform each time ran. -}
startup :: Annex Bool
diff --git a/doc/bugs/minor_bug:_errors_are_not_verbose_enough.mdwn b/doc/bugs/minor_bug:_errors_are_not_verbose_enough.mdwn
index 8def2e8c3..a6620f425 100644
--- a/doc/bugs/minor_bug:_errors_are_not_verbose_enough.mdwn
+++ b/doc/bugs/minor_bug:_errors_are_not_verbose_enough.mdwn
@@ -22,3 +22,5 @@ Better:
etc pp.
+
+> [[done]] --[[Joey]]