X-Git-Url: https://git.lukelau.me/?a=blobdiff_plain;f=src%2FLanguage%2FHaskell%2FLSP%2FTest%2FCompat.hs;h=a6151949a07d7486db2069662cba52c112038dd9;hb=d7dc40224e71d16d319f725556e217c0343cee18;hp=f48faa90cc7c40031779a9679b44ddcbf16e41b1;hpb=2477907622233c9462d919737955f9ad0d38a3aa;p=lsp-test.git diff --git a/src/Language/Haskell/LSP/Test/Compat.hs b/src/Language/Haskell/LSP/Test/Compat.hs index f48faa9..a615194 100644 --- a/src/Language/Haskell/LSP/Test/Compat.hs +++ b/src/Language/Haskell/LSP/Test/Compat.hs @@ -12,16 +12,24 @@ import System.IO -- We have to hide cleanupProcess for process-1.6.3.0 -- cause it is in the public api for 1.6.3.0 versions -- shipped with ghc >= 8.6 and < 8.6.4 -import System.Process hiding (getPid, cleanupProcess) +import System.Process hiding (getPid, cleanupProcess, withCreateProcess) # if MIN_VERSION_process(1,6,4) -import qualified System.Process (getPid, cleanupProcess) +import qualified System.Process (getPid, cleanupProcess, withCreateProcess) # else +import Foreign.C.Error +import GHC.IO.Exception ( IOErrorType(..), IOException(..) ) + import qualified System.Process (getPid) +import qualified Control.Exception as C # endif #else -import System.Process -import System.Process.Internals import Control.Concurrent.MVar +import Foreign.C.Error +import GHC.IO.Exception ( IOErrorType(..), IOException(..) ) +import System.Process hiding (withCreateProcess) +import System.Process.Internals + +import qualified Control.Exception as C #endif #ifdef mingw32_HOST_OS @@ -65,19 +73,47 @@ cleanupRunningProcess :: (Maybe Handle, Maybe Handle, Maybe Handle, ProcessHandl cleanupRunningProcess p@(_, _, _, ph) = getProcessExitCode ph >>= maybe (cleanupProcess p) (const $ return ()) -cleanupProcess :: (Maybe Handle, Maybe Handle, Maybe Handle, ProcessHandle) -> IO () +cleanupProcess + :: (Maybe Handle, Maybe Handle, Maybe Handle, ProcessHandle) -> IO () + +withCreateProcess + :: CreateProcess + -> (Maybe Handle -> Maybe Handle -> Maybe Handle -> ProcessHandle -> IO a) + -> IO a + #if MIN_VERSION_process(1,6,4) + cleanupProcess = System.Process.cleanupProcess + +withCreateProcess = System.Process.withCreateProcess + #else -cleanupProcess (mb_stdin, mb_stdout, mb_stderr, ph) = do - terminateProcess ph +cleanupProcess (mb_stdin, mb_stdout, mb_stderr, ph) = do + -- We ignore the spurious "permission denied" error in windows: + -- see https://github.com/haskell/process/issues/110 + ignorePermDenied $ terminateProcess ph -- Note, it's important that other threads that might be reading/writing -- these handles also get killed off, since otherwise they might be holding -- the handle lock and prevent us from closing, leading to deadlock. - maybe (return ()) hClose mb_stdin + maybe (return ()) (ignoreSigPipe . hClose) mb_stdin maybe (return ()) hClose mb_stdout maybe (return ()) hClose mb_stderr return () + where ignoreSigPipe = ignoreIOError ResourceVanished ePIPE + ignorePermDenied = ignoreIOError PermissionDenied ePERM + +ignoreIOError :: IOErrorType -> Errno -> IO () -> IO () +ignoreIOError ioErrorType errno = + C.handle $ \e -> case e of + IOError { ioe_type = iot + , ioe_errno = Just ioe } + | iot == ioErrorType && Errno ioe == errno -> return () + _ -> C.throwIO e + +withCreateProcess c action = + C.bracket (createProcess c) cleanupProcess + (\(m_in, m_out, m_err, ph) -> action m_in m_out m_err ph) + #endif