Hi Ivan, On Tue, 21 Mar 2017, Ivan Tham wrote: > Stefan Beller <sbeller@xxxxxxxxxx> wrote: > > On Mon, Mar 20, 2017 at 9:41 AM, Ivan Tham <pickfire@xxxxxxxxxx> wrote: > > > I am Ivan Tham. Currently studying in Computer Science in APIIT > > > Malaysia. I am interested particapate in Google Summer of Code 2017 > > > under git organization. I would like to attempt "Add more builtin > > > patterns for userdiff" particularly for shell for my microproject. > > > > I'd love to see proper shell support! Although there is already some > > support for shell (by looking at diffs on our test suite) ? So I am > > not sure what there is left to do? Can you clarify what you're trying > > there? > > Are you sure about that? From what I had looked into userdiff.c, there > is no support for shell. There just a recent patch for [go patterns][0]. > Or perhaps I should have rename it as "userdiff.c: patterns for "shell" > language"? I also could not find any shell patterns in the userdiff code... > > > I am interested to work on "Convert interactive rebase to C" > > > > +cc Johannes, who recently worked on rebase and the sequencer. Glad you are interested! Please note that large parts of the interactive rebase are already in C now, but there is enough work left in that corner. > > > aiming to port most builtins stuff to C in which we can reduce the > > > size of git. Additionally, I would also like to convert scripts to > > > builtins as an additional milestone. Careful. It is a ton of work to get the rebase -i conversion done, and then a ton of work to get it integrated. That will fill 3 months, very easily. > > > What do you think of these projects? Would it collide with Valery > > > Tolstov's Shell to Builtins proposal? I missed that proposal, and could only find submodule-related mails on the public-inbox server. Care to provide a pointer? > > Curious why all people ask about colliding with Valerys proposal here? > > I do not think it would collide, as submodules and rebase are very > > different areas of the code base. Indeed ;-) Ciao, Johannes