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

Re: gitweb: pathinfo feature breaks commitdiff/raw links

From
Wincent Colaiuta <win@wincent.com>
Date
Apr 17, 2008, 09:57 UTC
Message-ID
<869CFE40-B245-4744-A9D5-0F3FA623633F@wincent.com>
In-Reply-To
<20080417092339.GA17016@piper.oerlikon.madduck.net>
El 17/4/2008, a las 11:23, martin f krafft escribió:
Show 26 quoted lines
> Hello,
>
> as per suggestion in gitweb.cgi, I put the following into my
> gitweb.conf:
>
>  $feature{'pathinfo'}{'default'} = [1];
>
> This mostly works, but James Vega recently found a corner case where
> this is broken: the raw link of a commitdiff page, such as
>
>  http://git.madduck.net/v/code/mailplate.git?a=commitdiff;h=f7d2365ee0ab439217083dd613daa4757a4e3d6e
>
> suddenly becomes
>
>  http://git.madduck.net/v/ARRAY(0xbc3b70)?a=commitdiff_plain;h=f7d2365ee0ab439217083dd613daa4757a4e3d6e
>                           ^^^^^^^^^^^^^^^
> when the feature is turned on. It works when it's off.
>
> I discussed this a bit on IRC, with the following bottom line:
>
>  < jengelh> if you see ARRAY... in the url, and [1] in the source
>  the wise Perl person says "ah, of course"
>
> Unfortunately, I am neither wise nor a Perl person, so I resort to
> just pointing this out to you and can offer to test any patches on
> this live server.
Are you running an older version of gitweb? One which predates 7863c61?

Cheers, Wincent

Previous: martin f krafftNext: martin f krafft
Message 2 of 3 in “gitweb: pathinfo feature breaks commitdiff/raw links”
  1. martin f krafftApr 17, 2008
  2. Wincent ColaiutaApr 17, 2008
  3. martin f krafftApr 17, 2008

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.