Thread (9 messages) 9 messages, 3 authors, 2026-03-08

Re: [PATCH v2] t3310: avoid hiding failures from rev-parse in command substitutions

From: Francesco Paparatto <hidden>
Date: 2026-03-05 22:42:24

Possibly related (same subject, not in this thread)

Junio C Hamano [off-list ref] writes:
Hmph, did I suggest this?  I know Eric had comments on a previous
round, and the improvements in this patch seems to be influenced a
lot stronger by his input than whatever I may have said.
Sorry about the Suggested-by line. I added it because of your earlier
comment here:

    https://public-inbox.org/git/xmqqv7fioueg.fsf@gitster.g/

but you're right that the concrete changes in this version were mostly
influenced by Eric's review, so I'll drop that trailer.
On top of what commit is this patch designed to apply?
This patch is based on top of:

    b3ec5aec2367262f464a33d6eab7a9f49fd413f1
    ("t3310: replace test -f/-d with test_path_is_file/test_path_is_dir")

I'll reroll and send a v3 of the patch shortly.

Thanks,
Francesco
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help