On Thursday, August 1, 2024 10:44 AM, Patrick Steinhardt wrote: >On Thu, Aug 01, 2024 at 10:04:37AM -0400, rsbecker@xxxxxxxxxxxxx wrote: >> On Thursday, August 1, 2024 9:55 AM, Patrick Steinhardt wrote: >> >On Thu, Aug 01, 2024 at 09:53:52AM -0400, rsbecker@xxxxxxxxxxxxx wrote: >> >> On Thursday, August 1, 2024 9:50 AM, Patrick Steinhardt wrote: >> >> >On Thu, Aug 01, 2024 at 09:47:38AM -0400, rsbecker@xxxxxxxxxxxxx >wrote: >> >> >> On Thursday, August 1, 2024 9:37 AM, Patrick Steinhardt wrote: >> >> >> >As mentioned in another mail, we do not use its Makefile at all. >> >> >> >Did you >> >> >> check >> >> >> >whether the version I have proposed here works when running >> >> >> >`make >> test`? >> >> >> >> >> >> That is the commit I have been trying to use. make test in clar >> >> >> or >> git? >> >> > >> >> >In Git itself. `make test` builds and runs our unit tests, and >> >> >that now >> >> also includes >> >> >unit tests based on clar with this patch series. The clar Makefile >> >> >exists >> >> only because I >> >> >did a 1:1 import of the upstream dependency. We could just as well >> >> >remove >> >> it >> >> >altogether, including other bits that we don't end up using. >> >> >> >> I see. Well, the 2.46.0 test passes. I ran 'seen' 5 days ago and >> >> 'next' is going now. Would that catch it? >> > >> >No, as the patches in this thread are only up for discussion right >> >now and >> have not >> >been merged to any of the branches. You'd have to apply them on top >> >of >> v2.46.0 >> >first :) >> >> Do you happen to have a public fork? > >You can pull the branch pks-clar-unit-tests from https://gitlab.com/gitlab-org/git. This fix is better (and actually works unlike the previous): diff --git a/t/unit-tests/clar/clar/sandbox.h b/t/unit-tests/clar/clar/sandbox.h index 0ba1479620..1b14dbfd76 100644 --- a/t/unit-tests/clar/clar/sandbox.h +++ b/t/unit-tests/clar/clar/sandbox.h @@ -120,6 +120,12 @@ static int build_sandbox_path(void) if (_mktemp(_clar_path) == NULL) return -1; + if (mkdir(_clar_path, 0700) != 0) + return -1; +#elif defined(__TANDEM) + if (mktemp(_clar_path) == NULL) + return -1; + if (mkdir(_clar_path, 0700) != 0) return -1; #elif defined(_WIN32) This is based on the above repo/branch. Basically a separate block for __TANDEM for temp directories. Build is done. I'm running the full test now and may report more or success. Regards, Randall
