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
electron: 1.8.1 -> 1.7.9 #32768
electron: 1.8.1 -> 1.7.9 #32768
Conversation
@@ -18,23 +18,15 @@ let | |||
src = { | |||
i686-linux = fetchurl { | |||
url = "https://github.com/electron/electron/releases/download/v${version}/electron-v${version}-linux-ia32.zip"; | |||
sha256 = "0djqlcs4m9n9354idaqcs4cwskq2m3sf9mzvxpp4wy0a93pk78bw"; | |||
name = "${name}.zip"; | |||
sha256 = "0m87n7hqimg93z3m8pa1ggs69f3h5mjrsrrl7x80hxmp3w142krc"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please fix indentation nits :-)
Those facts combined make me think that it's more trouble than it's worth. |
BTW, I have a source-based Electron in the works, but it will take a while because their build system implies internet access, which makes it hard to get working, all dependencies are pinned/vendored, and builds take a very long time. |
This pull request follows the upstream. v1.7.9 is the latest stable version, v1.8.1 is beta. I don't believe that pre-release versions should be packaged by default. Rambox is just an HTML page and some JavaScript. It's a highly sandboxed environment, and if anything breaks to the point of a failed assert in such environment, release is probably buggy. |
Same for me. Thank you! |
840106a
to
8f106df
Compare
1.7.9 is the latest stable, and 1.8.1 is beta and it draws `rambox` as a white rectangle (after a few manipulations with the window such as hiding, showing, resizing): NixOS#32741 (comment)
8f106df
to
a9d212f
Compare
Motivation for this change
Downgrade Electron to latest stable version. See #32398, #32741, #32765.
I can't reproduce that bug in Rambox with Electron 1.7.9. /cc @gnidorah
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)