Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Embedded copy of catch2 is too old and buggy #130

Closed
pgrt opened this issue Sep 11, 2022 · 4 comments
Closed

Embedded copy of catch2 is too old and buggy #130

pgrt opened this issue Sep 11, 2022 · 4 comments

Comments

@pgrt
Copy link

pgrt commented Sep 11, 2022

Hello,

The code embeds a copy of catch2, but it lies several versions behind upstream which has solved some issues, e.g. catchorg/Catch2#2421 which showed up for me when using the tip of master of your TileDB to build GenomicsDB.

In Debian unstable, linking to the Debian-packaged catch2 instead has solved this problem.

Cheers,
Pierre

nalinigans added a commit that referenced this issue Sep 12, 2022
@nalinigans
Copy link
Collaborator

nalinigans commented Sep 12, 2022

sudo apt-get catch2 or catch2-devel results in E: Unable to locate package catch2. I am going to build/install Catch2 explicitly from the main branch for the builds at least for now using :

git clone https://github.com/catchorg/Catch2.git
cd Catch2
cmake -Bbuild -H. -DBUILD_TESTING=OFF
sudo cmake --build build/ --target install

I am open to any suggestions. Thanks.

@pgrt
Copy link
Author

pgrt commented Sep 13, 2022

Hello,

Thanks for considering; if you are on a Debian-based distribution I believe the catch2 package should be available in the repositories... but otherwise getting the package from the GitHub repo is also fine. I forgot to say I was using version 2.13.9:
https://github.com/catchorg/Catch2/releases/tag/v2.13.9
as the tip of the master branch will one day be tagged 3.{something}, and (among others) the catch.hpp file has been removed at the tip.
Does grasping version 2.13.9 work for you?

Best wishes,
Pierre

@nalinigans
Copy link
Collaborator

I had moved to version 3.10 in this PR #131 as brew install on the Mac used this version. But, I can change it to v2.13.9 if necessary. Thanks.

@nalinigans
Copy link
Collaborator

Removed embedded copy of Catch2 Header and support versions 2 or 3 of Catch2. See #131.

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

No branches or pull requests

2 participants