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

Re: [PATCH v2b 00/16, 2 updates] Make the msvc-build scripts work again

From
Junio C Hamano <gitster@pobox.com>
Date
Jul 21, 2015, 20:04 UTC
Message-ID
<xmqq1tg1uvjk.fsf@gitster.dls.corp.google.com>
In-Reply-To
<xmqqvbdduxd5.fsf@gitster.dls.corp.google.com>
Junio C Hamano <gitster@pobox.com> writes:
Show 11 quoted lines
> "Philip Oakley" <philipoakley@iee.org> writes:
>
>> ... Ideally, if part of this
>> mainstream Git, it would get picked up automatically by them
>> (rather than being local 'fixes' endlessly carried forward).
>
> Actually, that is not "ideal", but what I want to avoid.
>
> As I do not do Windows, it simply is wrong for me to apply changes
> that are very likely to affect Windows folks without seeing their
> support first,...

Just to clarify this part. I do not do RedHat, Solaris or OSX, either. Also MSVC may not be what GfW folks primarily target.

But the thing is that (1) Windows is so much different, and (2) GfW folks are much more qualified to judge platform-specific issues on Windows than I am.

Even though I may still need to have a say in the overall structure of the changes to the upstream tree coming from them (e.g. "Don't sprinkle #ifdef all over the generic code; instead add a wrapper or two in compat/ to keep the generic code generic" is something I may tell them when rejecting a change forwarded to me), I trust them a lot more than I trust myself when it comes to what the changes do in the platform-specific part and how they do it.

Thanks.
Previous: Junio C HamanoNext: Philip Oakley
Message 7 of 9 in “Make the msvc-build scripts work again”
  1. 00/16 Make the msvc-build scripts work againPhilip Oakley, Jul 20, 2015
  2. engine.pl: ignore invalidcontinue.obj which is known to MSVCPhilip Oakley, Jul 20, 2015
  3. engine.pl: delete the captured stderr file if emptyPhilip Oakley, Jul 20, 2015
  4. Junio C HamanoJul 20, 2015
  5. Philip OakleyJul 21, 2015
  6. Junio C HamanoJul 21, 2015
  7. Junio C HamanoJul 21, 2015
  8. Philip OakleyJul 21, 2015
  9. Ramsay JonesJul 22, 2015

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.