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

bitwarden: 2023.9.0 -> 2023.9.3 #260699

Closed
wants to merge 1 commit into from

Conversation

amarshall
Copy link
Member

@amarshall amarshall commented Oct 12, 2023

Diff: bitwarden/clients@desktop-v2023.9.0...desktop-v2023.9.3

Changelog: https://github.com/bitwarden/clients/releases/tag/desktop-v2023.9.3

Description of changes

Things done

Note I have only tested execution when built against nixpkgs with glibc 2.38 backported, as that’s what my NixOS is running and trying to execute Bitwarden compiled against glibc 2.37 on a 2.38 runtime fails with glibc version mismatch.

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 23.11 Release Notes (or backporting 23.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

@ofborg ofborg bot requested a review from Kiwi October 12, 2023 15:31
@ofborg ofborg bot added 11.by: package-maintainer This PR was created by the maintainer of the package it changes 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 1-10 10.rebuild-linux: 1 labels Oct 12, 2023
@amarshall amarshall marked this pull request as draft October 15, 2023 04:40
@amarshall
Copy link
Member Author

amarshall commented Oct 15, 2023

Please do not merge yet due to bitwarden/clients#6560. While downgrading electron to 25 “works”, app segfaults on quit which seems bad.

They released 2023.9.3, it still segfaults on exit so I guess that’s “expected behavior” for now while they fix more things.

@amarshall amarshall changed the title bitwarden: 2023.9.0 -> 2023.9.2 bitwarden: 2023.9.0 -> 2023.9.3 Oct 17, 2023
@amarshall amarshall marked this pull request as ready for review October 18, 2023 03:26
@SuperSandro2000
Copy link
Member

@ofborg eval
@ofborg build bitwarden

Copy link
Member

@SuperSandro2000 SuperSandro2000 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure why the build fails with fore me

Error: unknown error

Caused by:
    [55] Failed sending data to the peer

@@ -3,7 +3,7 @@
, buildNpmPackage
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

applyPatches is unused

@@ -26,7 +26,7 @@ let
icon = "bitwarden";

buildNpmPackage' = buildNpmPackage.override { nodejs = nodejs_18; };
electron = electron_24;
electron = electron_25;

desktopItem = makeDesktopItem {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would be nice if we would use copyDekstopItems instead of manually copying it.

@dotlambda dotlambda mentioned this pull request Oct 22, 2023
13 tasks
@dotlambda
Copy link
Member

Here's a version with the requested changes cause I don't have permission to push to this branch: #262797

Copy link
Member

@SuperSandro2000 SuperSandro2000 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rebasing the branch on master fixed my problem, now working like a charm

@amarshall
Copy link
Member Author

@SuperSandro2000 That build error is due to bug in fetching NPM deps, fixed in #260011

@amarshall amarshall deleted the bitwarden-update branch October 23, 2023 20:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin 10.rebuild-linux: 1-10 10.rebuild-linux: 1 11.by: package-maintainer This PR was created by the maintainer of the package it changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants