All of lore.kernel.org
 help / color / mirror / Atom feed
From: Junio C Hamano <gitster@pobox.com>
To: Tapasweni Pathak <tapaswenipathak@gmail.com>
Cc: Christian Couder <christian.couder@gmail.com>,
	Hariom verma <hariom18599@gmail.com>,
	Taylor Blau <me@ttaylorr.com>,
	git@vger.kernel.org
Subject: Re: Unify ref-filter formats with other pretty formats: GSoC'22
Date: Fri, 22 Apr 2022 11:03:08 -0700	[thread overview]
Message-ID: <xmqqilr19esj.fsf@gitster.g> (raw)
In-Reply-To: <CALOmtcWTw=bo=AqGBN8z9Hf602Peu9AVngwKUZZsCt0cr8jbDg@mail.gmail.com> (Tapasweni Pathak's message of "Fri, 22 Apr 2022 15:58:28 +0530")

Tapasweni Pathak <tapaswenipathak@gmail.com> writes:

>> > > As you can see "contrib/rerere-train.sh" is a 102 line long shell
>> > > script, so converting it to a new `train` command in
>> > > "builtin/rerere.c" seems like a not very difficult and not too big
>> > > task (especially if done the same way as the script). Junio will
>> > > likely be involved in this, as it looks like he is interested in it.
>> >
>> > Can I take it up immediately, if Junio is not working on it?
>> > ++gitster@pobox.com. Hi Junio. Do you suggest otherwise?
>>
>> I don't want to speak for him, but I don't think he is working on it.
>> In fact, as he suggested it for a GSoC, I think he doesn't plan to
>> work on it soon. Also he might not answer often to emails that aren't
>> also sent to the mailing list.

s/answer/see/ actually.  Other than that, everything you said is
correct, Christian ;-)  Mails to this mailbox that is not also going
to the list are usually sent to the spambox.

>> So if you want to work on it, I would suggest starting right away. You
>> can of course send an email on the mailing list to let people know
>> that you want to start working on this and to check if anyone is
>> already working on it or has suggestions about it.

Yes, that is a good advice.

If we were doing 'git rerere train', especially outside GSoC, we
would need to make sure we won't have to redo it later.

A GSoC "easy" version of "git rerere train" would probably write a
loop in C that repeatedly calls get_revision() to enumerate each
merge commit in the given range, and in each iteration, spawns "git
checkout --detach" to update the working tree and the index to that
of its first parent, spawns "git merge" to try merging the other
parent, and when it fails in conflict, updates the index and the
working tree files to match what is recorded in the merge commit,
and spawns "git rerere" to record that resolution.

The thing is, many people on the list are unhappy with the approach
various past GSoc and other tasks took to rewrite scripted Porcelain
commands to C, spawning individual Git command just like the
original scripts did.  These people will be very unhappy with such a
rewrite, I suspect, and would want it see done totally differently.

Now we have ort merge machinery that can be used to implement
in-core merges without touching neither the working tree or the
index, a proper C rewrite of "git rerere train" should work quite
differently from the scripted version, so "easy" version would
likely be discarded and rewritten later.

>> > > Another thing that was discussed recently is that the git-scm.org web
>> > > site could be converted to not use Heroku anymore. If you prefer
>> > > working on this kind of web oriented thing, this could be another
>> > > possibility.
>> >
>> > I have worked a lot on web-based projects, SAAS, IAAS, PAAS, FAAS,
>> > would it be better to share a proposal of comparison of those keeping
>> > git-scm.org in mind, or is it pre-decided where git-scm.org will make
>> > its new home/work?
>> >
>> > This shouldn't take more than 3 working days with no downtimes.
>>
>> Nice that you have some experience in this!

Yup.  This is not in my area of expertise, but it is very worthy
work.  If we can wean ourselves off of Heroku (the only thing we are
doing is to serve a pretty much static website) sooner, it would be
very good.

>> I added Taylor in Cc as he is involved in this and might answer your
>> questions much better than me. Now if you are interested in it, I
>> suggest starting right away and perhaps also sending an email to the
>> mailing list asking your questions and asking for advice there.
>
> Hi Taylor. Great if you can let me know what you want to see
> happening, and if we can discuss tech specifications, you have already
> decided or plan to.

  reply	other threads:[~2022-04-22 18:10 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <CALOmtcX4+ON7+J7K9X4rSYfWYnkciJdFE=v=Oeb7rUqwRt8_JA@mail.gmail.com>
     [not found] ` <CA+CkUQ8hf5m0GZEwcyqLgxMw1vavYey_R4P7oq8=OXc7iAj6MA@mail.gmail.com>
     [not found]   ` <CALOmtcW=2skmUPXpnano=21i38b7sq2MEWQjH5yF2jLacZ8sQw@mail.gmail.com>
     [not found]     ` <CA+CkUQ9ZnBXU8E=XKTk95hwM4qtZ1M_nX3LTCr5q5njPG3Es5Q@mail.gmail.com>
     [not found]       ` <CALOmtcX7amyw3cAuthMPuagUVzJghybHFNy8ciB50yC5SCUwow@mail.gmail.com>
     [not found]         ` <CA+CkUQ-YgmRa7ysP30GbkX07Cu_=EM5X66w3Vk=TpshP9xoi_w@mail.gmail.com>
     [not found]           ` <CAP8UFD2uUFveiGDAbxObDOh_krmexuLe860Gu397e9KddH0UCg@mail.gmail.com>
     [not found]             ` <CALOmtcVN8K59Zkv7XKM7w2zFCbdWKfKr=97-E3306nQHm4Zw=A@mail.gmail.com>
     [not found]               ` <CAP8UFD1kinAwq7AL68QXqFR_dxkNFaTp2vP_DUU0KKqkbsXCDQ@mail.gmail.com>
     [not found]                 ` <CALOmtcWaQgrPTkRA6F8bL3Hp64cOxYrB4xHUa1WC+P34ZdFEbw@mail.gmail.com>
     [not found]                   ` <CAP8UFD1HGbbmJWB_TksGVLa-7nNBz5pdFi59Y7LP3EODF9Lztg@mail.gmail.com>
     [not found]                     ` <CALOmtcWFwfV11Vxf1xd_fC=HYGPNO1PtKT=E=-A104cGnAbv4g@mail.gmail.com>
     [not found]                       ` <CAP8UFD29NgffUBz0CMKyt901Z7Bj5osoBT-4Mfxfp0h_4Qo+Qw@mail.gmail.com>
2022-04-22 10:28                         ` Unify ref-filter formats with other pretty formats: GSoC'22 Tapasweni Pathak
2022-04-22 18:03                           ` Junio C Hamano [this message]
2022-04-26  9:54                           ` Christian Couder
2022-05-31 16:24                             ` Tapasweni Pathak
     [not found]                               ` <YxvBBZkN6yxUquou@nand.local>
2022-09-12 13:15                                 ` Tapasweni Pathak
     [not found]                         ` <CAP8UFD1f6KCO+=+ZoUFr06XUgEZYOZi8G0V+QKCK=bNbAE9VrQ@mail.gmail.com>
2022-04-22 10:39                           ` Tapasweni Pathak
2022-04-25  8:37                             ` Christian Couder
2022-05-31 16:13                               ` Tapasweni Pathak
     [not found]                           ` <Ymc2Y+U4HQpgD7PE@nand.local>
2022-05-31 16:21                             ` Tapasweni Pathak

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=xmqqilr19esj.fsf@gitster.g \
    --to=gitster@pobox.com \
    --cc=christian.couder@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=hariom18599@gmail.com \
    --cc=me@ttaylorr.com \
    --cc=tapaswenipathak@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.