Thread (168 messages) 168 messages, 18 authors, 2026-01-21

Re: [PATCH v6 0/9] Introduce Rust and announce that it will become mandatory

From: Ezekiel Newren <hidden>
Date: 2025-09-23 20:15:48

On Tue, Sep 23, 2025 at 3:45 AM Patrick Steinhardt [off-list ref] wrote:
Your patch series has 2 critical problems:
  * meson doesn't check for "is windows and using msvc" -> <crate>.lib
else lib<crate>.a
  * Using the name "git" for the crate is problematic because both
Make and Meson already produce libgit.a which is different from the
libgit.a that cargo is producing. Change the name in Cargo.toml from
"git" to "gitcore".

I created some temporary patches on top of this patch series that
always forces Rust and then pushed it to GitHub. The only target that
failed was windows building with meson + msvc, everything else
including the 32 bit linux target passed.

I have some nitpicks about varint, but they're not worth mentioning
here. Fix those 2 points and you'll have my seal of approval for this
patch series.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help