From: Jakub Narebski <jnareb@gmail.com>
To: Petr Baudis <pasky@suse.cz>
Cc: Junio C Hamano <gitster@pobox.com>,
git@vger.kernel.org, "J.H." <warthog9@eaglescrag.net>,
John 'Warthog9' Hawley <warthog9@kernel.org>
Subject: Re: What's cooking in git.git (Jan 2010, #08; Sun, 24)
Date: Tue, 26 Jan 2010 01:07:24 +0100 [thread overview]
Message-ID: <201001260107.25796.jnareb@gmail.com> (raw)
In-Reply-To: <20100125231241.GA4159@machine.or.cz>
On Tue, Jan 26, 2010 at 00:12 +0100, Petr Baudis wrote:
> On Mon, Jan 25, 2010 at 02:43:01PM -0800, Jakub Narebski wrote:
> > You are right that actual caching support is in flux. The discussion
> > continues: it is very good that we have the voice from Pasky, too.
>
> Unfortunately, I can't give much time to it, or actual contributions.
> I think it would be great to apply at least the miscellanous-changes
> part to (i) move this further (ii) give jh some positive feedback. :)
> There are ugly minor things, but they can be patched up later. I think
> it's healthier for new gitweb stuff to develop more in-tree, even
> if that means few angry users annoyed about less-than-perfect UI stuff,
> rather than polish the diamond in infinite discussions; worse is better!
> The only thing we should be worried about is avoiding introducing new
> bad URL interfaces since we should keep backwards compatibility there.
I think that while some patches from the miscellaneous-changes part are
good now (and very nice to have, especially load checking and
gitweb/Makefile), there needs to be one last, final reroll of those
patches.
I am waiting for response from (I guess busy) J.H.; I can do the re-roll
if he is too busy to work on it.
> For the main caching patch, it seems like good idea to take Jakub's
> split-up series instead, let's see what is J.H.'s opinion on the series?
Let me at least make them into proper patches, with commit messages and
configureability at least of the original caching patch by J.H.
Also the question whether to create 'print -> print $out' patch, or to
manipulate *STDOUT instead must be solved, I think, before applying
those patches... well, at least beyond 'pu'.
I am waiting for promised J.H. comments, when he will have time for it...
--
Jakub Narebski
Poland
next prev parent reply other threads:[~2010-01-26 0:07 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-01-25 4:39 What's cooking in git.git (Jan 2010, #08; Sun, 24) Junio C Hamano
2010-01-25 18:29 ` Issues that need to be resolved before 1.7.0-rc1 Junio C Hamano
2010-01-25 21:17 ` Johannes Sixt
2010-01-25 22:43 ` What's cooking in git.git (Jan 2010, #08; Sun, 24) Jakub Narebski
2010-01-25 23:12 ` Petr Baudis
2010-01-25 23:47 ` Junio C Hamano
2010-01-26 0:07 ` Jakub Narebski [this message]
2010-01-26 0:21 ` J.H.
2010-01-26 0:59 ` Jakub Narebski
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=201001260107.25796.jnareb@gmail.com \
--to=jnareb@gmail.com \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=pasky@suse.cz \
--cc=warthog9@eaglescrag.net \
--cc=warthog9@kernel.org \
/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).