-
-
Notifications
You must be signed in to change notification settings - Fork 10.7k
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
Isolated build environment #3136
Comments
ping @rom1v 🙂 |
Using docker for that is a possibility, but the purpose of FYI, in the long term, I'd like to post a "portable" version with dependencies included, for simplicity (see #3283). |
That's what I was writing in my issues section. A solution could be to separate these processes into two shell scripts:
That sounds like just another output artifact to me, which is independent of the issues here. For me as a user it would be great, to either have a way to compile the app without cluttering my system. Or to have a prebuilt binary created by a CI action and attached to the according tag/release. This is already happening for the Windows target, but not for Linux. |
Is your feature request related to a problem? Please describe.
In order to compile the project on my machine it's required to install a couple of build dependencies that I don't want to pollute my system.
Describe the solution you'd like
I've created a build environment based on docker/podman to do the compilation task isolated in a docker container. This would also allow you to provide a failsafe build environment to all users.
Describe alternatives you've considered
Do what is described in the readme. My system would end up with dependencies that won't be required anymore, except for that one task from time to time. In the long term I gonna lose track of the dev dependencies.
My solution
Dockerfile-ubuntu-20.04
:fetch+compile.sh
:Issues
Currently, your
install_release.sh
combines two processes within a single file: compile and install. Since install requires the sudo command my script would fail (thus the workaround… || true
). It would be great to have these aspects split into separate files upstream.What do you think?
The text was updated successfully, but these errors were encountered: