ci: Move wine prefix to /tmp to avoid error D8037 in cl.exe #1327

pull real-or-random wants to merge 1 commits into bitcoin-core:master from real-or-random:202305-wineprefix changing 1 files +3 −1
  1. real-or-random commented at 12:27 pm on May 24, 2023: contributor

    Don’t ask me why this makes a difference. It may be some permission problem even though everything in Cirrus CI runs as root anyway. In any case, I’ll probably get mad if I investigate this further.

    Fixes #1326.

  2. ci: Move wine prefix to /tmp to avoid error D8037 in cl.exe
    Don't ask me why this makes a difference. It may be some permission
    problem even though everything in Cirrus CI runs as root anyway. In
    any case, I'll probably get mad if I investigate this further.
    
    Fixes #1326.
    27504d5c94
  3. real-or-random force-pushed on May 24, 2023
  4. hebasto approved
  5. hebasto commented at 1:33 pm on May 24, 2023: member

    ACK 27504d5c941df89bc828067248270179c9dcb04b, tested in my personal Cirrus account.

    Don’t ask me why this makes a difference.

    I won’t :)

  6. real-or-random merged this on May 24, 2023
  7. real-or-random closed this on May 24, 2023

  8. hebasto cross-referenced this on Jun 2, 2023 from issue ci: Fix error D8037 in `cl.exe` (attempt 2) by hebasto
  9. hebasto commented at 5:30 pm on June 2, 2023: member

    Apparently this change does not work – see CI in #1320.

    ACK 27504d5, tested in my personal Cirrus account.

    I’m going to skim my CI logs to try to figure out why it passed.

    Anyway, suggesting to revert this PR in #1337.

  10. real-or-random commented at 5:48 pm on June 2, 2023: contributor

    Apparently this change does not work – see CI in #1320.

    I’m pretty sure that I could reliably reproduce D8037 in Docker locally, and moving WINEPREFIX to /tmp made this go away reliably. So, I’m rather convinced that there’s some truth in this PR here, even if it fails now in #1320. (But yeah, we’ll need to investigate further. Not sure at the moment).

  11. real-or-random referenced this in commit 60556c9f49 on Jun 3, 2023
  12. vmta referenced this in commit e1120c94a1 on Jun 4, 2023
  13. sipa referenced this in commit 901336eee7 on Jun 21, 2023
  14. vmta referenced this in commit 8f03457eed on Jul 1, 2023
  15. hebasto referenced this in commit 270d2b37b8 on Jul 21, 2023

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin-core/secp256k1. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-11-21 20:15 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me