On 2023-11-08 20:10 +0100, Martin Budaj wrote:
> On Tue, Nov 7, 2023 at 4:25 PM Wookey <woo...@wookware.org> wrote:
> 
> > It looks like moving to catch3 and adding:
> > target_link_libraries(test PRIVATE Catch2::Catch2WithMain)
> > in the test targets should do the trick.
> >
> 
> Hi,
> 
> as we still need to maintain Catch2 v2 API compatibility to run CI tests
> and builds on older Ubuntu images, we can't simply migrate to v3.

Who is building 'latest' Therion on old Ubuntu? And are they getting
their sources from the Debian unstable package? Or from Upstream?

> For now, I'll just enable using the bundled Catch2 instead of v3 installed
> in the system.

That's not the right approach for the Debian package, and this bug is about the 
debian package.
Debian unstable has catch 3 in it. We should use it, not an old bundled catch2 
copy.

Upstream builds and Ubuntu builds can do something different if need
be but that's not a good reason for the Debian package not to
DTRT. And in general I'd expect current Ubuntu to have catch3 too so
using the system version will be appropriate there too.

Wookey
-- 
Principal hats:  Debian, Wookware, ARM
http://wookware.org/

Attachment: signature.asc
Description: PGP signature

Reply via email to