swipl-devel icon indicating copy to clipboard operation
swipl-devel copied to clipboard

Compiling 8.3.7 passes all tests but an error remains in the log file about plunit_coroutining

Open dtonhofer opened this issue 3 years ago • 1 comments

We find by grepping for ERROR:

ERROR: swipl-devel_original/src/Tests/core/test_coroutining.pl:50:
ERROR:    module/2: No permission to redefine module `plunit_coroutining' (Already loaded from swipl-devel_original/src/Tests/core/te....pl)
ERROR: swipl-devel_original/src/Tests/core/test_coroutining.pl:62:
ERROR:    PL-Unit: cannot close unit coroutining: no open unit

The filename of the offending source (something that begins with te is sadly elided (how might that be a good idea?)

But it''s one of those:

system/swipl-devel_original/src/Tests/core/test_bips.pl::- begin_tests(coroutining).
system/swipl-devel_original/src/Tests/core/test_bips.pl::- end_tests(coroutining).
system/swipl-devel_original/src/Tests/core/test_coroutining.pl::- begin_tests(coroutining).
system/swipl-devel_original/src/Tests/core/test_coroutining.pl::- end_tests(coroutining).
system/swipl-devel_original/src/Tests/core/test_exception.pl::- begin_tests(ex_coroutining).
system/swipl-devel_original/src/Tests/core/test_exception.pl::- end_tests(ex_coroutining).

dtonhofer avatar Sep 23 '20 09:09 dtonhofer

Thanks for spotting. Guess the one in test_bips.pl needs to be renamed. The other one already is. Possibly the tests in test_bips.pl can better be moved into the test_coroutining.pl. Feel free to have a look and propose a solution. I'm in some deep branch :smile:

JanWielemaker avatar Sep 23 '20 09:09 JanWielemaker