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

Re: [PATCH 1/2] gitweb: allow access to forks with strict_export

From
Matt McCutchen <matt@mattmccutchen.net>
Date
Dec 14, 2008, 02:06 UTC
Message-ID
<1229220398.3360.66.camel@mattlaptop2.local>
In-Reply-To
<200812132351.37420.jnareb@gmail.com>
On Sat, 2008-12-13 at 23:51 +0100, Jakub Narebski wrote:
> "no_hide" (currently "include_forks") allows us to _not_ passing this
> parameter in other places than project_in_list(); undef is falsy.

Right. That's why I made the current parameter $for_strict_export (so only project_in_list passes it) rather than the negation.

> By the way, doesn't git_project_index and perhaps git_opml also need
> this parameter passed to git_get_projects_list?

Yes, now that you mention it, I suppose they should show forks, though not hidden repositories. Then git_get_projects_list can be called in three different modes: include everything (project_in_list), include forks but not hidden (git_get_project_index and git_opml), or include neither forks nor hidden (git_project_list). Should we have two separate parameters to git_get_projects_list or a single three-valued one?

That raises another point. I was going to change git_get_projects_list so that forks of a hidden project that are not themselves hidden appear on the parent project's page but not in the main project list. This way, users who know about the parent project can navigate to the fork, but the fork does not give away the existence of the parent project by appearing in the main list. Then I guess git_project_index and git_opml should omit forks of hidden projects, meaning that some fork-checking still has to take place with "include forks" on but "include hidden" off. This will make git_get_projects_list somewhat more complex but not unmanageably so, and I do think it's the behavior we want.

I will send an updated patch.
-- 
Matt
Previous: Jakub NarebskiNext: Jakub Narebski
Message 5 of 7 in “gitweb: allow access to forks with strict_export”
  1. 1/2 gitweb: allow access to forks with strict_exportMatt McCutchen, Dec 13, 2008
  2. Jakub NarebskiDec 13, 2008
  3. Junio C HamanoDec 13, 2008
  4. Jakub NarebskiDec 13, 2008
  5. Matt McCutchenDec 14, 2008
  6. Jakub NarebskiDec 20, 2008
  7. Matt McCutchenDec 14, 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.