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
[20.09] element: 1.7.24 -> 1.7.25 #119241
Conversation
ChangeLog: https://github.com/vector-im/element-desktop/releases/tag/v1.7.25 (cherry picked from commit 9fed67f)
ChangeLog: https://github.com/vector-im/element-web/releases/tag/v1.7.25 (cherry picked from commit c1ebd28)
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.
- Diff LGTM
- Commits LGTM
- Builds via
nix-review
on x86-64-linux:
1 package built:
element-desktop
[nix-shell:~/.cache/nixpkgs-review/pr-119241]$ element-desktop --help
Keytar isn't installed; secure key storage is disabled.
Seshat isn't installed, event indexing is disabled.
Options:
--profile-dir {path}: Path to where to store the profile.
--profile {name}: Name of alternate profile to use, allows for running multiple accounts.
--devtools: Install and use react-devtools and react-perf.
--no-update: Disable automatic updating.
--hidden: Start the application hidden in the system tray.
--help: Displays this help message.
And more such as --proxy, see:https://electronjs.org/docs/api/command-line-switches
Yeah, have you tried, actually running |
Nope. When I did, I got the same error you did. I'm not familiar enough with |
https://github.com/electron/electron/releases/tag/v12.0.0 (cherry picked from commit a8fde9c)
https://github.com/electron/electron/releases/tag/v12.0.1 (cherry picked from commit 7669bae)
https://github.com/electron/electron/releases/tag/v12.0.2 (cherry picked from commit 5fd6188)
@TredwellGit done :) |
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.
ACK for the electron-related changes
Motivation for this change
Backport PR for #119233. For some reason
element-desktop
is breaking:We probably need electron 12 to fix this, but I didn't push this for now since I don't know and don't use element-desktop.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)