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

Re: [PATCH] repack: find -> /usr/bin/find, as for cygwin

From
Matthieu Moy <matthieu.moy@grenoble-inp.fr>
Date
Mar 20, 2011, 07:48 UTC
Message-ID
<vpqlj0ajkqa.fsf@bauges.imag.fr>
In-Reply-To
<AANLkTinFFnm7-TKcbTdbzJHx4i4L8rdi4xogr3R7=7s1@mail.gmail.com>
ryenus ◇ <ryenus@gmail.com> writes:
Show 6 quoted lines
> oops, corrected the script with the test strings in upper cases
>
> #!/bin/sh
> echo $(uname -s)
> case $(uname -s) in
> *MINGW*|*CYGWIN*)
         ^
This "|" means "or" in a case statement...
>   echo "detected MinGW/Cygwin"
>   ;;
> *MINGW*)

...so I can see no way to reach this point: if the string matches *MINGW*, it also matches *MINGW*|*CYGWIN*.

Show 6 quoted lines
>   echo "detected MinGW"
>   ;;
> *CYGWIN*)
>   echo "detected Cygwin"
>   ;;
> esac

But you've just showed that $(uname -s) of Cygwin did contain upper-case CYGWIN, which I think was the point to verify :-).

-- 
Matthieu Moy
http://www-verimag.imag.fr/~moy/
Previous: ryenus ◇Next: ryenus ◇
Message 12 of 14 in “repack: find -> /usr/bin/find, as for cygwin”
  1. repack: find -> /usr/bin/find, as for cygwinryenus ◇, Mar 19, 2011
  2. Nguyen Thai Ngoc DuyMar 19, 2011
  3. René ScharfeMar 19, 2011
  4. Nguyen Thai Ngoc DuyMar 19, 2011
  5. Nguyen Thai Ngoc DuyMar 19, 2011
  6. ryenus ◇Mar 19, 2011
  7. ryenus ◇Mar 19, 2011
  8. Nguyen Thai Ngoc DuyMar 19, 2011
  9. Junio C HamanoMar 19, 2011
  10. ryenus ◇Mar 20, 2011
  11. ryenus ◇Mar 20, 2011
  12. Matthieu MoyMar 20, 2011
  13. ryenus ◇Mar 20, 2011
  14. Erik Faye-LundMar 21, 2011

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.