MAKEFILE

[amosr]

wow

[amosr]

it works!

amos@Amos zebra $ touch src/Zebra/Foreign/Bindings.hsc
amos@Amos zebra $ /Users/amos/proj/ambiata/mafia-dev/dist/build/mafia/mafia build -w
(HSC) Converting -------- src/Zebra/Foreign/Bindings.hsc
(HSC) Dependencies ------ dist/build/make/Zebra/Foreign/Bindings_hsc_make.c
(Forget) ---------------- dist/build/Zebra/Foreign/Bindings.hs
Building ambiata-zebra-0.0.1...
Preprocessing library ambiata-zebra-0.0.1...
[ 2 of 31] Compiling Zebra.Foreign.Bindings ( dist/build/Zebra/Foreign/Bindings.hs, dist/build/Zebra/Foreign/Bindings.o )
[ 3 of 31] Compiling Zebra.Foreign.Util ( src/Zebra/Foreign/Util.hs, dist/build/Zebra/Foreign/Util.o ) [Zebra.Foreign.Bindings changed]
[ 4 of 31] Compiling Zebra.Foreign.Serial ( src/Zebra/Foreign/Serial.hs, dist/build/Zebra/Foreign/Serial.o ) [Zebra.Foreign.Util changed]

[tranma]

amazing

[amosr]

so you guys with your weird editors that do things will have to do something like

CFLAGS=$(mafia cflags)

good luck with that?

[tranma]

hm, let me try in emacs

[erikd-ambiata]

I'd like a bit more information about what this fixes.

It seems to me that this is just papering over bugs in cabal. Has anyone check the cabal-install github issues page to see if this issue this works around has already been raised? If it hasn't been raised already, maybe it should be.

[amosr]

Definitely. There are issues for the hsc2hs case (going back to 2012), but not for the C case. Either way, this is easier than fixing it in Cabal, and works now.

haskell/cabal#24
haskell/cabal#3087
https://github.com/haskell/cabal/blob/master/Cabal/Distribution/Simple/GHC.hs#L1327

[erikd-ambiata]

Either way, this is easier than fixing it in Cabal

Ever hear of the Tradgedy of the Commons?

[amosr]

I should have put that in the other order: the important part for me is that it works now. I don't care how. But given that fixing it in Cabal would take more time and then require waiting for a release, this is a fine interim solution.

[erikd-ambiata]

Its only an interim solution if someone actually does the work in Cabal.

[amosr]

Ok.. I don't think that's particularly relevant to this PR

[amosr]

Looks like they intend on replacing the Cabal build system with Shake, which should make it easier to implement this properly in Cabal. If this one day ends up being replaced with a shake cabal, all the better; if not it still works.

haskell/cabal#4174

haskell/cabal#4306

[amosr]

gonna merge this. too bad, suckers