Fix CMake config to build on Ubuntu 22.04

[donjan]

Closes #104 and #105

Testing was done on a fresh Ubuntu 22.04 VM running on a Ubuntu 22.04 host.
To reproduce, install VirtualBox on the host with:
sudo apt install virtualbox-qt
Then download the Ubuntu ISO from:
https://releases.ubuntu.com/22.04.2/ubuntu-22.04.2-desktop-amd64.iso
Launch the VirtualBox GUI, install the image, inside the VM install the common build tools with:

sudo apt install build-essential git ninja-build python3-pip python3-setuptools-scm
sudo pip install --upgrade pip
sudo pip install cmake  # newer version than repo
sudo pip install --upgrade conan==1.59.0
conan profile new default --detect
conan profile update settings.compiler.libcxx=libstdc++11 default

Copy the repo into the VM (either by git clone with valid credentials or a passthrough directory to the host) and cd into it.

mkdir build && cd build
conan install .. --build=missing
conan build ..

[taalexander]

LGTM. We're testing this internally before merging.

[taalexander]

This change breaks builds on OSX, is it necessary?

[donjan]

It will not build in my testing setup because CMake doesn't correctly find the Conan-provided zip library.
The proposed change uses the system library instead, which is installed by default on the image specified above.

If it breaks on OSX, we shouldn't merge it as is, of course.

A) provide a proper Conan/CMake fix to supply the Conan-based libzip
B) branch at this point to use system libzip if available
C) revert this line and soon put it into Linux install instructions

Right now I'm not seeing an obvious way to do A).
B) is ugly.
C) is the easiest but relies on the tough skin of Linux users ;) ... pick that for now?

[taalexander]

I'm wondering why Conan/Cmake isn't finding the correct library. Until I get an Ubuntu image spun up to test I think C. is easiest, but I really think this should be resolvable.

What conan/cmake command did you use to install/build?

[donjan]

LGTM. We're testing this internally before merging.

I very much rely on that, only testing with Ubuntu 22.04 here!

[taalexander]

Its weird it doesn't seem like CI has run here, hmm..

[donjan]

Due to the minuscule size of the change, I've force-pushed instead of reverting. Hope that's OK with you.

[taalexander]

Tested with OSX x86 and tests passed with a clean build