All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paolo Bonzini <bonzini@gnu.org>
To: Johannes Schindelin <Johannes.Schindelin@gmx.de>
Cc: Andreas Ericsson <ae@op5.se>, Alex Riesen <raa.lkml@gmail.com>,
	git@vger.kernel.org, spearce@spearce.org, gitster@pobox.com,
	peff@peff.net, srb@cuci.nl
Subject: Re: [PATCH 7/7] make "git fetch" update all fetch repositories
Date: Tue, 29 Apr 2008 18:40:24 +0200	[thread overview]
Message-ID: <48174F78.7020400@gnu.org> (raw)
In-Reply-To: <alpine.DEB.1.00.0804291711450.27457@eeepc-johanness>


>>> Some of them point to my colleagues laptops, where temporary
>>> git-daemons sometimes serve content, and sometimes it doesn't.
>> Again, you should probably have skipDefaultUpdate set on those remotes even
>> now!
> 
> Umm, why?

Because I don't want to make sure my colleagues *all* have their laptops 
turned on when I run "git remote update" (on the default group).  Those 
remotes should be fetched individually as the need arises, or maybe with 
a special group, but not by default.

>> A patch that makes a "wrong" (or incomplete) configuration more
>> apparent, is not necessarily wrong in itself.
> 
> Your suggested changes are actively _making_ them wrong.  They are not 
> wrong now.

Sure, you can consider a configuration that affects "git remote update" 
to be correct if you're not using "git remote update" at all.  But a 
wrong you never saw/experienced is not a right.  My changes are actively 
making the wrongness apply more broadly, that's clear.

> Happily, I am quite certain that Junio will not allow such dramatic 
> changes into Git, at least not without a long, long time of warning, 
> during which I can safely undo the changes in my personal branch.

I wouldn't be surprised if it didn't go in 1.5.6 -- indeed.

Actually I think the "git pull" changes (patches 5 and 6) and the 
matching refs refspec (patches 1 and 2) can go in sooner, but I sort of 
expect the other 3 patches to lie in limbo for a longer time.

> I absolutely hate the idea of "origin" _not_ being special.

Well, Junio and Shawn convinced me of the contrary, so I guess they 
disagree with you on this one...

Paolo

  reply	other threads:[~2008-04-29 16:41 UTC|newest]

Thread overview: 74+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-04-26  7:31 [PATCH resend] make "git push" update origin and mirrors, "git push --mirror" update mirrors Paolo Bonzini
2008-04-26 17:01 ` Shawn O. Pearce
2008-04-26 17:46   ` Junio C Hamano
2008-04-27  4:30     ` Shawn O. Pearce
2008-04-27  4:40       ` Shawn O. Pearce
2008-04-27  5:23         ` Junio C Hamano
2008-04-27 17:34           ` Shawn O. Pearce
2008-04-27 20:13             ` Junio C Hamano
2008-04-27 20:22               ` Paolo Bonzini
2008-04-28  1:26                 ` Jeff King
2008-04-28  5:07                   ` Paolo Bonzini
2008-04-28  9:09                     ` Jeff King
2008-04-28  9:11                       ` Jeff King
2008-04-28  9:19                         ` Paolo Bonzini
2008-04-28 10:33                           ` Johannes Schindelin
2008-04-28 11:24                             ` Paolo Bonzini
2008-04-28 11:57                               ` Johannes Schindelin
2008-04-28  3:32               ` Shawn O. Pearce
2008-04-28  5:03                 ` Paolo Bonzini
2008-04-28  6:08                 ` Stephen R. van den Berg
2008-04-28  1:21             ` Jeff King
2008-04-27  9:03         ` Paolo Bonzini
2008-04-27  7:23   ` Paolo Bonzini
2008-04-28 15:32 ` [PATCH 0/7] limit the usage of the default remote "origin" to the minimum Paolo Bonzini
2008-04-28 15:32   ` [PATCH 1/7] add special "matching refs" refspec Paolo Bonzini
2008-04-30  9:23     ` Junio C Hamano
2008-04-30  9:35       ` Paolo Bonzini
2008-04-28 15:32   ` [PATCH 2/7] add push line in git-clone Paolo Bonzini
2008-05-01  6:28     ` Junio C Hamano
2008-05-06  8:37       ` Paolo Bonzini
2008-05-14 15:20         ` Paolo Bonzini
2008-05-14 18:16           ` Junio C Hamano
2008-05-14 19:07             ` Paolo Bonzini
2008-05-14 19:23               ` Junio C Hamano
2008-05-14 19:40                 ` Paolo Bonzini
2008-04-28 15:32   ` [PATCH 3/7] Add a --push option to "git-remote add" Paolo Bonzini
2008-04-28 15:32   ` [PATCH 4/7] make "git push" update all push repositories Paolo Bonzini
2008-04-28 15:32   ` [PATCH 5/7] don't rely on zero-argument "git fetch" from within git pull Paolo Bonzini
2008-04-28 15:32   ` [PATCH 6/7] warn on "git pull" without a given branch.<name>.remote value Paolo Bonzini
2008-04-28 15:32   ` [PATCH 7/7] make "git fetch" update all fetch repositories Paolo Bonzini
2008-04-28 18:10     ` Alex Riesen
2008-04-28 18:19       ` Paolo Bonzini
2008-04-28 21:33         ` Alex Riesen
2008-04-29  4:52           ` Paolo Bonzini
2008-04-29  5:38             ` Alex Riesen
2008-04-29  6:05               ` Andreas Ericsson
2008-04-29  6:55                 ` Paolo Bonzini
2008-04-29 16:13                   ` Johannes Schindelin
2008-04-29 16:40                     ` Paolo Bonzini [this message]
2008-04-29 20:34                       ` Alex Riesen
2008-04-29  6:50               ` Paolo Bonzini
2008-04-29  7:16                 ` Andreas Ericsson
2008-04-29  7:57                   ` Paolo Bonzini
2008-04-29  8:48                     ` Andreas Ericsson
2008-04-29  9:02                       ` Paolo Bonzini
2008-04-29 21:08                         ` しらいしななこ
     [not found]                       ` <200804292108.m3TL8moV011790@mi1.bluebottle.com>
2008-04-29 21:21                         ` Paolo Bonzini
2008-04-29 22:21                           ` Johannes Schindelin
2008-04-29 20:44                     ` Alex Riesen
2008-04-29 21:15                       ` Paolo Bonzini
2008-04-29 21:33                         ` Alex Riesen
2008-04-29 21:41                           ` Paolo Bonzini
2008-04-29 21:53                             ` Alex Riesen
2008-04-29 22:26                             ` Johannes Schindelin
2008-04-29 23:02                               ` Jeff King
2008-04-29 23:17                                 ` Junio C Hamano
2008-04-30  5:28                                   ` Paolo Bonzini
2008-04-29 21:39                       ` Johannes Schindelin
2008-04-29 20:24                 ` Alex Riesen
2008-04-29 19:35   ` [PATCH 0/7] limit the usage of the default remote "origin" to the minimum Jeff King
2008-04-29 21:42     ` Alex Riesen
2008-04-29 21:56     ` Junio C Hamano
2008-04-29 23:12       ` Jeff King
2008-04-30  5:24         ` Paolo Bonzini

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=48174F78.7020400@gnu.org \
    --to=bonzini@gnu.org \
    --cc=Johannes.Schindelin@gmx.de \
    --cc=ae@op5.se \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.com \
    --cc=peff@peff.net \
    --cc=raa.lkml@gmail.com \
    --cc=spearce@spearce.org \
    --cc=srb@cuci.nl \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.