From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 77777C433FE for ; Fri, 7 Oct 2022 13:59:15 +0000 (UTC) Received: from localhost ([::1]:60524 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1ognsg-00081l-8a for qemu-devel@archiver.kernel.org; Fri, 07 Oct 2022 09:59:14 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:51682) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1oglEn-0003xW-PV for qemu-devel@nongnu.org; Fri, 07 Oct 2022 07:09:53 -0400 Received: from us-smtp-delivery-124.mimecast.com ([170.10.129.124]:58858) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1oglEj-0006x5-JP for qemu-devel@nongnu.org; Fri, 07 Oct 2022 07:09:51 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1665140988; h=from:from:reply-to:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=U5/KKQKZhGuh5pVdWl0nmsSVyF+ITn22I2iainSzJU8=; b=Eus2HWDBfCKQ0do5exCq+nC+EH/98VexPeJkHbepgoKIk2J9X6yDBqCpIcY0Ge7mmEXy/F 2kRXBQqlJZk8ARJl50HbI/EK+ie5BBv5Bp8wwT9PLFo62cwsEyjsIdC+CPIqF0053f43of pn4FWpA3Ffl3LY56GKEJIULVvm2dDD8= Received: from mimecast-mx02.redhat.com (mx3-rdu2.redhat.com [66.187.233.73]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-77-xP_5vVpzNciekrokkH64-g-1; Fri, 07 Oct 2022 07:09:47 -0400 X-MC-Unique: xP_5vVpzNciekrokkH64-g-1 Received: from smtp.corp.redhat.com (int-mx07.intmail.prod.int.rdu2.redhat.com [10.11.54.7]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id 99A401C0515C; Fri, 7 Oct 2022 11:09:45 +0000 (UTC) Received: from redhat.com (unknown [10.33.36.42]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 8E0721402C01; Fri, 7 Oct 2022 11:09:44 +0000 (UTC) Date: Fri, 7 Oct 2022 12:09:40 +0100 From: Daniel =?utf-8?B?UC4gQmVycmFuZ8Op?= To: "Michael S. Tsirkin" , qemu-devel@nongnu.org, Thomas Huth , Philippe =?utf-8?Q?Mathieu-Daud=C3=A9?= , Stefan Hajnoczi , Alex =?utf-8?Q?Benn=C3=A9e?= Subject: Re: [PATCH] gitmodules: recurse by default Message-ID: References: <20221006113906.179963-1-mst@redhat.com> <20221006202250-mutt-send-email-mst@kernel.org> <20221007061037-mutt-send-email-mst@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/2.2.7 (2022-08-07) X-Scanned-By: MIMEDefang 3.1 on 10.11.54.7 Received-SPF: pass client-ip=170.10.129.124; envelope-from=berrange@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -20 X-Spam_score: -2.1 X-Spam_bar: -- X-Spam_report: (-2.1 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: Daniel =?utf-8?B?UC4gQmVycmFuZ8Op?= Errors-To: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Sender: "Qemu-devel" On Fri, Oct 07, 2022 at 11:45:56AM +0100, Daniel P. Berrangé wrote: > On Fri, Oct 07, 2022 at 06:11:25AM -0400, Michael S. Tsirkin wrote: > > On Fri, Oct 07, 2022 at 09:07:17AM +0100, Daniel P. Berrangé wrote: > > > On Thu, Oct 06, 2022 at 08:24:01PM -0400, Michael S. Tsirkin wrote: > > > > On Thu, Oct 06, 2022 at 07:54:52PM +0100, Daniel P. Berrangé wrote: > > > > > On Thu, Oct 06, 2022 at 07:39:07AM -0400, Michael S. Tsirkin wrote: > > > > > > The most commmon complaint about submodules is that > > > > > > they don't follow when one switches branches in the > > > > > > main repo. Enable recursing into submodules by default > > > > > > to address that. > > > > > > > > > > > > Signed-off-by: Michael S. Tsirkin > > > > > > --- > > > > > > .gitmodules | 23 +++++++++++++++++++++++ > > > > > > 1 file changed, 23 insertions(+) snip > > I just retested and it's not working for me either :( > > I was sure it worked but I guess the testing wasn't done properly. > > Back to the drawing board sorry. > > I think the problem is that this setting doesn't apply in the context > of .gitmodules. Various commands take a '--recurse-submodules' parameter, > and like many params this can be set in the .git/config file. The > problem is .git/config isn't a file we can influence automatically, > it is upto the dev to set things for every clone they do :-( With the correct setting in my .git/config, I've just discovered an unexpected & undesirable consequence of using recurse=true. It affects the 'push' command. If your submodule contains a hash that is not present in the upstream of the submodule, then when you try to push, it will also try to push the submodule change. eg, I have a qemu.git branch 'work' and i made a change to ui/keycodemapdb. If I try to push to my gitlab fork, whose remote I called 'gitlab', then it will also try to push ui/keycodemapdb to a fork called 'gitlab'. Except I don't have any such fork existing, so my attempt to push my qemu.git changes fails because of the submodule. This is going to be annoying to people who are working on branches with updates to the git submodules if we were to set recurse=true by default, as they'll have to also setup remotes for submodules they work on. With regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|