Print an error before usage if we can't parse the page.

[dead/lwn-epub.git] / src / Main.hs

diff --git a/src/Main.hs b/src/Main.hs
index 373d423c9f322ba885db966fecff5b7e190c4dd4..1f6d7c5f436a583d5143b05c1cf55aaf4126995e 100644 (file)
--- a/src/Main.hs
+++ b/src/Main.hs
@@ -1,24 +1,24 @@
-{-# LANGUAGE ScopedTypeVariables, RecordWildCards #-}
+{-# LANGUAGE DoAndIfThenElse #-}
 module Main
 where
 
+import Control.Concurrent.ParallelIO (stopGlobalPool)
+import System.Directory (doesFileExist)
 import System.IO (
   Handle,
   IOMode (WriteMode),
+  hPutStrLn,
   openBinaryFile,
+  stderr,
   stdout
   )
-import Text.XML.HXT.Core
 
-import CommandLine (Args(..), apply_args)
-import LWN.Page
+import CommandLine (show_help)
+import Configuration (Cfg(..), get_cfg)
+import LWN.Article (real_article_path)
+import LWN.HTTP (get_login_cookie)
+import LWN.Page (epublish, page_from_url)
 
-my_read :: String -> IOStateArrow s b XmlTree
-my_read =
-  readDocument [ withValidate  no,
-                 withParseHTML yes,
-                 withInputEncoding utf8,
-                 withWarnings no ]
 
 -- | If we're given an empty path, return a handle to
 --   'stdout'. Otherwise, open the given file and return a read/write
@@ -31,24 +31,28 @@ get_output_handle path =
     openBinaryFile path WriteMode
 
 
--- | Convert the given article to either a URL or a filesystem
---   path. If the given article exists on the filesystem, we assume
---   it's a file. Otherwise, we check to see if it's a URL. Failing
---   that, we try to construct a URL from what we're given and do our
---   best.
-real_article_path :: String -> IO String
-real_article_path = return . id
+argument_is_file :: Cfg -> IO Bool
+argument_is_file cfg = do
+  path <- real_article_path (article cfg)
+  doesFileExist path
 
 main :: IO ()
 main = do
-  Args{..} <- apply_args
-  output_handle <- get_output_handle output
-  input_path <- real_article_path article
-  let html = my_read input_path
-  result <- parse html
-
-  case result of
-    Just stuff  -> epublish stuff output_handle
-    Nothing     -> return ()
-
-  putStrLn "Done."
+  cfg' <- get_cfg
+  aif <- argument_is_file cfg'
+  cfg  <- case aif of
+           False -> get_login_cookie cfg'
+           True  -> return cfg'
+
+  page <- page_from_url cfg (article cfg)
+  case page of
+    Just p -> do
+      output_handle <- get_output_handle (output cfg)
+      epublish p output_handle
+    Nothing -> do
+      hPutStrLn stderr "ERROR: could not parse an LWN page from the given URL."
+      _ <- show_help
+      return ()
+
+  -- Necessary, for some reason.
+  stopGlobalPool