git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Junio C Hamano <gitster@pobox.com>
To: Seyi Kuforiji <kuforiji98@gmail.com>
Cc: git@vger.kernel.org,  ps@pks.im,  phillip.wood@dunelm.org.uk
Subject: Re: [PATCH v4 06/10] t/unit-tests: convert reftable table test to use clar
Date: Thu, 05 Jun 2025 10:04:10 -0700	[thread overview]
Message-ID: <xmqqikla86id.fsf@gitster.g> (raw)
In-Reply-To: <20250605140644.239199-7-kuforiji98@gmail.com> (Seyi Kuforiji's message of "Thu, 5 Jun 2025 15:06:40 +0100")

Seyi Kuforiji <kuforiji98@gmail.com> writes:

> diff --git a/Makefile b/Makefile
> index 3cccc73073..0227fdb3e1 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -1368,6 +1368,7 @@ CLAR_TEST_SUITES += u-reftable-basics
> ...
> @@ -3972,4 +3972,4 @@ $(LIBGIT_HIDDEN_EXPORT): $(LIBGIT_PARTIAL_EXPORT)
>  	$(OBJCOPY) --localize-hidden $^ $@
>  
>  contrib/libgit-sys/libgitpub.a: $(LIBGIT_HIDDEN_EXPORT)
> -	$(AR) $(ARFLAGS) $@ $^
> +	$(AR) $(ARFLAGS) $@ $^
> \ No newline at end of file

I am sure I pointed this breakage out, locally fixed it in my tree,
and told you about the consequences of such breakage.  Next time
please make sure you fetch back from what upstream has its 'seen'
and start your work from that state.

In addition, perhaps you should fix your editor settings to avoid
files with such an incomplete line at the end.  There may be other
incomplete lines in this seris.

And a possible #leftoverbits is probably to teach the whitespace
error checking code to optionally flag a new incomplete line as an
error (this is obviously way outside the topic of this series).

Thanks.

  reply	other threads:[~2025-06-05 17:04 UTC|newest]

Thread overview: 33+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-06-05 14:06 [PATCH v4 00/10] t/unit-tests: convert unit-tests to use clar Seyi Kuforiji
2025-06-05 14:06 ` [PATCH v4 01/10] t/unit-tests: implement clar specific reftable test helper functions Seyi Kuforiji
2025-06-05 14:06 ` [PATCH v4 02/10] t/unit-tests: convert reftable basics test to use clar test framework Seyi Kuforiji
2025-06-05 14:06 ` [PATCH v4 03/10] t/unit-tests: convert reftable block test to use clar Seyi Kuforiji
2025-06-05 14:06 ` [PATCH v4 04/10] t/unit-tests: convert reftable merged " Seyi Kuforiji
2025-06-05 14:06 ` [PATCH v4 05/10] t/unit-tests: convert reftable pq " Seyi Kuforiji
2025-06-05 14:06 ` [PATCH v4 06/10] t/unit-tests: convert reftable table " Seyi Kuforiji
2025-06-05 17:04   ` Junio C Hamano [this message]
2025-06-05 14:06 ` [PATCH v4 07/10] t/unit-tests: convert reftable readwrite " Seyi Kuforiji
2025-06-05 14:06 ` [PATCH v4 08/10] t/unit-tests: convert reftable record " Seyi Kuforiji
2025-06-05 14:06 ` [PATCH v4 09/10] t/unit-tests: convert reftable stack " Seyi Kuforiji
2025-07-24  6:47   ` Patrick Steinhardt
2025-06-05 14:06 ` [PATCH v4 10/10] t/unit-tests: finalize migration of reftable-related tests Seyi Kuforiji
2025-07-23 21:30 ` [PATCH v4 00/10] t/unit-tests: convert unit-tests to use clar Junio C Hamano
2025-07-24  6:47   ` Patrick Steinhardt
2025-07-24 13:28 ` [PATCH v5 0/5] " Seyi Kufoiji
2025-07-24 13:28   ` [PATCH v5 1/5] t/unit-tests: convert reftable table test " Seyi Kufoiji
2025-07-24 13:28   ` [PATCH v5 2/5] t/unit-tests: convert reftable readwrite " Seyi Kufoiji
2025-07-24 13:28   ` [PATCH v5 3/5] t/unit-tests: convert reftable record " Seyi Kufoiji
2025-07-24 13:28   ` [PATCH v5 4/5] t/unit-tests: convert reftable stack " Seyi Kufoiji
2025-07-24 14:28 ` [PATCH v5 00/10] t/unit-tests: convert unit-tests " Seyi Kufoiji
2025-07-24 14:28   ` [PATCH v5 01/10] t/unit-tests: implement clar specific reftable test helper functions Seyi Kufoiji
2025-07-24 14:28   ` [PATCH v5 02/10] t/unit-tests: convert reftable basics test to use clar test framework Seyi Kufoiji
2025-07-24 14:28   ` [PATCH v5 03/10] t/unit-tests: convert reftable block test to use clar Seyi Kufoiji
2025-07-24 14:28   ` [PATCH v5 04/10] t/unit-tests: convert reftable merged " Seyi Kufoiji
2025-07-24 14:28   ` [PATCH v5 05/10] t/unit-tests: convert reftable pq " Seyi Kufoiji
2025-07-24 14:28   ` [PATCH v5 06/10] t/unit-tests: convert reftable table " Seyi Kufoiji
2025-07-24 14:28   ` [PATCH v5 07/10] t/unit-tests: convert reftable readwrite " Seyi Kufoiji
2025-07-24 14:28   ` [PATCH v5 08/10] t/unit-tests: convert reftable record " Seyi Kufoiji
2025-07-24 14:28   ` [PATCH v5 09/10] t/unit-tests: convert reftable stack " Seyi Kufoiji
2025-07-24 14:28   ` [PATCH v5 10/10] t/unit-tests: finalize migration of reftable-related tests Seyi Kufoiji
2025-07-24 14:41   ` [PATCH v5 00/10] t/unit-tests: convert unit-tests to use clar Patrick Steinhardt
2025-07-24 18:48     ` Junio C Hamano

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=xmqqikla86id.fsf@gitster.g \
    --to=gitster@pobox.com \
    --cc=git@vger.kernel.org \
    --cc=kuforiji98@gmail.com \
    --cc=phillip.wood@dunelm.org.uk \
    --cc=ps@pks.im \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).