Skip to content

fix sandboxing by setting zig cache#2

Open
avsm wants to merge 1 commit into
chris-armstrong:mainfrom
avsm:fix-sandboxing
Open

fix sandboxing by setting zig cache#2
avsm wants to merge 1 commit into
chris-armstrong:mainfrom
avsm:fix-sandboxing

Conversation

@avsm
Copy link
Copy Markdown

@avsm avsm commented Feb 22, 2025

fixes #1

This works on macOS for me now! Trying to get a rPi binary from my mac next...

@chris-armstrong
Copy link
Copy Markdown
Owner

Thanks - this does solve the problem, but it's no longer working on Ubuntu builds because the $TMPDIR variable is not set.

I think that long term, we want to either inject or add into the build and install wrap scripts supplied by opam an exception to the default zig read/write cache directories. Another option is to redirect them in the build and install scripts if we detect the directories provided are not writeable.

I'll create an issue to track this but I think this solution is good enough to avoid issues and complications for now.

1 similar comment
@chris-armstrong
Copy link
Copy Markdown
Owner

Thanks - this does solve the problem, but it's no longer working on Ubuntu builds because the $TMPDIR variable is not set.

I think that long term, we want to either inject or add into the build and install wrap scripts supplied by opam an exception to the default zig read/write cache directories. Another option is to redirect them in the build and install scripts if we detect the directories provided are not writeable.

I'll create an issue to track this but I think this solution is good enough to avoid issues and complications for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

doesn't work if opam sandboxing is enabled

2 participants