Thread (1 message) 1 message, 1 author, 2024-09-08

Re: [PATCH v3 6/6] Makefile: add option to build and test libgit-rs and libgit-rs-sys

From: Junio C Hamano <hidden>
Date: 2024-09-08 21:33:22

Sean Allred [off-list ref] writes:
Calvin Wan [off-list ref] writes:
quoted
Add libgitrs, libgitrs-sys, libgitrs-test, and libgitrs-sys-test targets
to their respective Makefiles so they can be built and tested without
having to run cargo build/test.
I feel like clippy should be run as part of these somehow, but I'm not
sure where.
quoted
+libgitrs-sys:
+	$(QUIET)(\
+		cd contrib/libgit-rs/libgit-sys && \
+		cargo build \
+	)
+.PHONY: libgitrs
+libgitrs:
+	$(QUIET)(\
+		cd contrib/libgit-rs && \
+		cargo build \
+	)
We should definitely be setting `RUSTFLAGS=-Dwarnings` as an analog to
`-Wall` in the C world, no? These crates should build without warnings.

Very excited to see this work; thank you for putting the time into it
:-)
Thanks for helping.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help