Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ci: MSVC fails after rebuild of Docker image #1326

Closed
real-or-random opened this issue May 24, 2023 · 0 comments · Fixed by #1327
Closed

ci: MSVC fails after rebuild of Docker image #1326

real-or-random opened this issue May 24, 2023 · 0 comments · Fixed by #1327
Labels
ci

Comments

@real-or-random
Copy link
Contributor

As part of testing #1320, I forced a rebuild of the Docker image (after the successful CI run on the old image, i.e., after @hebasto's ACK.) I think we should anyway do this from time to time...

Now MSVC refuses to run because it can't write some temp files. See for example:
https://github.com/bitcoin-core/secp256k1/pull/1320/checks?check_run_id=13686337961

Some resources:
@real-or-random real-or-random mentioned this issue May 24, 2023
Merged
hebasto pushed a commit to hebasto/secp256k1 that referenced this issue May 24, 2023
@real-or-random @hebasto
ci: Move wine prefix to /tmp to avoid error D8037 in cl.exe
17c4016 
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 bitcoin-core#1326.
real-or-random added a commit that referenced this issue May 24, 2023
@real-or-random
Merge #1327: ci: Move wine prefix to /tmp to avoid error D8037 in cl.exe
09df0bf 
27504d5 ci: Move wine prefix to /tmp to avoid error D8037 in cl.exe (Tim Ruffing)

Pull request description:

  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.

ACKs for top commit:
  hebasto:
    ACK 27504d5, tested in my personal Cirrus account.

Tree-SHA512: 08bb1734827579b59c705a44ee8fad6d504031eb5659c2743649be95fb048794b95ac0869a994bfa732f7f0714b4d12674c325637fe079b2266f18a3c14bbec0
This was referenced Jun 1, 2023
Closed
Merged
dderjoel pushed a commit to dderjoel/secp256k1 that referenced this issue Jun 21, 2023
@real-or-random @dderjoel
ci: Move wine prefix to /tmp to avoid error D8037 in cl.exe
69e6f9e 
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 bitcoin-core#1326.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
ci
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

ci: Move wine prefix to /tmp to avoid error D8037 in cl.exe real-or-random/secp256k1
1 participant
@real-or-random