Re: guile --listen fix

[Mark H Weaver]

On 09/19/2012 03:59 PM, Ian Price wrote:
> Mark H Weaver<address@hidden>  writes:
>
> > Your fixes look good to me.  It would be good to add tests for the
> > listen option also.
>
> (system repl server) does need tests, no question. But, I'm not sure how
> I would test the command-line stuff. I suppose we could run a guile
> instance with various arguments, have it output relevant variables, and
> check the output. Hmm. This seems like it would be problematic with
> --listen though, since we'd need to run two instances, and make some
> visible changes to the instance. Maybe I just haven't thought it through
> enough.

I agree that it's not trivial to implement these tests.  We shouldn't 
let this get in the way of committing your fix, which is clearly 
important.  Since there have been no objections, I think you should go 
ahead and commit it to the stable-2.0 branch.

    Mark