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

Re: ./t3310-notes-merge-manual-resolve.sh broken on pu under Mac OS ?

From
Eric Sunshine <sunshine@sunshineco.com>
Date
Dec 31, 2015, 06:08 UTC
Message-ID
<CAPig+cQZq-aDYv5G3eNivvqC0eCTfyPtA4gEuop0PWq2LFT=9Q@mail.gmail.com>
In-Reply-To
<5683DA04.6000007@web.de>
On Wed, Dec 30, 2015 at 8:20 AM, Torsten Bögershausen <tboegi@web.de> wrote:
> I got 2 failures on pu under Mac OS, (Linux is OK)
> I did some very basic debugging, it seems as if grep doesn't find
> a needed string.
> Does anybody have an idea here ?
I'm unable to reproduce these failures on Mac.
Show 56 quoted lines
> --------------------
> Failure 1:
> Add some debug code:
> git diff
>
> -       grep -q refs/notes/z merge_commit_msg &&
> +       echo >&2 before grep &&
> +       echo >&2 before cat &&
> +       cat >&2 merge_commit_msg &&
> +       echo >&2 after cat &&
> +       grep  refs/notes/z merge_commit_msg >&2 &&
>         # Merge commit mentions conflicting notes
> +       echo >&2 after grep &&
>
> ------ run the test:
> debug=t verbose=t   ./t3310-notes-merge-manual-resolve.sh 2>&1 | less
> -------- get this output:
> before grep
> before cat
> Merged notes from z into refs/notes/m
>
> Conflicts:
>         04ed9a0bbe0285f41c913e2002cb09ef8449d4a6
>         0f7aa3ec6325aeb88b910453bb3eb37c49d75c11
>         7a4ca6ee52a974a66cbaa78e33214535dff1d691
>         d07d62e5208f22eb5695e7eb47667dc8b9860290
>
> after cat
> not ok 13 - finalize conflicting merge (z => m)
>
>
> -----------------------
> Failure 2:
> index d557212..92fabf0 100755
> --- a/t/t3310-notes-merge-manual-resolve.sh
> +++ b/t/t3310-notes-merge-manual-resolve.sh
> @@ -472,6 +472,9 @@ EOF
>         test "$(git rev-parse refs/notes/m^2)" = "$(cat pre_merge_z)" &&
>         # Merge commit mentions the notes refs merged
>         git log -1 --format=%B refs/notes/m > merge_commit_msg &&
> +       echo >&2 before cat &&
> +       cat merge_commit_msg >&2 &&
> +       echo >&2 after cat &&
>
> not ok 17 - add + remove notes in finalized merge (z => m)
> ---------- gives this:
> before cat
> Merged notes from z into refs/notes/m
>
> Conflicts:
>         04ed9a0bbe0285f41c913e2002cb09ef8449d4a6
>         0f7aa3ec6325aeb88b910453bb3eb37c49d75c11
>         7a4ca6ee52a974a66cbaa78e33214535dff1d691
>         d07d62e5208f22eb5695e7eb47667dc8b9860290
>
> after cat
Previous: Torsten BögershausenNext: Ramsay Jones
Message 2 of 4 in “./t3310-notes-merge-manual-resolve.sh broken on pu under Mac OS ?”
  1. Torsten BögershausenDec 30, 2015
  2. Eric SunshineDec 31, 2015
  3. Ramsay JonesDec 31, 2015
  4. Torsten BögershausenDec 31, 2015

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.