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

Re: [PATCH v4 00/12] Wildmatch v4

From
Torsten Bögershausen <tboegi@web.de>
Date
Oct 12, 2012, 20:28 UTC
Message-ID
<50787D5D.3080509@web.de>
In-Reply-To
<7vbog74o59.fsf@alter.siamese.dyndns.org>
On 10/12/2012 07:05 PM, Junio C Hamano wrote:
Show 11 quoted lines
>> Some problems even here (Mac OS) commit 2aeb6d4d7884f4c4425
>>
>> not ok 61 - wildmatch 0 0 \ \
>> not ok 62 - wildmatch 0 0 /\ */\
>> not ok 69 - wildmatch 1 1 [ab] [[:]ab]
>> not ok 71 - wildmatch 1 1 [ab] [[:digit]ab]
>> not ok 80 - wildmatch 1 0 1 [[:digit:][:upper:][:space:]]
>> not ok 81 - wildmatch 0 0 1 [[:digit:][:upper:][:spaci:]]
>> not ok 88 - wildmatch 1 0 _ [[:alnum:][:alpha:][:blank:][:cntrl:][:digit:][:graph:][:lower:][:print:][:punct:][:space:][:upper:][:xdigit:]]
>> not ok 89 - wildmatch 1 0 _ [[:alnum:][:alpha:][:blank:][:cntrl:][:digit:][:graph:][:lower:][:print:][:punct:][:space:][:upper:][:xdigit:]]
>> not ok 93 - wildmatch 1 1 y [a-c[:digit:]x-z]
> Nguyen, how about updating the match () shell function in 3070 so
> that it not just says not-ok, but indicates what failed (wildmatch
> failed, or wildmatch passed but fnmatch failed), at least when the
> test is run as "./t3070-*.sh -v -i"?

It seams to be the fnmatch, since removing the fnmatch makes t3070 pass.

However, pullling in compat/fnmatch in the Makefile
  ifeq ($(uname_S),Darwin)
+       NO_FNMATCH = YesPlease
================
gives this result:
  ./t3070-wildmatch.sh 2>&1 | grep "not ok"
not ok 23 - wildmatch 1 1 a]b a[]]b
not ok 25 - wildmatch 1 1 a]b a[]-]b
not ok 43 - wildmatch 1 1 /foo **/foo
not ok 79 - wildmatch 1 1 A [[:digit:][:upper:][:space:]]
not ok 84 - wildmatch 1 1 . [[:digit:][:punct:][:space:]]
not ok 85 - wildmatch 1 1 5 [[:xdigit:]]
not ok 86 - wildmatch 1 1 f [[:xdigit:]]
not ok 87 - wildmatch 1 1 D [[:xdigit:]]
not ok 90 - wildmatch 1 1 . 
[^[:alnum:][:alpha:][:blank:][:cntrl:][:digit:][:lower:][:space:][:upper:][:xdigit:]]
not ok 92 - wildmatch 1 1 b [a-c[:digit:]x-z]
not ok 95 - wildmatch 1 1 ] [\\-^]
not ok 98 - wildmatch 1 1 ] [\]]
(Confused)
Previous: Junio C HamanoNext: Nguyen Thai Ngoc Duy
Message 27 of 28 in “Wildmatch v4”
  1. 00/12 Wildmatch v4Nguyễn Thái Ngọc Duy, Oct 10, 2012
  2. 01/12 ctype: make sane_ctype[] const arrayNguyễn Thái Ngọc Duy, Oct 10, 2012
  3. 02/12 ctype: support iscntrl, ispunct, isxdigit and isprintNguyễn Thái Ngọc Duy, Oct 10, 2012
  4. 03/12 Import wildmatch from rsyncNguyễn Thái Ngọc Duy, Oct 10, 2012
  5. 04/12 wildmatch: remove unnecessary functionsNguyễn Thái Ngọc Duy, Oct 10, 2012
  6. Michael HaggertyOct 10, 2012
  7. 05/12 Integrate wildmatch to gitNguyễn Thái Ngọc Duy, Oct 10, 2012
  8. Johannes SixtOct 12, 2012
  9. Junio C HamanoOct 12, 2012
  10. 06/12 wildmatch: make wildmatch's return value compatible with fnmatchNguyễn Thái Ngọc Duy, Oct 10, 2012
  11. 07/12 wildmatch: remove static variable force_lower_caseNguyễn Thái Ngọc Duy, Oct 10, 2012
  12. 08/12 wildmatch: fix case-insensitive matchingNguyễn Thái Ngọc Duy, Oct 10, 2012
  13. 09/12 wildmatch: adjust "**" behaviorNguyễn Thái Ngọc Duy, Oct 10, 2012
  14. 10/12 wildmatch: make /**/ match zero or more directoriesNguyễn Thái Ngọc Duy, Oct 10, 2012
  15. 11/12 Support "**" wildcard in .gitignore and .gitattributesNguyễn Thái Ngọc Duy, Oct 10, 2012
  16. 12/12 t3070: disable two fnmatch tests that have different results on different libcNguyễn Thái Ngọc Duy, Oct 10, 2012
  17. Johannes SixtOct 12, 2012
  18. Nguyen Thai Ngoc DuyOct 12, 2012
  19. Junio C HamanoOct 10, 2012
  20. Junio C HamanoOct 11, 2012
  21. Nguyen Thai Ngoc DuyOct 11, 2012
  22. Junio C HamanoOct 11, 2012
  23. Junio C HamanoOct 11, 2012
  24. Junio C HamanoOct 11, 2012
  25. Torsten BögershausenOct 12, 2012
  26. Junio C HamanoOct 12, 2012
  27. Torsten BögershausenOct 12, 2012
  28. Nguyen Thai Ngoc DuyOct 13, 2012

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.