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

Re: [PATCH 0/4] Link worktrees with relative paths

From
Eric Sunshine <sunshine@sunshineco.com>
Date
Oct 6, 2024, 07:45 UTC
Message-ID
<CAPig+cQZmEdx1Th9VAHbEkJ=zm73TP7Wuda_G5CzFw7j9HTFQQ@mail.gmail.com>
In-Reply-To
<AtSpzKynOkmMPZ3bR4qx-eA93Xo-1miPCIQSxIKP534tD-qpDU1AI74vqB78RsukQ0XP5eKdvzHcwy_lf8lpL9wAoHoom-koo5GKcbrcuX0=@pm.me>
On Sun, Oct 6, 2024 at 2:23 AM Caleb White <cdwhite3@pm.me> wrote:
Show 8 quoted lines
> On 10/6/24 01:16, Eric Sunshine <sunshine@sunshineco.com> wrote:
> >  Indeed, I'm seeing the same line-wrapping breakage in the mailing list
> >  archive and in my Inbox for v2.
> >
> >  I presume that you've verified that the raw patches are not broken
> >  like that, and that the problem is happening at email time?
>
> Yes, I've checked the patch files and they are good. I'm also not seeing any breakages in my inbox which is strange.

I've been trying to hand-edit v2 to fix the breakage in order to get it applied, but I'm not having much success. I did manage to get patch [1/4] applied, but patch [2/4] is base64-encoded for some reason (you can see it here[*]), so I haven't been able to edit it easily in order to fix the breakage. `git am` isn't particularly happy with it either; it complains:

    % git am PATCH-v2-0-4-Link-worktrees-with-relative-paths.mbox
    Applying: worktree: refactor infer_backlink() to use *strbuf
    warning: quoted CRLF detected
    Applying: worktree: link worktrees with relative paths
    .git/rebase-apply/patch:60: trailing whitespace.
    #!/bin/sh
    .git/rebase-apply/patch:61: trailing whitespace.
    .git/rebase-apply/patch:62: trailing whitespace.
    test_description='test worktrees linked with relative paths'
    .git/rebase-apply/patch:63: trailing whitespace.
    .git/rebase-apply/patch:64: trailing whitespace.
    TEST_PASSES_SANITIZE_LEAK=true
    error: patch failed: builtin/worktree.c:414
    error: builtin/worktree.c: patch does not apply
    error: patch failed: worktree.c:110
    error: worktree.c: patch does not apply
    Patch failed at 0002 worktree: link worktrees with relative paths

Can you try GitGitGadget instead (preferable) or perhaps publish this series somewhere (less preferable).

[*]: https://lore.kernel.org/git/20241006060017.171788-3-cdwhite3@pm.me/raw
Previous: Caleb WhiteNext: Kristoffer Haugsbakk
Message 18 of 23 in “Link worktrees with relative paths”
  1. 0/4 Link worktrees with relative pathsCaleb White, Oct 6, 2024
  2. 1/4 worktree: refactor infer_backlink() to use *strbufCaleb White, Oct 6, 2024
  3. 2/4 worktree: link worktrees with relative pathsCaleb White, Oct 6, 2024
  4. 3/4 worktree: sync worktree paths after gitdir moveCaleb White, Oct 6, 2024
  5. 4/4 worktree: prevent null pointer dereferenceCaleb White, Oct 6, 2024
  6. Eric SunshineOct 6, 2024
  7. Caleb WhiteOct 6, 2024
  8. Eric SunshineOct 6, 2024
  9. Eric SunshineOct 6, 2024
  10. Eric SunshineOct 6, 2024
  11. Caleb WhiteOct 6, 2024
  12. Eric SunshineOct 6, 2024
  13. Caleb WhiteOct 6, 2024
  14. Eric SunshineOct 6, 2024
  15. Caleb WhiteOct 6, 2024
  16. Eric SunshineOct 6, 2024
  17. Caleb WhiteOct 6, 2024
  18. Eric SunshineOct 6, 2024
  19. Kristoffer HaugsbakkOct 6, 2024
  20. Caleb WhiteOct 6, 2024
  21. Caleb WhiteOct 6, 2024
  22. Eric SunshineOct 6, 2024
  23. Caleb WhiteOct 6, 2024

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.