On Sun, Jun 01, 2008 at 05:30:41PM +0100, Ian Lynagh wrote:
> On Sat, May 31, 2008 at 03:51:39AM -0700, David Roundy wrote:
> > On Sat, May 31, 2008 at 3:21 AM, David Roundy <[EMAIL PROTECTED]> wrote:
> > > On Fri, May 30, 2008 at 03:08:49PM +0000, [EMAIL PROTECTED] wrote:
> > >> I thought I might re-send Ian's non-conflictor patches.
> > >>
> > >> Wed Apr 30 01:11:12 BST 2008  Ian Lynagh <[EMAIL PROTECTED]>
> > >>   * Add Show/Show2 instances for :/\:
> > >>
> > > I've applied these first two.
> > 
> > The first patch caused an error in compiling the unit tests, since we
> > already define this instance elsewhere, so it's not being pushed.
> 
> It would be nice to move the instance to the module defining the type,
> partly to avoid orphan modules[1], but mostly just so that it's always
> available when working with ghci on a subset of the modules.

I'd rather keep debugging code separate from code we use in darcs itself,
just to avoid bloat.
-- 
David Roundy
Department of Physics
Oregon State University
_______________________________________________
darcs-users mailing list
[email protected]
http://lists.osuosl.org/mailman/listinfo/darcs-users

Reply via email to