[m-rev.] diff: fix sub-module build problem

Peter Ross peter.ross at miscrit.be
Fri Nov 16 08:54:25 AEDT 2001


Fergus wrote:
> On 15-Nov-2001, Peter Ross <peter.ross at miscrit.be> wrote:
> > It doesn't pass it on my system.
>
> The test that you originally posted passes.
> The test that you committed doesn't.
>
Neither test passed on my system

> (In between posting and committing, you seem to have
> added some extra stuff which tests having a nested
> module inside a separate sub-module.)
>
I made the test case a little harder before commiting, as my change handled
both cases correctly.  We should also test the situation Simon pointed out
which is a sub-module which doesn't have a fully qualified name for its
file-name.

Shall I disable the test case or have you tried your new change with it?

Pete


--------------------------------------------------------------------------
mercury-reviews mailing list
post:  mercury-reviews at cs.mu.oz.au
administrative address: owner-mercury-reviews at cs.mu.oz.au
unsubscribe: Address: mercury-reviews-request at cs.mu.oz.au Message: unsubscribe
subscribe:   Address: mercury-reviews-request at cs.mu.oz.au Message: subscribe
--------------------------------------------------------------------------



More information about the reviews mailing list