public inbox for git@vger.kernel.org 
 help / color / mirror / Atom feed
From: Eric Wong <normalperson@yhbt•net>
To: Jakub Narebski <jnareb@gmail•com>
Cc: Pavan Kumar Sunkara <pavan.sss1991@gmail•com>, git@vger•kernel.org
Subject: Re: [PATCH GSoC 2/3] git-instaweb: Configure it to work with a global server root and projects list
Date: Tue, 18 May 2010 15:54:45 -0700	[thread overview]
Message-ID: <20100518225445.GA15414@dcvr.yhbt.net> (raw)
In-Reply-To: <m3ljbhcp46.fsf@localhost.localdomain>

Jakub Narebski <jnareb@gmail•com> wrote:
> Second, in my opinion it is not a good change at all.  Currently you
> can run "git instaweb" when inside git repository, and get a web
> browser (or a new tab in existing session of a running web browser)
> with current repository in it, to browse its history.  It is similar
> to running gitk (or other graphical history browser, like qgit, tig,
> etc.), or running "git log", but with web interface.
> 
> Now, current git-instaweb behavior has its quirks, but having
> git-instaweb show _current_ repository is a very important feature,
> and I'd rather we didn't lose it in transition.

I agree completely with Jakub here.  My original inspiration for
instaweb was being able to make local commits, fire up instaweb in my
working directory, and then to get a non-git-using cow-orker on my LAN
to review some changes before I pushed them upstream.

I'm no longer in a situation with non-git-using cow-orkers anymore, but
I suspect many folks still are.

-- 
Eric Wong

  parent reply	other threads:[~2010-05-18 22:54 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-05-15 19:58 [PATCHv2 GSoC 1/3] gitweb: Set default destination directory for installing gitweb in Makefile Pavan Kumar Sunkara
2010-05-15 19:58 ` [PATCH GSoC 2/3] git-instaweb: Configure it to work with a global server root and projects list Pavan Kumar Sunkara
2010-05-16  0:16   ` Dévai Tamás
2010-05-17 20:22     ` Pavan Kumar Sunkara
2010-05-18 13:33   ` Jakub Narebski
2010-05-18 18:56     ` Pavan Kumar Sunkara
2010-05-18 23:05       ` Jakub Narebski
2010-05-19  8:58         ` Pavan Kumar Sunkara
2010-05-19 16:48           ` Jakub Narebski
2010-05-18 22:54     ` Eric Wong [this message]
2010-05-15 19:58 ` [PATCH 3/3] git-web--browse: Add support for google chrome Pavan Kumar Sunkara
2010-05-16 21:01   ` Erik Faye-Lund

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=20100518225445.GA15414@dcvr.yhbt.net \
    --to=normalperson@yhbt$(echo .)net \
    --cc=git@vger$(echo .)kernel.org \
    --cc=jnareb@gmail$(echo .)com \
    --cc=pavan.sss1991@gmail$(echo .)com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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