Jeff King <peff@xxxxxxxx> writes: > On Thu, May 22, 2025 at 02:06:26AM -0400, Jeff King wrote: > >> As a side note, we do still use the SANITIZE_LEAK prereq in a few >> places, and I believe that it does actually work. It might be nice to >> clean up any leaks in those few spots, though we probably want to keep >> the prereq around forever (e.g., if you introduce a test which shows off >> a leak and then fixes it later). All orthogonal to your patch, though, >> which looks good to me. > > Ah, nevermind. These are all due to 8415595203 (t5601: work around leak > sanitizer issue, 2024-11-20). They are leak-free (and the tests pass on > my system), but apparently some upstream bug can cause issues. > > -Peff Yup. Someday in the future we can cleanup the whole thing. That would be nice! - Karthik
Attachment:
signature.asc
Description: PGP signature
