From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kevin Ballard Subject: Re: [PATCH v2 0/3] Teach fetch and pull to recursively fetch submodules too Date: Wed, 13 Oct 2010 12:34:56 -0700 Message-ID: <1691DBFD-6075-479E-8847-EB4595BB9E10@sb.org> References: <4C7A819B.3000403@web.de> <7vocckhcb6.fsf@alter.siamese.dyndns.org> <778BC76C-FDFA-4EF0-AA94-6631272DEC02@sb.org> <89574F83-293C-4E3E-A99D-EB6CE6D47646@sb.org> <4C9221B6.7070807@web.de> <4C9351A7.7050609@web.de> <4C9359D4.2030109@viscovery.net> <4C935D77.3080008@web.de> <329A2E43-ADE3-467C-A2A6-24ACB9DF641E@sb.org> <4C953DE5.6020900@web.de> <4C963D00.9050207@web.de> <28BC3A45-D3CF-4A8C-A818-B92A9827C8FB@sb.org> <4CAB8DDF.8080004@web.de> <4CB0BFDB.7010503@web.de> <4CB5C6D8.1070108@xiplink.com> <4CB6093F.3040800@web.de> Mime-Version: 1.0 (Apple Message framework v1081) Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 8BIT Cc: Marc Branchaud , Junio C Hamano , Git Mailing List To: Jens Lehmann X-From: git-owner@vger.kernel.org Wed Oct 13 21:35:31 2010 Return-path: Envelope-to: gcvg-git-2@lo.gmane.org Received: from vger.kernel.org ([209.132.180.67]) by lo.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1P676n-0007po-OD for gcvg-git-2@lo.gmane.org; Wed, 13 Oct 2010 21:35:30 +0200 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752783Ab0JMTfA (ORCPT ); Wed, 13 Oct 2010 15:35:00 -0400 Received: from mail-pv0-f174.google.com ([74.125.83.174]:43772 "EHLO mail-pv0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752277Ab0JMTe6 convert rfc822-to-8bit (ORCPT ); Wed, 13 Oct 2010 15:34:58 -0400 Received: by pvc7 with SMTP id 7so443932pvc.19 for ; Wed, 13 Oct 2010 12:34:58 -0700 (PDT) Received: by 10.142.164.18 with SMTP id m18mr1031843wfe.375.1286998497931; Wed, 13 Oct 2010 12:34:57 -0700 (PDT) Received: from [10.8.0.89] ([69.170.160.74]) by mx.google.com with ESMTPS id q13sm11986731wfc.17.2010.10.13.12.34.56 (version=TLSv1/SSLv3 cipher=RC4-MD5); Wed, 13 Oct 2010 12:34:56 -0700 (PDT) In-Reply-To: <4CB6093F.3040800@web.de> X-Mailer: Apple Mail (2.1081) Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Archived-At: On Oct 13, 2010, at 12:32 PM, Jens Lehmann wrote: >>> There are use cases like mine where automatic recursion is just the right >>> thing to do. But I would be fine with having to turn the recursion on >>> explicitly in the configuration if most people think recursion is not a >>> desirable default. It would be really nice to hear from other submodule >>> users what they think about that ... >> >> I tend to think that the right default for fetch is to employ the same level >> of recursion that was used for the initial clone. So if the clone was made >> with --recursive then fetch should default to using --recursive. > > That's a very interesting idea. I'm not sure it's correct though. For example, with my scenario every single submodule is required for a correct build, but most submodules should definitely not be updated unless their parent submodule updates its gitlink. So --recursive is recommended for `git clone`, but a non-recursive fetch would be the correct behavior going forward. -Kevin Ballard