Subject: Re: [PATCH v2 1/2] build-system/dune: Automatically deduce test-target in most cases.

From: pukkamustard <[email-masked]>
Date: Thu, 12 Jan 2023 18:44:03 +0000

pukkamustard <[email-masked]> writes:

> Thinking of this, maybe we can drop the `test-target` argument
> completely. wdyt?

Yeah, I'm pretty sure we should drop the test-target argument. I don't
see any reason why it should be used with the dune build-system.

The two remaining packages that have a test-target that is not "test",
"tests" or "." (and do not have tests disabled):

- ocaml-frontc: Builds as expected when test-target is removed.
- ocaml-cohttp: test-target points to a test helper library, not the
  tests themselves. Removing the test-target runs the tests, which
  currently fail.

-pukkamustard