git@vger.kernel.org mailing list mirror (one of many)
 help / color / mirror / code / Atom feed
From: Heiko Voigt <hvoigt@hvoigt.net>
To: Junio C Hamano <gitster@pobox.com>
Cc: Pat Thoyts <patthoyts@googlemail.com>, git@vger.kernel.org
Subject: future of git-gui as subsytem or submodule, WAS: [PATCH] git-gui: document the gui.maxfilesdisplayed variable
Date: Mon, 14 Feb 2011 23:03:18 +0100	[thread overview]
Message-ID: <20110214220318.GD50815@book.hvoigt.net> (raw)
In-Reply-To: <7v39nruk9j.fsf@alter.siamese.dyndns.org>

Hi,

On Sun, Feb 13, 2011 at 11:51:04PM -0800, Junio C Hamano wrote:
> Two opposing thoughts.
> 
>  1. We can keep git-gui and git proper separate projects, move git-gui
>     documentation out of git to git-gui, and with clever Makefile trick
>     include and build git-gui related documentation conditionally only
>     when git-gui appears part of the git project tree (this anticipates a
>     future where git-gui is bound to git not with the subtree merge
>     strategy as we currently do, but as a submodule).
> 
>  2. Just like the Linux kernel project, we can make each subsystem with
>     separate maintainers just different repositories of the same git
>     project with their own focus.  We already do this for git-svn (which I
>     delegate to Eric Wong by pulling from him) and some parts of contrib/
>     tree; we have already been halfway there for gitweb/ (which I don't
>     regularly "pull from", but I mainly act as a patch monkey without
>     actively managing that part myself).  I don't see why we cannot extend
>     that model to git-gui and gitk.

I would vote for 2. Not because I think submodules will not become as easy to
use so they are ready for that but I think there are mainly two reasons
for using a submodule

  A. The submodule contains shared code which is used by multiple projects

  B. A submodule is used to keep large collection of files seperate from
     a project because most times they are not needed and would
     interfere with the project.

There are maybe more but these two do not apply to git-gui and I like
the way it is currently integrated in one repository with git. It also
underlines the fact that git-gui is AFAICS the standard and best
developed gui for git.

Another plus, if we extend that model to gitk, is that both could start
sharing code between each other (maybe relocate to the same directory).

Although 1. would be a good choice for getting more people involved in
enhancing submodule support, from a philosophical standpoint, I think 2.
is the more natural choice.

What do others think?

Cheers Heiko

  reply	other threads:[~2011-02-14 22:03 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-02-13 12:53 [PATCH] git-gui: document the gui.maxfilesdisplayed variable Heiko Voigt
2011-02-14  7:51 ` Junio C Hamano
2011-02-14 22:03   ` Heiko Voigt [this message]
2011-02-14 22:17     ` future of git-gui as subsytem or submodule Jonathan Nieder
2011-02-14 22:46     ` future of git-gui as subsytem or submodule, WAS: [PATCH] git-gui: document the gui.maxfilesdisplayed variable Jens Lehmann

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: http://vger.kernel.org/majordomo-info.html

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20110214220318.GD50815@book.hvoigt.net \
    --to=hvoigt@hvoigt.net \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.com \
    --cc=patthoyts@googlemail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://80x24.org/mirrors/git.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).