From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Shawn O. Pearce" Subject: Re: [PATCH 2/4] fast-import: define a new option command Date: Thu, 13 Aug 2009 10:25:08 -0700 Message-ID: <20090813172508.GO1033@spearce.org> References: <1250140186-12363-2-git-send-email-srabbelier@gmail.com> <1250140186-12363-3-git-send-email-srabbelier@gmail.com> <20090813144327.GK1033@spearce.org> <20090813150446.GM1033@spearce.org> <20090813152419.GN1033@spearce.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Johannes Schindelin , Git List , Junio C Hamano To: Sverre Rabbelier X-From: git-owner@vger.kernel.org Thu Aug 13 19:25:18 2009 Return-path: Envelope-to: gcvg-git-2@gmane.org Received: from vger.kernel.org ([209.132.176.167]) by lo.gmane.org with esmtp (Exim 4.50) id 1Mbe3B-0007lk-B8 for gcvg-git-2@gmane.org; Thu, 13 Aug 2009 19:25:17 +0200 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755163AbZHMRZI (ORCPT ); Thu, 13 Aug 2009 13:25:08 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1755161AbZHMRZH (ORCPT ); Thu, 13 Aug 2009 13:25:07 -0400 Received: from george.spearce.org ([209.20.77.23]:50495 "EHLO george.spearce.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752533AbZHMRZH (ORCPT ); Thu, 13 Aug 2009 13:25:07 -0400 Received: by george.spearce.org (Postfix, from userid 1001) id 1F6A3381FD; Thu, 13 Aug 2009 17:25:08 +0000 (UTC) Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.17+20080114 (2008-01-14) Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Archived-At: Sverre Rabbelier wrote: > On Thu, Aug 13, 2009 at 10:07, Johannes > Schindelin wrote: > > ... and will import the marks twice? > > Ah, you're right :(. What's the best way to do this? Should we dump > any previous marks when importing new ones? Uh, well, yes. We shouldn't define :5 if it was in the file that appeared in the stream, but isn't in the file on the command line. Worse, what happens if we do this: echo "option import-marks=/not/found" \ | git fast-import --import-marks=my.marks I want this to work, even though /not/found does not exist, but my.marks does. So that does complicate things... -- Shawn.