rr icon indicating copy to clipboard operation
rr copied to clipboard

Building rr using external blake2 and brotli

Open wcohen opened this issue 5 years ago • 5 comments

Fedora packaging guidelines discourage the use of bundled packages for building rpms.

https://docs.fedoraproject.org/en-US/packaging-guidelines/#bundling

Fedora has system copies of blake2 (libb2) and brotli available. Is there some way to configure rr to use the system versions of those packages?

It looks like some patching is needed as there is a hard coded include to the third-party directory: $ grep -r third-party PackCommand.cc:#include "../third-party/blake2/blake2.h"

wcohen avatar Jan 30 '20 17:01 wcohen

We don't support that right now but I'd accept a patch to support it. We aren't patching blake2 or brotli and they're very stable so it shouldn't be hard.

rocallahan avatar Jan 30 '20 19:01 rocallahan

To be clear I'd accept a patch to opt into that.

rocallahan avatar Jan 30 '20 19:01 rocallahan

This would be simple enough using FindPkgConfig, which version of CMake rr should support?

lu-zero avatar Jul 05 '25 13:07 lu-zero

Current minimum version is 3.12. We could raise it a bit if we need to.

rocallahan avatar Jul 07 '25 06:07 rocallahan

It looks like that would be useable with Fedora 41 (the oldest currently supported Fedora). It has cmake-3.30.8-1.fc41.

wcohen avatar Jul 07 '25 13:07 wcohen