[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