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

[git-gui] bug report: "Open existing repository" dialog fails on submodules

From
Rémi Rampin <remirampin@gmail.com>
Date
Jan 30, 2015, 21:46 UTC
Message-ID
<CAMto89Dvz-u+at4CPLGQRak4niOJk1trSCn2wQugLUnD1h=Fjw@mail.gmail.com>
Hi,

This bug report concerns git-gui. Apologies if this is not the right mailing-list.

By submodule I mean a repository for which .git is not a regular Git directory, but rather a "gitdir: ..." file. While running "git gui" from such a directory will work fine, trying to open it from the choose_repository window will fail with "Not a Git repository". This is because of the simplistic implementation of proc _is_git in lib/choose_repository.tcl.

I suggest fixing that function, or using Git directly to perform that check, for instance checking "git rev-parse --show-toplevel". I'd attempt a patch but my tcl-fu is weak.

Best
-- 
Rémi Rampin
Next: Chris Packham
Message 1 of 14 in “[git-gui] bug report: "Open existing repository" dialog fails on submodules”
  1. Rémi RampinJan 30, 2015
  2. Chris PackhamFeb 2, 2015
  3. Chris PackhamFeb 2, 2015
  4. Rémi RampinFeb 2, 2015
  5. 1/2 Fixes _is_gitRemi Rampin, Feb 2, 2015
  6. 2/2 Makes _do_open2 set _gitdir to actual pathRemi Rampin, Feb 2, 2015
  7. Chris PackhamFeb 3, 2015
  8. Rémi RampinFeb 3, 2015
  9. Chris PackhamFeb 3, 2015
  10. Rémi RampinFeb 3, 2015
  11. Chris PackhamFeb 5, 2015
  12. 0/2 gitfile support git git-guiRemi Rampin, Feb 5, 2015
  13. 1/2 Fixes chooser not accepting gitfilesRemi Rampin, Feb 5, 2015
  14. 2/2 Makes chooser set 'gitdir' to the resolved pathRemi Rampin, Feb 5, 2015

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.