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

Re: [PATCH v3 1/3] submodule: add test to demonstrate that shallow recursive clones fail

From
Junio C Hamano <gitster@pobox.com>
Date
Mar 15, 2016, 20:12 UTC
Message-ID
<xmqqpouv7b65.fsf@gitster.mtv.corp.google.com>
In-Reply-To
<CAGZ79kYzbmzK1UwvWheugkqG0FwhpGMU0_K_jyCKOnoKb5oTxQ@mail.gmail.com>
Stefan Beller <sbeller@google.com> writes:
Show 5 quoted lines
>> +test_expect_failure shallow-clone-recursive '
>> +       URL="file://$(pwd | sed "s/[[:space:]]/%20/g")/repo" &&
>
> This would break if the test suite is in a path containing any other white space
> than U+0020 such as a tab? (Not that I am encouraging using such paths...)
Good spotting.

I thought use of [[:named character classes:]] is explicitly disallowed by CodingGuidelines.

Previous: Stefan BellerNext: Lars Schneider
Message 4 of 14 in “add test to demonstrate that shallow recursive clones fail”
  1. 0/3 add test to demonstrate that shallow recursive clones faillarsxschneider@gmail.com, Dec 20, 2015
  2. 1/3 submodule: add test to demonstrate that shallow recursive clones faillarsxschneider@gmail.com, Dec 20, 2015
  3. Stefan BellerMar 15, 2016
  4. Junio C HamanoMar 15, 2016
  5. Lars SchneiderMar 20, 2016
  6. Jeff KingMar 20, 2016
  7. Stefan BellerMar 20, 2016
  8. 2/3 submodule: fix &&-chain breakagelarsxschneider@gmail.com, Dec 20, 2015
  9. 3/3 submodule: extend die message on failed checkout with depth argumentlarsxschneider@gmail.com, Dec 20, 2015
  10. Stefan BellerJan 11, 2016
  11. Lars SchneiderJan 12, 2016
  12. Stefan BellerJan 12, 2016
  13. Lars SchneiderJan 7, 2016
  14. Stefan BellerJan 11, 2016

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.