git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Shawn O. Pearce" <spearce@spearce.org>
To: Junio C Hamano <junkio@cox.net>
Cc: git@vger.kernel.org
Subject: Re: Merge git-gui into 1.5.0 ?
Date: Sun, 11 Feb 2007 17:41:58 -0500	[thread overview]
Message-ID: <20070211224158.GA31488@spearce.org> (raw)
In-Reply-To: <7vwt2oba8s.fsf@assigned-by-dhcp.cox.net>

Junio C Hamano <junkio@cox.net> wrote:
> I do not have objection per-se, but I have two choices on the
> procedure, and I hate having choices this close to the final
> release ;-).
...
> I was actually hoping I can do so with Kay, but from his point
> of view merging gitweb to git.git was so that he does not have
> to worry about it anymore, so it did not work well.

I'm OK with either approach here.

Originally I had intended to wipe out everything except git-gui.sh,
then do an ancestor-less merge into git.git's top level directory,
and finally tweak git.git's master Makefile to add git-gui.sh to
the list of known shell scripts.  Then I was going to ask you to
pull that resulting tree.

But it may make a *lot* more sense to treat is a true subproject in
its own directory.  Unlike Kay, I'm not looking to merge git-gui
into git.git to abandon it.  I just think we should offer a GUI
out of the box, git-gui has the same dependencies as gitk, and I
happen to like git-gui.  ;-)

git-gui development is going to continue past 1.5.0's release.
There are still a lot of operations it should support that it
currently does not do, and there are certainly user interface
improvements that can still be made.

It may be saner for all involved if that development happens in
the git-gui.git repository, with drops made to git.git by way of
merging the "subproject" every so often.

It may make patching slightly more interesting though, as some
users new to git-gui development may generate a patch in git.git
(using a/git-gui/git-gui.sh as the path) which then would not apply
as-is to the master git-gui development tree.

Entirely your call Junio.

-- 
Shawn.

  reply	other threads:[~2007-02-11 22:42 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-02-11  8:40 Merge git-gui into 1.5.0 ? Shawn O. Pearce
2007-02-11 14:57 ` Mark Levedahl
2007-02-11 22:49   ` Shawn O. Pearce
2007-02-11 16:39 ` Johannes Schindelin
2007-02-11 22:27 ` Junio C Hamano
2007-02-11 22:41   ` Shawn O. Pearce [this message]
2007-02-11 22:53     ` Johannes Schindelin
2007-02-11 23:02       ` Shawn O. Pearce
2007-02-11 23:25         ` Johannes Schindelin
2007-02-12  5:40         ` Michael S. Tsirkin
2007-02-12  5:48           ` Junio C Hamano
2007-02-12  5:59             ` Michael S. Tsirkin
2007-02-12  6:25               ` Michael S. Tsirkin
2007-02-12 11:51                 ` add negative -p to git-am, " Johannes Schindelin
2007-02-12 11:59                   ` Michael S. Tsirkin
2007-02-12 12:06                     ` Johannes Schindelin
2007-02-12 12:26                       ` Michael S. Tsirkin
2007-02-12 21:55     ` Junio C Hamano
2007-02-12 23:34       ` Shawn O. Pearce
2007-02-12 23:38       ` Johannes Schindelin
2007-02-12 23:42         ` Shawn O. Pearce
2007-02-12 23:53           ` Johannes Schindelin
2007-02-13  0:34             ` Jakub Narebski
2007-02-13  0:38               ` Shawn O. Pearce

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

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

  git send-email \
    --in-reply-to=20070211224158.GA31488@spearce.org \
    --to=spearce@spearce.org \
    --cc=git@vger.kernel.org \
    --cc=junkio@cox.net \
    /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.
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 NNTP newsgroup(s).