From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jeff King Subject: Re: [PATCH v5 01/15] fast-export: avoid importing blob marks Date: Sun, 11 Nov 2012 11:38:28 -0500 Message-ID: <20121111163827.GA11408@sigill.intra.peff.net> References: <1352642392-28387-1-git-send-email-felipe.contreras@gmail.com> <1352642392-28387-2-git-send-email-felipe.contreras@gmail.com> <509FD425.5030702@web.de> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: Felipe Contreras , git@vger.kernel.org, Junio C Hamano , Johannes Schindelin , Max Horn , Sverre Rabbelier , Brandon Casey , Brandon Casey , Jonathan Nieder , Ilari Liusvaara , Pete Wyckoff , Ben Walton , Matthieu Moy , Julian Phillips To: Torsten =?utf-8?Q?B=C3=B6gershausen?= X-From: git-owner@vger.kernel.org Sun Nov 11 17:38:49 2012 Return-path: Envelope-to: gcvg-git-2@plane.gmane.org Received: from vger.kernel.org ([209.132.180.67]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1TXaYa-0004z8-3n for gcvg-git-2@plane.gmane.org; Sun, 11 Nov 2012 17:38:48 +0100 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753606Ab2KKQib convert rfc822-to-quoted-printable (ORCPT ); Sun, 11 Nov 2012 11:38:31 -0500 Received: from 75-15-5-89.uvs.iplsin.sbcglobal.net ([75.15.5.89]:43918 "EHLO peff.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753598Ab2KKQia (ORCPT ); Sun, 11 Nov 2012 11:38:30 -0500 Received: (qmail 574 invoked by uid 107); 11 Nov 2012 16:39:17 -0000 Received: from sigill.intra.peff.net (HELO sigill.intra.peff.net) (10.0.0.7) (smtp-auth username relayok, mechanism cram-md5) by peff.net (qpsmtpd/0.84) with ESMTPA; Sun, 11 Nov 2012 11:39:17 -0500 Received: by sigill.intra.peff.net (sSMTP sendmail emulation); Sun, 11 Nov 2012 11:38:28 -0500 Content-Disposition: inline In-Reply-To: <509FD425.5030702@web.de> Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Archived-At: On Sun, Nov 11, 2012 at 05:36:53PM +0100, Torsten B=C3=B6gershausen wro= te: > On 11.11.12 14:59, Felipe Contreras wrote: > > test_expect_success 'test biridectionality' ' > > + echo -n > marks-cur && > > + echo -n > marks-new && > Unless I messed up the patch: >=20 > Minor issue: still a typo "biridectionality" > Major issue: "echo -n" is still not portable. >=20 > Could we simply use >=20 > touch marks-cur && > touch marks-new Yes, "echo -n" is definitely not portable. Our preferred way of creating an empty file is just ">file". -Peff