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.

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

add option to disable opam sandboxing #6

Closed
sykhro opened this issue Feb 21, 2020 · 7 comments · Fixed by #66
Closed

add option to disable opam sandboxing #6

sykhro opened this issue Feb 21, 2020 · 7 comments · Fixed by #66
Labels
enhancement New feature or request

Comments

@sykhro
Copy link

sykhro commented Feb 21, 2020

It would be useful to have an option to init opam with the --disable-sandboxing flag, allowing this action to be used inside docker containers

@smorimoto smorimoto added the enhancement New feature or request label Apr 21, 2020
@avsm
Copy link
Member

avsm commented May 1, 2020

That sounds useful. Out of interest, can you use this action within a GH Action in Docker mode, or is this for running the action locally?

@sykhro
Copy link
Author

sykhro commented May 1, 2020

Running this in Docker on GH Actions I couldn't get past the opam init

@smorimoto
Copy link
Member

@avsm If you agree with this idea, I will add an implementation for this. Well, it will be implemented as an input only available on Ubuntu and macOS, as there is no option other than disabled on Windows.

@smorimoto
Copy link
Member

This will be supported in the next version.

@Risto-Stevcev
Copy link

+1 on this, would be really useful for testing locally

@smorimoto smorimoto linked a pull request Mar 26, 2021 that will close this issue
4 tasks
@smorimoto
Copy link
Member

The functionality that allows us to do this will finally be introduced in v2.

@smorimoto smorimoto mentioned this issue Mar 28, 2021
4 tasks
@smorimoto
Copy link
Member

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants