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

electron_32: init at 32.1.1, bitwarden-desktop: 2024.8.2 -> 2024.9.0 #343097

Merged
merged 6 commits into from
Sep 23, 2024

Conversation

dotlambda
Copy link
Member

Description of changes

Diff: bitwarden/clients@desktop-v2024.8.2...desktop-v2024.9.0

Changelog: https://github.com/bitwarden/clients/releases/tag/desktop-v2024.9.0

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • 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/)
  • 24.11 Release Notes (or backporting 23.11 and 24.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.

Add a 👍 reaction to pull requests you find important.

@dotlambda dotlambda changed the title bitwarden-desktop: 2024.8.2 -> 2024.9.0 electron_32: init at 32.1.1, bitwarden-desktop: 2024.8.2 -> 2024.9.0 Sep 19, 2024
@dotlambda dotlambda marked this pull request as draft September 19, 2024 22:48
Copy link
Member

@amarshall amarshall left a comment

Choose a reason for hiding this comment

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

Built and tested bitwarden-desktop (after rebasing this onto latest nixos-unstable branch). This included building electron_32 from source successfully on x86_64-linux.

While the other changes look good, my approval should probably not be seen as “enough” for them.

Note also that electron-chromedriver update got merged in #341973.

@dotlambda
Copy link
Member Author

Note also that electron-chromedriver update got merged in #341973.

Not the update to 32.1.1.

@dotlambda dotlambda marked this pull request as ready for review September 21, 2024 04:17
@dotlambda

This comment was marked as outdated.

@dotlambda dotlambda added the backport release-24.05 Backport PR automatically label Sep 21, 2024
@dotlambda
Copy link
Member Author

Result of nixpkgs-review pr 343097 run on x86_64-linux 1

2 packages marked as broken and skipped:
  • archivebox
  • archivebox.dist
2 packages failed to build:
  • bilibili
  • gotenberg
109 packages built:
  • affine
  • antares
  • antimatter-dimensions
  • armcord
  • aws-azure-login
  • bitwarden-desktop
  • bitwarden-directory-connector
  • blockbench
  • bruno
  • camunda-modeler
  • caprine
  • chatd
  • chromedriver
  • chromium
  • chromium.sandbox
  • deltachat-desktop
  • drawio
  • drawio-headless
  • electron (electron_32)
  • electron-bin (electron_32-bin)
  • electron-chromedriver (electron-chromedriver_32)
  • electron_30
  • electron_31
  • element-desktop
  • element-desktop-wayland
  • fast-cli
  • fcast-receiver
  • feishin
  • flaresolverr
  • freetube
  • goofcord
  • grafana-kiosk
  • headset
  • heroic
  • heroic-unwrapped
  • httptoolkit
  • itch
  • jitsi-meet-electron
  • koodo-reader
  • logseq
  • lx-music-desktop
  • mattermost-desktop
  • mermaid-cli
  • mermaid-filter
  • morgen
  • mqtt-explorer
  • nix-tour
  • obsidian
  • open-stage-control
  • pandoc-drawio-filter
  • pandoc-drawio-filter.dist
  • percollate
  • playwright-test
  • podman-desktop
  • pritunl-client
  • proton-pass
  • protonmail-desktop
  • puppeteer-cli
  • python311Packages.great-tables
  • python311Packages.great-tables.dist
  • python311Packages.mkdocs-drawio-exporter
  • python311Packages.mkdocs-drawio-exporter.dist
  • python311Packages.pytest-playwright
  • python311Packages.pytest-playwright.dist
  • python311Packages.shiny
  • python311Packages.shiny.dist
  • python312Packages.great-tables
  • python312Packages.great-tables.dist
  • python312Packages.mkdocs-drawio-exporter
  • python312Packages.mkdocs-drawio-exporter.dist
  • python312Packages.pytest-playwright
  • python312Packages.pytest-playwright.dist
  • python312Packages.shiny
  • python312Packages.shiny.dist
  • r2modman
  • redisinsight
  • revolt-desktop
  • ride
  • selendroid
  • selenium-server-standalone
  • sharedown
  • single-file-cli
  • sitespeed-io
  • siyuan
  • standardnotes
  • stretchly
  • super-productivity
  • teams-for-linux
  • terra-station
  • tetrio-desktop
  • thedesk
  • threema-desktop
  • uhk-agent
  • uhk-udev-rules
  • uivonim
  • undetected-chromedriver
  • ungoogled-chromium
  • ungoogled-chromium.sandbox
  • vesktop
  • vhs
  • vieb
  • wayback
  • webcord
  • webcord-vencord
  • webtorrent_desktop
  • yandex-music
  • youtube-music
  • ytdownloader
  • ytmdesktop

Copy link
Member

@yayayayaka yayayayaka left a comment

Choose a reason for hiding this comment

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

Thanks a lot for taking care of it! ❤️

Diff looking good to me, should be good to go.

@dotlambda dotlambda merged commit 64a773b into NixOS:master Sep 23, 2024
28 of 30 checks passed
@dotlambda dotlambda deleted the bitwarden-desktop branch September 23, 2024 14:31
Copy link
Contributor

Backport failed for release-24.05, because it was unable to cherry-pick the commit(s).

Please cherry-pick the changes locally and resolve any conflicts.

git fetch origin release-24.05
git worktree add -d .worktree/backport-343097-to-release-24.05 origin/release-24.05
cd .worktree/backport-343097-to-release-24.05
git switch --create backport-343097-to-release-24.05
git cherry-pick -x 90effb99edf82b41a082c3cf6e8726585f4b59c6 483e68aa6c6ad60b8ee283dac8eb3252ed2325a4 9e4edca155dbc29551f46bb6ebccc3948999753d b075f7d893fcc0c05a80714354598067ff6ccd13 bac1f69114aace9b58a5eeb68c15ba9c7f1ae575 f01b67406414907f2c95cd132003ae46590c757c

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants