bartavelle / language-puppet

A library to work with Puppet manifests, test them and eventually replace everything ruby.
BSD 3-Clause "New" or "Revised" License
51 stars 8 forks source link

build failure with ghc 8.2: ProcessHandle constructor now has 3 args #214

Closed DanBurton closed 7 years ago

DanBurton commented 7 years ago

As seen on the Stackage build server:

[50 of 50] Compiling SafeProcess      ( SafeProcess.hs, dist/build/SafeProcess.o )

SafeProcess.hs:62:10: error:
    • The constructor ‘ProcessHandle’ should have 3 arguments, but has been given 2
    • In the pattern: ProcessHandle pmvar _
      In a pattern binding: (ProcessHandle pmvar _) = ph
      In the expression:
        do let (ProcessHandle pmvar _) = ph
           readMVar pmvar
             >>=
               \case
                 OpenHandle pid -> signalProcessGroup 15 pid
                 _ -> return ()
   |
62 |     let (ProcessHandle pmvar _) = ph
   |          ^^^^^^^^^^^^^^^^^^^^^
PierreR commented 7 years ago

Thanks for the feedback.

@bartavelle a new release would fix this as I have removed the SafeProcess module in a recent commit.

bartavelle commented 7 years ago

Done!