On Thu, 2008-11-06 at 13:11 +0000, Eric Kow wrote:
> On Thu, Nov 06, 2008 at 08:07:51 -0500, David Roundy wrote:
> > It turns out (unsurprisingly) that I broke ctrl-C handling myself...
> > 
> > Another option for this would be to require that we have
> > System.Posix.Signals whenever we aren't running on windows.  This would
> > probably be a better option, but slightly more invasive.
> 
> > Thu Nov  6 08:04:31 EST 2008  David Roundy <[EMAIL PROTECTED]>
> >   * resolve issue1189: define HAVE_SIGNALS in franchise build.
> >   Incidentally, ctrl-C handling seems also to be broken on the non-Windows
> >   cabal build in release/, but I can't see how to fix it.  Or maybe
> >   HAVE_SIGNALS is defined using some Deep Magic?
> 
> Duncan, any comments?

I was using the franchise Setup.hs as reference, so the cabal build must
suffer from the same problem.

The fix is probably:

  if !os(windows)
    build-depends: unix == 2.3.*
+   cpp-options:   -DHAVE_SIGNALS

If you don't want to hard code the choice based on the os then you can
do that instead at the cost of slightly more syntax.

> resolve issue1189: define HAVE_SIGNALS in franchise build.
> ----------------------------------------------------------
> > +               withModule "System.Posix.Signals" $ define "HAVE_SIGNALS"
> 
> Looks good to me!


Duncan

_______________________________________________
darcs-users mailing list
[email protected]
http://lists.osuosl.org/mailman/listinfo/darcs-users

Reply via email to