public inbox for git@vger.kernel.org
 help / color / mirror / Atom feed
From: SoutrikDas <valusoutrik@gmail.com>
To: karthik.188@gmail.com
Cc: ayu.chandekar@gmail.com, chandrapratap3519@gmail.com,
	christian.couder@gmail.com, git@vger.kernel.org,
	jltobler@gmail.com, siddharthasthana31@gmail.com,
	valusoutrik@gmail.com
Subject: Re: [GSOC Proposal] Complete and extend the remote-object-info command for git cat-file
Date: Tue, 17 Mar 2026 20:43:40 +0530	[thread overview]
Message-ID: <20260317151340.85141-1-valusoutrik@gmail.com> (raw)
In-Reply-To: <CAOLa=ZQhzvgA2bpmUgx2qMTrxFaR5_6GET8e1y+A=m2nboDAiw@mail.gmail.com>

Hi there,

> As far as I can recall, the command allowed users to enter multiple OIDs
> in a single line to reduce the to-fro with the server. But you could
> still fetch single OID info.

Yeah that was what I meant, but from Chistian Couder's feedback, I realized
that cat-file is not a good home for such a subcommand. 

> Nice, have you tried with a more recent 'master'? I assume there are
> merge conflicts?

Yup, I will add these issues in my proposal v2.

> Any idea how much work is left post v11?

From the v11 thread 
- a lot of design decision fix , like comment alignment and blank lines
- the max remote obj info logic is a bit wrong as Junio pointed out [1]
- one test case for max obj limit
- use of size_t for looping
- the placeholder check ie the even with only objectsize the checking of
formatting string is a bit incorrect [2]
- Implementing an allow list for placeholders
- print empty string for unsupported placeholders, ie those not on the
allow list
- remove usage of split_cmdline since neither url nor oid will have spaces
in them, so a strchr would suffice, I think ?

Above is for just for part 1 ie to get eric jus patch accepted

> As per the guidelines, it says
> 
>   Any work done on the Project prior to acceptance of the Project
>   Proposal will not be considered for Evaluations.

I meant like in the May 1-24 duration, which is after the acceptance
of the project ( april 30 ) but before coding officially begins (may 25)

This is the timeline on gsocs page [3]:
> April 30 - 18:00 UTC
>   Accepted GSoC contributor projects announced
> May 1 - 24
>   Community Bonding Period | GSoC contributors get to know mentors, 
>   read documentation, get up to speed to begin working on their projects
> May 25
>   Coding officially begins!

I was planning to also ask design questions in this period.

> How will you manage reviews, considering generally they take a long
> time?

I will adjust the timeline to give more time to rebase previously done work.
I was wondering... I cannot start on part 2 ie adding support for more object
fields without first integrating old work ... so about 50% of time will go to
rebasing and 30% to adding new fields ? and 20% for emergency or any mishap.

> I do agree that something like that would be useful indeed, I'm not sure
> of what that design looks like though.
> I do see benefits of this too. But I do wonder if 'git rev-list' is a
> better command for something like this.

I will clarify questions at the beginning of gsoc duration.


> What I missed from the proposal:
> 1. Where did the work from Eric and Calvin stop at, what review comments
> need to be addressed.
> 2. How do you plan to handle reviews and iterations taking time.

Will update the timeline as well as mention the current outstanding tasks,
as far as I have understood them.

Thank you for your feedback.


[1] : xmqqo6yr3wc4.fsf@gitster.g/
[2] : 20250224234720.GC729825@coredump.intra.peff.net/
[3] : https://developers.google.com/open-source/gsoc/timeline

  reply	other threads:[~2026-03-17 15:14 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-03-05 20:48 [GSOC Proposal] Complete and extend the remote-object-info command for git cat-file SoutrikDas
2026-03-15 10:11 ` SoutrikDas
2026-03-16 12:08 ` Christian Couder
2026-03-17 13:06   ` SoutrikDas
2026-03-16 20:46 ` Karthik Nayak
2026-03-17 15:13   ` SoutrikDas [this message]
2026-03-20 13:12 ` [GSoC Proposal v2] " SoutrikDas
  -- strict thread matches above, loose matches on Subject: below --
2026-03-13 10:17 [GSoC] Proposal: " Pablo
2026-03-14  5:58 ` Chandra Pratap
2026-03-14 18:31   ` Pablo
2026-03-15  9:20     ` Chandra Pratap
2026-03-16 11:21     ` Christian Couder
2026-03-16 21:38     ` Karthik Nayak
2026-03-18 10:45       ` Pablo

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=20260317151340.85141-1-valusoutrik@gmail.com \
    --to=valusoutrik@gmail.com \
    --cc=ayu.chandekar@gmail.com \
    --cc=chandrapratap3519@gmail.com \
    --cc=christian.couder@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=jltobler@gmail.com \
    --cc=karthik.188@gmail.com \
    --cc=siddharthasthana31@gmail.com \
    /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