<div dir="ltr"><div dir="ltr">Hi Jonathan,<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Sep 15, 2020 at 5:32 PM Jonathan Wilkes <<a href="mailto:jon.w.wilkes@gmail.com">jon.w.wilkes@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
1. Does that mean that now when a user runs `make` the tests get run?<br></blockquote><div><br></div><div>No, they're separate targets, but that's customary. You wouldn't want the tests to be mandatory like that. If you want to test right after the build, just run `make all check`, simple as that. There's also the possibility to run `make all check1` if you only want to do the basic regression test.<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
2. If not, can we change the build system so that it runs the tests as<br>
part of the build?<br></blockquote><div><br></div><div>See 1. :)</div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

I guess the one tricky part is `make light`. It wouldn't succeed on<br>
the external tests.</blockquote><div><br></div><div>It already fails regression_tests.pd, because that test invokes the cxc/hex2dec external which the light build doesn't include. So `make light check1` returns a nonzero exit status.<br></div><div><br></div><div></div><div>The idea of the light build is that it only includes the absolutely essential externals so that Purr Data will work and nothing else (getting as close to a vanilla-like package as possible), and I'd really like it to stay that way.</div><div><br></div><div> I'd suggest to simply add a stripped-down version of regression_tests.pd, say regression_tests0.pd, that doesn't need cxc/hex2dec (or maybe replaces that part of the test with something else that's in the core). Then I can add another check0 target, so that you could run `make light check0` to build and test that version.<br></div><div><br></div>Albert<br clear="all"></div><br>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr">Dr. Albert Gr"af<br>Computer Music Research Group, JGU Mainz, Germany<br>Email: <a href="mailto:aggraef@gmail.com" target="_blank">aggraef@gmail.com</a>, web: <a href="https://agraef.github.io/" target="_blank">https://agraef.github.io/</a></div></div></div></div></div></div></div>