From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko Voigt Subject: Re: Re: Re: Re: Updating a submodule with a compatible version from another submodule version using the parent meta-repository Date: Sun, 13 Feb 2011 14:30:19 +0100 Message-ID: <20110213133019.GB31986@book.hvoigt.net> References: <4D407875.7080607@web.de> <7v7hdrl7nw.fsf@alter.siamese.dyndns.org> <7v1v3zjp6w.fsf@alter.siamese.dyndns.org> <20110129110807.GA21864@book.hvoigt.net> <20110209193610.GA67890@book.hvoigt.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Junio C Hamano , Jens Lehmann , git@vger.kernel.org To: Julian Ibarz X-From: git-owner@vger.kernel.org Sun Feb 13 14:31:52 2011 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 1Poc3L-000484-J2 for gcvg-git-2@lo.gmane.org; Sun, 13 Feb 2011 14:31:51 +0100 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754461Ab1BMNaX (ORCPT ); Sun, 13 Feb 2011 08:30:23 -0500 Received: from darksea.de ([83.133.111.250]:39919 "HELO darksea.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1754209Ab1BMNaV (ORCPT ); Sun, 13 Feb 2011 08:30:21 -0500 Received: (qmail 13917 invoked from network); 13 Feb 2011 14:30:19 +0100 Received: from unknown (HELO localhost) (127.0.0.1) by localhost with SMTP; 13 Feb 2011 14:30:19 +0100 Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.19 (2009-01-05) Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Archived-At: Hi Julian, On Sat, Feb 12, 2011 at 03:32:43PM -0500, Julian Ibarz wrote: > Thanks for all your feedbacks. Before your first mail I had already > implemented command 1 you are talking about and almost finished > command 2. Today I could spend some time again on this side project > and I have finished command 2. Here is what you do: > > git submodulecheckout --find-versions > > cd > git submodulecheckout --find-closest < I do not quite understand how this is used. I just tried it on the msysgit repository. Which is a good candidate for testing since you can easily verify the results because for each git version there is a corresponding tag in the superproject. You can find it on repo: http://repo.or.cz/w/msysgit.git As far as I understand the sha1 you are searching for is determined by the currently checked out revision. So here we go: cd msysgit/git git checkout v1.7.2.3.msysgit.0~3 cd .. git submodulecheckout --find-versions git > git-versions.txt cd git git submodulecheckout --find-closest < ../git-versions.txt > submodulecheckout: no commit found Ok lets not be so hard: git checkout v1.7.2.3.msysgit.0 cd .. git submodulecheckout --find-versions git > git-versions.txt cd git git submodulecheckout --find-closest < ../git-versions.txt > 820fd48475eb72cdd4f2 Close but not quite the right commit. See the output of git log: 820fd48 Chomp CRLF before feeding to tar when building netinstaller. 83e164c Avoid quoting environment variable values. daafc61 (tag: Git-1.7.2.3-preview20100911) Updated git and git documents and ReleaseNotes to v1.7.2.3 Whats strange is that this commit did not even change the submodule. I think currently it does not make sense to implement the scripting since the interface is not yet cleaned up enough. It should be possible to specify a commit as parameter and the path should probably go after that like the other commands do as well. The other issue is that we need to change the working directory. IMO, everything should be possible to do from superproject. Have a look at add_submodule_odb() in submodule.c it lets you load the database of a submodule. This way you can run the --find-closest from the superproject. So what I would be looking for is something like this: # This looks up all commits of ever registered in this # project git submodulecheckout --find-versions > out.tmp # This reads the previous list and outputs a list of of all the commits # that contain . Function in_merge_bases(treeish, i&commit, 1) # can be of help here. Afterwards you have to prune all later commits # that contain earlier ones. I did this in find_first_merges(). I # suggest to refactor that code into a function so that you can use it. git submodulecheckout --find-closest -- < out.tmp > out2.tmp # And now we can search for the commit in the other submodule C git submodulecheckout --list-commits < out2.tmp > out3.tmp And if out3.tmp just contained one line we could checkout that one. I am still not sure in which command these options could go. Junio do you maybe have a suggestion? Since its all about submodules it would make sense to put it in 'git submodule' but thats already the script. Maybe a new 'git-submodule-tool' ? I suggest that you start implementing some basic tests if you found the right commits using the testsuite. Basically you build up a sample repository where you know which commits of the submodules are tied together with a sequence of commands. Then you check whether your commands find the correct one. Maybe have a look at the submodule tests Jens is implementing on his branch: https://github.com/jlehmann/git-submod-enhancements Hope that helps. Cheers Heiko