search

eskimor / servant-purescript

Translate servant API to purescript code, with the help of purescript-bridge.
BSD 3-Clause "New" or "Revised" License
105 stars 44 forks source link

not compatible with servant 0.15 / lts-13 #37

Closed legrostdg closed 5 years ago

legrostdg commented 5 years ago

When trying to compile servant-purescript with lts-13, I get:

--  While building package servant-purescript-0.9.0.2 using:
      /home/gueux/.stack/setup-exe-cache/x86_64-linux-tinfo6/Cabal-simple_mPHDZzAJ_2.4.0.1_ghc-8.6.4 --builddir=.stack-work/dist/x86_64-linux-tinfo6/Cabal-2.4.0.1 build --ghc-options " -ddump-hi -ddump-to-file -fdiagnostics-color=always"
    Process exited with code: ExitFailure 1
    Logs have been written to: /home/gueux/src/hoarder/.stack-work/logs/servant-purescript-0.9.0.2.log

    Configuring servant-purescript-0.9.0.2...
    Preprocessing library for servant-purescript-0.9.0.2..
    Building library for servant-purescript-0.9.0.2..
    [1 of 6] Compiling Servant.API.BrowserHeader ( src/Servant/API/BrowserHeader.hs, .stack-work/dist/x86_64-linux-tinfo6/Cabal-2.4.0.1/build/Servant/API/BrowserHeader.o )

    /tmp/stack7733/servant-purescript-0.9.0.2/src/Servant/API/BrowserHeader.hs:28:10: error:
        • Number of parameters must match family declaration; expected 2
        • In the type instance declaration for ‘MkLink’
          In the instance declaration for
            ‘HasLink (BrowserHeader sym a :> sub)’
       |
    28 |     type MkLink (BrowserHeader sym a :> sub) = MkLink (Header sym a :> sub)
       |          ^^^^^^
legrostdg commented 5 years ago

It seems to be fixed in master, with https://github.com/eskimor/servant-purescript/commit/b6d5e9e7ee61861a048ddc16a3d704ded8adf8fb... Would it be possible to push a new release?

eskimor commented 5 years ago

Done: http://hackage.haskell.org/package/servant-purescript-0.9.0.3