[m-rev.] for review: rewrite rest of options_file.m

Zoltan Somogyi zoltan.somogyi at runbox.com
Wed Jun 10 01:04:37 AEST 2020


2020-06-09 10:33 GMT+10:00 Julien Fischer<jfischer at opturion.com>:
> I think the commit that broke it was either afe2887 or 520ce39 since
> they are the ones that changed the value of TEST_DIRS to hardcode it
> to '..'.
> 
> The problem being that when you now do "mmake runtests" in the tests
> directory itself, it looks for various files in the top-level of
> the the source tree, rather than in the tests directory itself,
> e.g.
> 
>     Mmake.common:82: ../DEFNS_FOR_TESTS: No such file or directory mercury_compile:
>     cannot open ../TESTS_FLAGS: can't open input file: No such file or directory
>     mercury_compile: cannot open ../TESTS_FLAGS: can't open input file: No such file or directory
>     gmake: *** No rule to make target `../DEFNS_FOR_TESTS'.  Stop.

The attached diff fixes this problem, at least for me.
Can you please have a look at it post-commit?

Zoltan.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Log.rt
Type: application/octet-stream
Size: 1074 bytes
Desc: not available
URL: <http://lists.mercurylang.org/archives/reviews/attachments/20200610/12335c09/attachment-0002.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: DIFF.rt
Type: application/octet-stream
Size: 20531 bytes
Desc: not available
URL: <http://lists.mercurylang.org/archives/reviews/attachments/20200610/12335c09/attachment-0003.obj>


More information about the reviews mailing list