Re: [PATCH v3 0/6] [GSoC] bitmap: integrate a lookup table extension to the bitmap format

From: Abhradeep Chakraborty <chakrabortyabhradeep79@gmail.com>
Date: Jul 7, 2022, 08:48 UTC
Message-ID: <20220707084818.79881-1-chakrabortyabhradeep79@gmail.com>
In-Reply-To: <xmqqiloagi80.fsf@gitster.g>

Junio C Hamano <gitster@pobox.com> wrote:

> > This is the second version which addressed all (I think) the reviews. Please
> > notify me if some reviews are not addressed :)
>
> Is this the second version that is labeled as "v3" ;-)?

Hi junio,

No, it is actually the third version. I forgot to update the cover letter :P.

I am using Github's gitgitgadget to submit PRs and it uses PR description as the cover letter.

So before submitting a new version of patchset, PR description must be updated which I missed this time.

I wrote a reply comment[1] where you can find a summary of all the new changes.

[1] https://lore.kernel.org/git/20220704163506.76162-1-chakrabortyabhradeep79@gmail.com/

Show 5 quoted lines

> >  Documentation/technical/bitmap-format.txt |  39 ++
>
> I haven't tried merging it yet, but doesn't [1/6] overlap with and
> semantically depend on your other series that touch the formatting
> of this file?

Correct, [1/6] indeed depends on my previous patch series[2] and it is assuming that that series has already been merged. As far as it seems, it will not create any merge conflicts while merging but I am not sure. This would be interesting to see.

[2] https://lore.kernel.org/git/pull.1246.v4.git.1655355834.gitgitgadget@gmail.com/

Thanks :)