Branch: refs/heads/master
Revision: c2294c1
Author: e-kayrakli
Log Message:
Merge pull request #17667 from e-kayrakli/fix-paratest-compopts
Fix a paratest bug that caused compopts to be dropped
Fixes a bug that Convert memleaks nightly script into a correctness test on memory leaks by e-kayrakli · Pull Request #17528 · chapel-lang/chapel · GitHub introduced.
paratest.server
passes $memleaksflag
to paratest.client
. But that flag
doesn't have a default value. So, if you don't throw --memleaks
it is empty
string. This causes paratest.server
to pass one less flag to paratest.client
which causes compopts flag to be read in the wrong order.
[Reviewed by @ronawho]
Test:
-
[x]
-compopts --verify
inrelease/examples
(checked the logs) -
[x]
-compopts --verify
full suite -
[x]
-compopts --llvm
inrelease/examples
(checked the logs) -
[x]
-compopts --llvm
full suite (checked the logs) -
[x]
-memleaks
inrelease/examples
andtest/memory
-
[x]
-memleakslog logfile
inrelease/examples
andtest/memory
Modified Files:
M util/test/paratest.serverCompare: Comparing 9e24475a07c3...c2294c13931c · chapel-lang/chapel · GitHub