<div dir="ltr">Thanks for the feedback everyone. If/when we decide to backport the patch, we'll be sure to make the commit available for others interested in running a custom build of GHC. And thank you for the offer of help Eric, it's much appreciated :)<br></div><br><div class="gmail_quote">On Mon, Mar 23, 2015 at 8:14 PM Eric Seidel <<a href="mailto:eric@seidel.io">eric@seidel.io</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">IIRC the patch doesn't directly depend on -fwarn-redundant-constraints,<br>
but I think I ran into some merge conflicts that had to be resolved. I<br>
agree with Austin that backporting it should be doable, and would be<br>
happy to help if there's interest.<br>
<br>
Also, I still mean to submit a follow-on patch that uses the new srcLoc<br>
infrastructure to add locations to explicitly-failing functions (i.e.<br>
error, undefined, and maybe assert). I don't know if this will be<br>
palatable to GHC-HQ as it changes base, but I think it's at least a<br>
discussion worth having. Unfortunately I got sidetracked by school stuff<br>
and haven't had a chance to throw the patch together yet..<br>
<br>
On Mon, Mar 23, 2015, at 10:50, Austin Seipp wrote:<br>
> Hi Michael,<br>
><br>
> I actually tried to adopt this patch into 7.10, because I thought it<br>
> was needed for another dependent patch we wanted. Unfortunately, the<br>
> backtrace-via-implicit-params patch seems to depend on some prior work<br>
> by Simon PJ in the typechecker (-fwarn-redundant-constraints, a rather<br>
> large patch), which we *did not* want in 7.10 (the *textual* diff<br>
> applied fine, but there were some API changes the backtrace patch<br>
> needed, so it failed to build). So, in the end, it was easier to<br>
> surgically remove this patch from the one that depended on it, and it<br>
> had a much lower 'surface area' of changes, than adopting both. Hope<br>
> that makes sense.<br>
><br>
> Also, as Simon said, we don't normally do big changes like this in<br>
> point releases, so I think this is unlikely to happen.<br>
><br>
> So the short story is "afraid not". But a backport should be possible,<br>
> if you're willing to get your hands a bit dirty (any conflict will<br>
> likely be fairly easy to fix, but it will involve some textual<br>
> munging).<br>
><br>
> On Mon, Mar 23, 2015 at 6:49 AM, Michael Snoyman <<a href="mailto:michael@snoyman.com" target="_blank">michael@snoyman.com</a>><br>
> wrote:<br>
> > It looks like the srcLoc change[1] is something that some of our (FP<br>
> > Complete's) customers would be quite interested in getting access to sooner<br>
> > rather than later. Would there be any possibility of getting that patch<br>
> > merged into the 7.10 branch of GHC? I'm not sure if I'd try my luck at<br>
> > actually including it in 7.10.1, but would it be on the table for 7.10.2?<br>
> ><br>
> > We do of course have the option of backporting it ourselves and including it<br>
> > in a custom GHC we provide customers, but we generally try to stay as close<br>
> > to upstream as possible.<br>
> ><br>
> > [1] <a href="https://ghc.haskell.org/trac/ghc/ticket/9049" target="_blank">https://ghc.haskell.org/trac/<u></u>ghc/ticket/9049</a><br>
> ><br>
> > ______________________________<u></u>_________________<br>
> > ghc-devs mailing list<br>
> > <a href="mailto:ghc-devs@haskell.org" target="_blank">ghc-devs@haskell.org</a><br>
> > <a href="http://mail.haskell.org/cgi-bin/mailman/listinfo/ghc-devs" target="_blank">http://mail.haskell.org/cgi-<u></u>bin/mailman/listinfo/ghc-devs</a><br>
> ><br>
><br>
><br>
><br>
> --<br>
> Regards,<br>
><br>
> Austin Seipp, Haskell Consultant<br>
> Well-Typed LLP, <a href="http://www.well-typed.com/" target="_blank">http://www.well-typed.com/</a><br>
> ______________________________<u></u>_________________<br>
> ghc-devs mailing list<br>
> <a href="mailto:ghc-devs@haskell.org" target="_blank">ghc-devs@haskell.org</a><br>
> <a href="http://mail.haskell.org/cgi-bin/mailman/listinfo/ghc-devs" target="_blank">http://mail.haskell.org/cgi-<u></u>bin/mailman/listinfo/ghc-devs</a><br>
______________________________<u></u>_________________<br>
ghc-devs mailing list<br>
<a href="mailto:ghc-devs@haskell.org" target="_blank">ghc-devs@haskell.org</a><br>
<a href="http://mail.haskell.org/cgi-bin/mailman/listinfo/ghc-devs" target="_blank">http://mail.haskell.org/cgi-<u></u>bin/mailman/listinfo/ghc-devs</a><br>
</blockquote></div>