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

Re: gitweb bug: broken "next" and other links

From
Jakub Narebski <jnareb@gmail.com>
Date
Feb 12, 2008, 11:48 UTC
Message-ID
<200802121248.49002.jnareb@gmail.com>
In-Reply-To
<200802121239.06287.jnareb@gmail.com>
Jakub Narebski wrote:
Show 6 quoted lines
> Actually the error is there even without using 'pathinfo' feature,
> namely if you use pathinfo to provide project, for example:
>   http://www.example.com/w/project.git?a=shortlog
> then 'next' link on the page lacks project (instead of having
> dereferenced anonymous array reference, i.e. ARRAY(sth)), like below
>   http://www.example.com/w/?a=shortlog;pg=1

Errr... what I forgot to write here is that I have checked that the patch send (the first version) fixes this bug, as checked "by hand", i.e. by examining gitweb running as Apache CGI module.

> But I could not come up with automated test which would _fail_
> on this error. So please check, and reply if this patch helps.
-- 
Jakub Narebski
Poland
Previous: Jakub NarebskiNext: Wincent Colaiuta
Message 6 of 16 in “gitweb bug: broken "next" and other links”
  1. Wincent ColaiutaFeb 11, 2008
  2. Jakub NarebskiFeb 11, 2008
  3. Wincent ColaiutaFeb 11, 2008
  4. Jakub NarebskiFeb 11, 2008
  5. Jakub NarebskiFeb 12, 2008
  6. Jakub NarebskiFeb 12, 2008
  7. Wincent ColaiutaFeb 12, 2008
  8. Jakub NarebskiFeb 12, 2008
  9. Wincent ColaiutaFeb 12, 2008
  10. Jakub NarebskiFeb 15, 2008
  11. Wincent ColaiutaFeb 17, 2008
  12. Junio C HamanoFeb 17, 2008
  13. Jakub NarebskiFeb 18, 2008
  14. Junio C HamanoFeb 18, 2008
  15. Jakub NarebskiFeb 19, 2008
  16. Wincent ColaiutaFeb 19, 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.