Re: [PATCH 10/14] rust: add a build.rs script for tests
From: Patrick Steinhardt <hidden>
Date: 2025-10-30 06:26:42
On Wed, Oct 29, 2025 at 04:12:05PM -0700, Junio C Hamano wrote:
Ezekiel Newren [off-list ref] writes:quoted
quoted
Hmm, I recall Ezekiel earlier arguing to roll reftable and xdiff libraries into libgit.a as it is a lot more cumbersome to have to link with multiple libraries (sorry, I may be misremembering and do not have reference handy), but if the above is all it takes to link with these, perhaps it is not such a huge deal?I think Brian might have written this before my series was merged in. ...quoted
I am a bit confused. XDIFF_LIB and REFTABLE_LIB are gone from Makefile on 'master' already. Perhaps we should revert earlier series from him?... I don't think we should revert my series.The order of events does not really matter, does it? If we can happily link with more than one libraries [*], it would give us a much more pleasant developer experience than having to roll everything into a single library archive, no? Or are you saying that the way this series links these multiple libraries somehow does not work? You somehow manged to confuse me even more ... X-<.
Simplification was only one of the reasons we had. The other reason was to unify how Meson and Makefiles build libgit.a, where the former wasn't ever building separate xdiff and reftable libraries. The question I have here is what the benefit would be to have separate libraries. I don't really see the "more pleasant developer experience", and I'm not really aware of any other benefits. So personally, I'm all for the build system simplification that Ezekiel introduced. Patrick