agocorona / MFlow

(Haskell) Web application server with stateful, type safe user interactions and widget combinators
http://mflowdemo.herokuapp.com
Other
100 stars 12 forks source link

Undeclared dependency on cpphs #63

Closed peti closed 9 years ago

peti commented 9 years ago

MFlow needs cpphs to compile, but that dependency is not declared in the Cabal file. This causes build failures on NixOS and probably on other distributions, too.

agocorona commented 9 years ago

Hi Peter, Do you mean the last version 0.4.5.10?

It does not depend on cpphs. That dependency is in one of the examples and cpphs is not a library but an executable preprocessor, so cabal can not express such dependency.

It was required in some previous version of MFlow to compile the examples, but for that reason the examples were suppressed from the installation process in latest versions.

2015-08-29 20:01 GMT+02:00 Peter Simons notifications@github.com:

MFlow needs cpphs to compile, but that dependency is not declared in the Cabal file. This causes build failures on NixOS and probably on other distributions, too.

— Reply to this email directly or view it on GitHub https://github.com/agocorona/MFlow/issues/63.

Alberto.

peti commented 9 years ago

Yes, I mean version 0.4.5.10.

It does not depend on cpphs. That dependency is in one of the examples and cpphs is not a library but an executable preprocessor, so cabal can not express such dependency.

Huh? Of course it can! Isn't that exactly what the build-tool stanza is for?

agocorona commented 9 years ago

Oh, sorry..

I mxed cpphs with monadloc_pp, which is the preprocessor that the examples need

How to use build_tool? I find no documentation about it .

looking at https://github.com/haskell/cabal/issues/1466

I think that it needs a line somewhere in the cabal file like;

build_tool: cpphsProgram

But I don't know

peti commented 9 years ago

https://www.haskell.org/cabal/users-guide/developing-packages.html describes the build-tools: stanza.

agocorona commented 9 years ago

Uploaded 0.4.5.11

2015-08-30 1:02 GMT+02:00 Peter Simons notifications@github.com:

https://www.haskell.org/cabal/users-guide/developing-packages.html describes the build-tools: stanza.

— Reply to this email directly or view it on GitHub https://github.com/agocorona/MFlow/issues/63#issuecomment-136059246.

Alberto.

peti commented 9 years ago

Thank you for the quick fix!

agocorona commented 9 years ago

Thank you for the interest in MFlow!

2015-08-30 2:11 GMT+02:00 Peter Simons notifications@github.com:

Thank you for the quick fix!

— Reply to this email directly or view it on GitHub https://github.com/agocorona/MFlow/issues/63#issuecomment-136064958.

Alberto.

peti commented 8 years ago

Why has the declaration of the build-tools: cpphs dependency vanished again from 0.4.5.11's Cabal file? Now we're back to the same build error that was already fixed before. :disappointed:

geraldus commented 8 years ago

Hmm... I can't be certain, but I have never used (and even installed) cpphs on my OS X laptop, and my current MFlow projects compile just fine with stack tool.

agocorona commented 8 years ago

It was due to a bad synchronization. Fixing it to drop cpphs as a dependency. Sorry

2015-09-21 7:34 GMT+02:00, Peter Simons notifications@github.com:

Why has the declaration of the build-tools: cpphs dependency vanished again from 0.4.5.11's Cabal file? Now we're back to the same build error that was already fixed before. :disappointed:


Reply to this email directly or view it on GitHub: https://github.com/agocorona/MFlow/issues/63#issuecomment-141880870

Alberto.

agocorona commented 8 years ago

uploading 4.6.0

2015-09-21 11:18 GMT+02:00, Alberto G. Corona agocorona@gmail.com:

It was due to a bad synchronization. Fixing it to drop cpphs as a dependency. Sorry

2015-09-21 7:34 GMT+02:00, Peter Simons notifications@github.com:

Why has the declaration of the build-tools: cpphs dependency vanished again from 0.4.5.11's Cabal file? Now we're back to the same build error that was already fixed before. :disappointed:


Reply to this email directly or view it on GitHub: https://github.com/agocorona/MFlow/issues/63#issuecomment-141880870

Alberto.

Alberto.