Skip to content

Revert "chore(suite-desktop-core): bump openpgp"#15305

Merged
Lemonexe merged 1 commit intodevelopfrom
chore/revert-openpgp-bump
Nov 7, 2024
Merged

Revert "chore(suite-desktop-core): bump openpgp"#15305
Lemonexe merged 1 commit intodevelopfrom
chore/revert-openpgp-bump

Conversation

@Lemonexe
Copy link
Copy Markdown
Contributor

@Lemonexe Lemonexe commented Nov 7, 2024

This reverts commit a66ece5 (downgrades OpenPGP back to 5.11.2)

See slack convo, it broke windows build 😱
This fixes the windows build ✔️

Reason not yet known, but I guess that when they rewritten the library to mjs, it's not exported as it should, or maybe has hacky code that hardcodes CWD at build time, IDK, I'll have to check.

@Lemonexe Lemonexe marked this pull request as ready for review November 7, 2024 15:27
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Nov 7, 2024

🚀 Expo preview is ready!

  • Project → trezor-suite-preview
  • Platforms → android, ios
  • Scheme → trezorsuitelite
  • Runtime Version → 10
  • More info

Learn more about 𝝠 Expo Github Action

Copy link
Copy Markdown
Member

@martykan martykan left a comment

Choose a reason for hiding this comment

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

Confirming this seems to fix it

@Lemonexe Lemonexe merged commit 9b5c70e into develop Nov 7, 2024
@Lemonexe Lemonexe deleted the chore/revert-openpgp-bump branch November 7, 2024 16:00
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

Successfully merging this pull request may close these issues.

2 participants