git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] t9600: require cvsps 2.1 to perform tests

From
Jeff King <peff@peff.net>
Date
Dec 5, 2007, 01:02 UTC
Message-ID
<20071205010202.GA4713@coredump.intra.peff.net>
In-Reply-To
<7vhciy5pg0.fsf@gitster.siamese.dyndns.org>
On Tue, Dec 04, 2007 at 09:39:59AM -0800, Junio C Hamano wrote:
Show 7 quoted lines
> You are right about "we may be depending on what 2.1 has other than
> -A".  Will apply as-is.
> 
> Thanks, both.
> 
> The primary reason I asked about '\[-A\]' was what will happen if
> somebody uses 2.2.

Yes, it would be nice to be able to easily check >2.1. GNU expr seems to handle this ok:

  $ expr 2.2 '>' 2.1
  1
  $ expr 2.0 '>' 2.1
  0

but POSIX seems to mention only integers and string comparison (though if all are of the form "x.y", string comparison works). I have no idea how portable this is.

-Peff
Previous: Junio C HamanoNext: Junio C Hamano
Message 17 of 18 in “make test failure with latest master”
  1. A Large Angry SCMDec 2, 2007
  2. Jakub NarebskiDec 2, 2007
  3. A Large Angry SCMDec 2, 2007
  4. Jeff KingDec 2, 2007
  5. A Large Angry SCMDec 2, 2007
  6. Johannes SchindelinDec 2, 2007
  7. Jeff KingDec 3, 2007
  8. Johannes SixtDec 3, 2007
  9. Jeff KingDec 3, 2007
  10. A Large Angry SCMDec 3, 2007
  11. Junio C HamanoDec 3, 2007
  12. A Large Angry SCMDec 3, 2007
  13. t9600: require cvsps 2.1 to perform testsJeff King, Dec 4, 2007
  14. Junio C HamanoDec 4, 2007
  15. Jeff KingDec 4, 2007
  16. Junio C HamanoDec 4, 2007
  17. Jeff KingDec 5, 2007
  18. Junio C HamanoDec 5, 2007

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.