git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [RFC] Contributing to Git (on Windows)

From
Junio C Hamano <gitster@pobox.com>
Date
Mar 5, 2018, 20:23 UTC
Message-ID
<xmqqwoyq8e61.fsf@gitster-ct.c.googlers.com>
In-Reply-To
<nycvar.QRO.7.76.6.1803051711210.20700@ZVAVAG-6OXH6DA.rhebcr.pbec.zvpebfbsg.pbz>
Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
Show 14 quoted lines
>> > "What if they say my code is not good enough?"
>> 
>> Sure, though there is something implied in what is Junio is saying
>> that is useful for such people.
>> 
>> It is patience.  It is the message that if you miss a portability bug,
>> we won't be disappointed in you, and it in fact happens all the time
>> to the best of contributors.
>> 
>> If there's a straightforward way to convey that in the text, I agree
>> with Junio that it's worth conveying.
>
> That is not how I read Junio's statement. I read it more like "If your
> code is flawed, we'll let you know."
I think you are talking about this part?
    In fact, portability issues in a patch originally written for a
    platform is rather quickly discovered if the original platform
    is more minor than the others, so while it is good advice to
    test your ware before you make it public, singling out the
    portability issues may not add much value.

It's more like "Just like everybody else you are expected to be imperfect, but those on list can and will help spot and fix if you made mistakes. Do not worry too much about things like portability over to a system you usually do not work on."

The other side of the coin is that we are expected to be imperfect, so even if your code is flawed, we may not even notice, so we may not let you know. It's mutual process in which we try to help each other.

Previous: Eric Sunshine
Message 14 of 14 in “[RFC] Contributing to Git (on Windows)”
  1. Derrick StoleeMar 1, 2018
  2. Jonathan NiederMar 2, 2018
  3. Derrick StoleeMar 2, 2018
  4. Junio C HamanoMar 2, 2018
  5. Johannes SchindelinMar 3, 2018
  6. Jonathan NiederMar 3, 2018
  7. Derrick StoleeMar 5, 2018
  8. Jonathan NiederMar 5, 2018
  9. Johannes SchindelinMar 7, 2018
  10. Johannes SchindelinMar 5, 2018
  11. Jonathan NiederMar 5, 2018
  12. Jonathan NiederMar 5, 2018
  13. Eric SunshineMar 5, 2018
  14. Junio C HamanoMar 5, 2018

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.