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

Re: [PATCH] perl bindings fix compilation errors

From
Junio C Hamano <junkio@cox.net>
Date
Sep 27, 2006, 02:40 UTC
Message-ID
<7vwt7qjal5.fsf@assigned-by-dhcp.cox.net>
In-Reply-To
<Pine.LNX.4.63.0609261031430.25371@wbgn013.biozentrum.uni-wuerzburg.de>
Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
Show 11 quoted lines
> Hi,
>
> On Mon, 25 Sep 2006, Junio C Hamano wrote:
>
>> It is certainly incorrect to say that it is a driver error to
>> git-pull (or git-checkout to switch branches) without running
>> "make clean" first to get rid of perl/Git.c while your Makefile
>> still knows it is a generated file.  Our Makefile should be more
>> helpful.
>
> How about this:

The problem is on which branch we would apply this and how long we will carry it. I suspect you would have to have it forever and everywhere. After all your bisect can hit a version with Git.xs and then next build suggested by bisect may be a version without it. And the user is doing that bisect long after we release version 47.

The worst part is that the version bisect suggests that does not have Git.xs may be a version in the past -- which we obviously cannot apply your patch to.

Previous: Johannes SchindelinNext: Johannes Schindelin
Message 7 of 8 in “perl bindings fix compilation errors”
  1. perl bindings fix compilation errorsAndy Whitcroft, Sep 25, 2006
  2. Johannes SchindelinSep 25, 2006
  3. Andy WhitcroftSep 25, 2006
  4. Johannes SchindelinSep 25, 2006
  5. Junio C HamanoSep 26, 2006
  6. Johannes SchindelinSep 26, 2006
  7. Junio C HamanoSep 27, 2006
  8. Johannes SchindelinSep 27, 2006

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.