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

Re: [PATCH 1/1] Windows: force-recompile git.res for differing architectures

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
Nov 7, 2018, 11:25 UTC
Message-ID
<nycvar.QRO.7.76.6.1811071225230.39@tvgsbejvaqbjf.bet>
In-Reply-To
<xmqq36sdcl9t.fsf@gitster-ct.c.googlers.com>
Hi,
On Wed, 7 Nov 2018, Junio C Hamano wrote:
Show 27 quoted lines
> Johannes Sixt <j6t@kdbg.org> writes:
> 
> > Am 07.11.18 um 02:32 schrieb Junio C Hamano:
> >> Johannes Sixt <j6t@kdbg.org> writes:
> >>> On Linux, when I recompile for a different architecture, CFLAGS would
> >>> change, so I would have thought that GIT-CFLAGS were the natural
> >>> choice for a dependency. Don't they change in this case on Windows,
> >>> too?
> >>
> >> Depending on GIT-CFLAGS would have a better chance of being safe, I
> >> guess, even though it can at times be overly safe, than GIT-PREFIX,
> >> I suspect.  As a single user target in Makefile, which is only used
> >> by Dscho, who intends to stick to /mingw(32|64)/ convention til the
> >> end of time, I think the posted patch is OK, though.
> >
> > I think that it's not only Dscho who uses the target (my build
> > environment, for example, is different from Dscho's and compiles
> > git.res, too). But since the patch helps him most and doesn't hurt
> > others, it is good to go. No objection from my side.
> 
> Yeah, that was phrased poorly.  What I meant was by "only by Dscho"
> was "only by those who share the convention that GIT-PREFIX is
> changed if and only if targetting a different arch".
> 
> Anyway, I just wanted to say that GIT-PREFIX may not be precise
> enough but would give sufficient signal; GIT-CFLAGS may be a more
> cautious choice, but it may change a bit too often ;-).
I am fine with GIT-CFLAGS, too. Do you want me to "rick-roll" a v2?

Ciao, Dscho

Previous: Junio C Hamano
Message 7 of 7 in “Windows: force-recompile git.res for differing architectures”
  1. 0/1 Windows: force-recompile git.res for differing architecturesJohannes Schindelin via GitGitGadget, Nov 6, 2018
  2. 1/1 Windows: force-recompile git.res for differing architecturesJohannes Schindelin via GitGitGadget, Nov 6, 2018
  3. Johannes SixtNov 6, 2018
  4. Junio C HamanoNov 7, 2018
  5. Johannes SixtNov 7, 2018
  6. Junio C HamanoNov 7, 2018
  7. Johannes SchindelinNov 7, 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.