From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jens Lehmann Subject: Re: submodules' shortcomings, was Re: RFC: display dirty submodule working directory in git gui and gitk Date: Wed, 06 Jan 2010 19:20:13 +0100 Message-ID: <4B44D45D.3070509@web.de> References: <4B3F6742.6060402@web.de> <4B421F90.4090402@web.de> <4B42F425.4010901@web.de> <7v1vi428w0.fsf@alter.siamese.dyndns.org> <4B43292C.5060106@web.de> <7vd41oz9mp.fsf@alter.siamese.dyndns.org> <4B439A86.3020500@web.de> <7vbph8oxg0.fsf@alter.siamese.dyndns.org> <4B4498BC.5040400@web.de> <7vbph7181x.fsf@alter.siamese.dyndns.org> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Cc: Johannes Schindelin , Git Mailing List , "Shawn O. Pearce" , Paul Mackerras , Heiko Voigt , Lars Hjemli , Avery Pennarun To: Junio C Hamano X-From: git-owner@vger.kernel.org Wed Jan 06 19:21:41 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.50) id 1NSaVo-0003SC-RA for gcvg-git-2@lo.gmane.org; Wed, 06 Jan 2010 19:21:41 +0100 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932528Ab0AFSVg (ORCPT ); Wed, 6 Jan 2010 13:21:36 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S932526Ab0AFSVg (ORCPT ); Wed, 6 Jan 2010 13:21:36 -0500 Received: from fmmailgate01.web.de ([217.72.192.221]:57308 "EHLO fmmailgate01.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932498Ab0AFSVg (ORCPT ); Wed, 6 Jan 2010 13:21:36 -0500 Received: from smtp06.web.de (fmsmtp06.dlan.cinetic.de [172.20.5.172]) by fmmailgate01.web.de (Postfix) with ESMTP id 6DA9F14473912; Wed, 6 Jan 2010 19:21:34 +0100 (CET) Received: from [80.128.107.143] (helo=[192.168.178.26]) by smtp06.web.de with asmtp (WEB.DE 4.110 #314) id 1NSaUQ-0007c1-00; Wed, 06 Jan 2010 19:20:14 +0100 User-Agent: Mozilla/5.0 (X11; U; Linux i686; de; rv:1.9.1.5) Gecko/20091204 Thunderbird/3.0 In-Reply-To: <7vbph7181x.fsf@alter.siamese.dyndns.org> X-Provags-ID: Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Archived-At: Am 06.01.2010 18:01, schrieb Junio C Hamano: > Jens Lehmann writes: > >> Am 06.01.2010 02:04, schrieb Junio C Hamano: >>> After all we _were_ in agreement. We both think the user should be told >>> about untracked files in submodule directory when inspecting the status to >>> make a commit in the superproject. >> >> Thanks. So i'll take a closer look at the diff core (but i suspect i'll >> need some time until i can come up with some patches because i don't know >> this part of git very well). > > I don't see a direct connection between "the user should be told about > untracked in the submodule before committing" and diffcore. It is just > the matter of "git status" and "git commit" running another instance of > "git status" via run_command() interface in the submodule directory, no? Basically yes. But i also would like to teach "git diff" (when diffing against the working directory of the superproject) to show these submodule states too so that git gui and gitk will display them.