Re: [PATCH 2/4] rev-parse test: use test_must_fail, not "if <command>; then false; fi"

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Subject: Re: [PATCH 2/4] rev-parse test: use test_must_fail, not "if <command>; then false; fi"
From: Felipe Contreras <felipe.contreras@xxxxxxxxx>
Date: Tue, 3 Sep 2013 16:56:25 -0500
Cc: SZEDER Gábor <szeder@xxxxxxxxxx>, Jeff King <peff@xxxxxxxx>, git@xxxxxxxxxxxxxxx
In-reply-to: <20130903170617.GC29921@google.com>

> rev-parse test: use test_must_fail, not "if <command>; then false; fi"

That is way more than 50 characters, and doesn't tell me why you want
to do that.

-- 
Felipe Contreras
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

References:
- [PATCH 0/4] t: rev-parse-parents: cleanups
  - From: Felipe Contreras
- [PATCH 3/4] t: rev-parse-parents: avoid yoda conditions
  - From: Felipe Contreras
- Re: [PATCH 3/4] t: rev-parse-parents: avoid yoda conditions
  - From: Jeff King
- Re: [PATCH 3/4] t: rev-parse-parents: avoid yoda conditions
  - From: SZEDER Gábor
- Re: [PATCH 3/4] t: rev-parse-parents: avoid yoda conditions
  - From: Jeff King
- Re: [PATCH 3/4] t: rev-parse-parents: avoid yoda conditions
  - From: Felipe Contreras
- Re: [PATCH 3/4] t: rev-parse-parents: avoid yoda conditions
  - From: SZEDER Gábor
- Re: [PATCH 3/4] t: rev-parse-parents: avoid yoda conditions
  - From: Felipe Contreras
- Re: [PATCH 3/4] t: rev-parse-parents: avoid yoda conditions
  - From: SZEDER Gábor
- [PATCH 0/4] Re: [PATCH 3/4] t: rev-parse-parents: avoid yoda conditions
  - From: Jonathan Nieder
- [PATCH 2/4] rev-parse test: use test_must_fail, not "if <command>; then false; fi"
  - From: Jonathan Nieder

Prev by Date: Re: [PATCH 0/4] Re: [PATCH 3/4] t: rev-parse-parents: avoid yoda conditions
Next by Date: Re: [PATCH 3/4] rev-parse test: use test_cmp instead of "test" builtin
Previous by thread: [PATCH 2/4] rev-parse test: use test_must_fail, not "if <command>; then false; fi"
Next by thread: [PATCH 3/4] rev-parse test: use test_cmp instead of "test" builtin
Index(es):
- Date
- Thread

[Index of Archives] [Linux Kernel Development] [Gcc Help] [IETF Annouce] [DCCP] [Netdev] [Networking] [Security] [V4L] [Bugtraq] [Yosemite] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Linux SCSI] [Fedora Users]