QubesOS / qubes-rpm-oxide

Rust libraries for processing RPM packages
4 stars 4 forks source link

Link against the versioned RPM libraries #22

Open DemiMarie opened 2 years ago

DemiMarie commented 2 years ago

This guards against breakage due to future librpm ABI changes.

DemiMarie commented 2 years ago

Marking as draft as this breaks on CentOS Stream and Fedora 25. I suspect I will need to write C wrappers anyway :(

marmarek commented 2 years ago

R4.0 build fails...

marmarek commented 2 years ago

Maybe better use alternative branch for R4.0? Or simply merge this only after R4.0 EOL (rather soon)?

DemiMarie commented 2 years ago

Maybe better use alternative branch for R4.0? Or simply merge this only after R4.0 EOL (rather soon)?

It’s not just R4.0 that fails; CentOS Stream does too. A better solution is to bind at the API level, not ABI level, by using a C shim. That is just more robust overall.