threads / discuss / 55968

[GSoC] My Git Dev Blog - Week 5

Subject: [GSoC] My Git Dev Blog - Week 5

## tl;dr

5 messages between Jun 19, 2021 and Jun 21, 2021.

replies: 4people: 3as markdown or json

Atharva Raykar· Jun 19, 2021, 13:32 UTC · lore

Here's the latest instalment of my weekly Git blog: http://atharvaraykar.me/gitnotes/week5

A peek at what's inside:

 * The blog starts with a summary of all the relevant things
   that have happened so far in my time with Git, including
   the status of my submodule-related work.

 * A question I had about reading the index into memory
   (any kind of help would be appreciated!):
   http://atharvaraykar.me/gitnotes/week5#some-challenges-with-the-changes-that-are-cooking

 * Reflections, mostly as a note to myself and to people who
   might be in a similar position as me in the future.

Have a great weekend :)

--- Atharva Raykar ಅಥರ್ವ ರಾಯ್ಕರ್ अथर्व रायकर

Christian Couder· Jun 20, 2021, 05:52 UTC · re: Atharva Raykar · lore

Re: [GSoC] My Git Dev Blog - Week 5

On Sat, Jun 19, 2021 at 3:32 PM Atharva Raykar <raykar.ath@gmail.com> wrote:

>
> Here's the latest instalment of my weekly Git blog:
> http://atharvaraykar.me/gitnotes/week5

Great!

Just a note about:

> if (ps_matched[0]) {
>      if (!force)
>          die(_("'%s' already exists in the index"), path);
>      else if (!S_ISGITLINK(active_cache[i]->ce_mode))

The "else" above is not needed.

>          die(_("'%s' already exists in the index "
>              "and is not a submodule"), path);
>      break;
> }

Show 20 quoted lines

> A peek at what's inside:
>
>  * The blog starts with a summary of all the relevant things
>    that have happened so far in my time with Git, including
>    the status of my submodule-related work.
>
>  * A question I had about reading the index into memory
>    (any kind of help would be appreciated!):
>    http://atharvaraykar.me/gitnotes/week5#some-challenges-with-the-changes-that-are-cooking
>
>  * Reflections, mostly as a note to myself and to people who
>    might be in a similar position as me in the future.
>
> Have a great weekend :)
>
> ---
> Atharva Raykar
> ಅಥರ್ವ ರಾಯ್ಕರ್
> अथर्व रायकर
>

Atharva Raykar· Jun 21, 2021, 06:20 UTC · re: Christian Couder · lore

Re: [GSoC] My Git Dev Blog - Week 5

On 20-Jun-2021, at 11:22, Christian Couder <christian.couder@gmail.com> wrote:

Show 16 quoted lines

> 
> On Sat, Jun 19, 2021 at 3:32 PM Atharva Raykar <raykar.ath@gmail.com> wrote:
>> 
>> Here's the latest instalment of my weekly Git blog:
>> http://atharvaraykar.me/gitnotes/week5
> 
> Great!
> 
> Just a note about:
> 
>> if (ps_matched[0]) {
>>     if (!force)
>>         die(_("'%s' already exists in the index"), path);
>>     else if (!S_ISGITLINK(active_cache[i]->ce_mode))
> 
> The "else" above is not needed.

Noted. Thanks!

Kaartic Sivaraam· Jun 20, 2021, 19:16 UTC · re: Atharva Raykar · lore

Re: [GSoC] My Git Dev Blog - Week 5

On 19/06/21 7:02 pm, Atharva Raykar wrote:

> Here's the latest instalment of my weekly Git blog:
> http://atharvaraykar.me/gitnotes/week5
>

Nice blog.

Show 10 quoted lines

> A peek at what's inside:
> 
>   * The blog starts with a summary of all the relevant things
>     that have happened so far in my time with Git, including
>     the status of my submodule-related work.
> 
>   * A question I had about reading the index into memory
>     (any kind of help would be appreciated!):
>     http://atharvaraykar.me/gitnotes/week5#some-challenges-with-the-changes-that-are-cooking
>

I've tried to clarify this to an extent in a comment [1] on your branch.

A few other things:

> After Rafael and Eric commented on my patch, I forgot to CC them when
> I rerolled. New contributors: please check your CC’s before hitting send!

To add to that, Cc-ing people in the e-mail of all the patches in the series and not just in the cover letter would be nice too. :) Of course, if you have a specific reason to not do that, then it's fine.

> And my other blooper was to forget signing off one of my patches. Oops.

Are you aware of `format.signOff` [2] which could save you from issues like these?

[1]: https://github.com/tfidfwastaken/git/commit/3c46c108a195c42edaab939ab6dd6731e52aefc3#r52396546

[2]: https://git-scm.com/docs/git-config#Documentation/git-config.txt-formatsignOff

-- 
Sivaraam

Atharva Raykar· Jun 21, 2021, 06:28 UTC · re: Kaartic Sivaraam · lore

Re: [GSoC] My Git Dev Blog - Week 5

On 21-Jun-2021, at 00:46, Kaartic Sivaraam <kaartic.sivaraam@gmail.com> wrote:

Show 17 quoted lines

> 
> On 19/06/21 7:02 pm, Atharva Raykar wrote:
>> Here's the latest instalment of my weekly Git blog:
>> http://atharvaraykar.me/gitnotes/week5
>> 
> 
> Nice blog.
> 
>> A peek at what's inside:
>>  * The blog starts with a summary of all the relevant things
>>    that have happened so far in my time with Git, including
>>    the status of my submodule-related work.
>>  * A question I had about reading the index into memory
>>    (any kind of help would be appreciated!):
>>    http://atharvaraykar.me/gitnotes/week5#some-challenges-with-the-changes-that-are-cooking
> 
> I've tried to clarify this to an extent in a comment [1] on your branch.

Thanks, I'll have a look.

Show 8 quoted lines

> A few other things:
> 
>> After Rafael and Eric commented on my patch, I forgot to CC them when
>> I rerolled. New contributors: please check your CC’s before hitting send!
> 
> To add to that, Cc-ing people in the e-mail of all the patches in the series
> and not just in the cover letter would be nice too. :) Of course, if you have
> a specific reason to not do that, then it's fine.

Okay, I will remember to do this.

>> And my other blooper was to forget signing off one of my patches. Oops.
> 
> Are you aware of `format.signOff` [2] which could save you from issues
> like these?

Thanks, this should have been in my config all along. Unfortunately, it won't be possible to automate adding the other trailers in my patches, unless Git learns to read my mind ;-)

Show 7 quoted lines

> 
> [1]: https://github.com/tfidfwastaken/git/commit/3c46c108a195c42edaab939ab6dd6731e52aefc3#r52396546
> 
> [2]: https://git-scm.com/docs/git-config#Documentation/git-config.txt-formatsignOff
> 
> -- 
> Sivaraam