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 support for ELECTRON_BUILDER_BINARIES_CUSTOM_DIR #99

Open
mirayashi opened this issue Jan 19, 2024 · 2 comments
Open

Add support for ELECTRON_BUILDER_BINARIES_CUSTOM_DIR #99

mirayashi opened this issue Jan 19, 2024 · 2 comments

Comments

@mirayashi
Copy link

mirayashi commented Jan 19, 2024

Currently it is impossible to customize the directory part of the URL when downloading winCodeSign-[version].7z for example

This line needs to be changed so we can specify a custom value from env instead of id:

return DownloadArtifact(id, GetGithubBaseUrl()+id+"/"+id+".7z", checksum)

The main electron-builder repo already supports ELECTRON_BUILDER_BINARIES_CUSTOM_DIR so why not use it here?

@timothyjmtan
Copy link

timothyjmtan commented Dec 18, 2024

I agree, this will help teams who are trying to get it working while behind a corporate firewall which presents loads of network connectivity restrictions.

I have an offline copy of winCodeSign-2.6.0.7z, but I'm not able to point app-builder-bin towards this because it's hardcoded to download from github which will always fail.

Ideally, it can check for the presence of files in ELECTRON_BUILDER_BINARIES_CUSTOM_DIR, before attempting to download if there aren'tt any.

The enhancement could perhaps be somewhat aligned to these two related issues:
Prevent duplicate downloads of electron binary #7079
feat: electron builder binaries base url from env #19

What do you think, @mmaietta?

@timothyjmtan
Copy link

timothyjmtan commented Dec 19, 2024

I've also tried the Docker image, but the build process still attempts to download winCodeSign. If all dependencies (including winCodeSign, etc) could be packaged into the docker image, that would be very helpful and helps circumvent the above connectivity issues. Otherwise, teams will be totally blocked from using this great tool.

https://hub.docker.com/r/electronuserland/builder

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

No branches or pull requests

2 participants