Skip to content

Fix library name of oUnit#663

Merged
djs55 merged 1 commit into
moby:masterfrom
tarides:ounit-name
May 20, 2026
Merged

Fix library name of oUnit#663
djs55 merged 1 commit into
moby:masterfrom
tarides:ounit-name

Conversation

@Leonidas-from-XIV
Copy link
Copy Markdown
Contributor

When running tests the tests fail because of the fact that some reshuffling happened in OUnit and the library name changed:

  + dune runtest
  File "src/dns_test/dune", line 3, characters 18-23:
  3 |   (libraries  dns oUnit pcap-format bigarray bigarray-compat lwt)
                        ^^^^^
  Error: Library "oUnit" not found.
  -> required by _build/default/src/dns_test/test.exe
  -> required by alias src/dns_test/runtest in src/dns_test/dune:7

This PR updates it to use ounit2 which seems to pass the tests (with no changes to the dependency definitions).

Signed-off-by: Marek Kubica <marek@tarides.com>
@djs55 djs55 merged commit 89f59f7 into moby:master May 20, 2026
5 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants