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

Include libadwaita and other dependencies in the app bundle on macOS #271

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

mrled
Copy link
Contributor

@mrled mrled commented Feb 26, 2025

Copy all the requisite homebrew libraries, including libadwaita, into the macOS bundle. This makes it a standalone app that can be executed directly on any system without installing Homebrew and libadwaita first. Also does away with the requirement that Homebrew is installed to the default location, which is otherwise required because the binary refers to dynamic libraries via their absolute paths.

@mrled mrled force-pushed the include-libadwaita-macos-bundle branch from a1d3c7b to 832ce91 Compare February 26, 2025 21:28
@mrled
Copy link
Contributor Author

mrled commented Feb 27, 2025

Not sure what's up with that Linux failure, I didn't change any of the application code. Maybe it's using some newer version of Rust with more warnings enabled?

@feschber
Copy link
Owner

Not sure what's up with that Linux failure, I didn't change any of the application code. Maybe it's using some newer version of Rust with more warnings enabled?

It seems like there are some new clippy lints in the newest version. I will fix that real quick

@feschber
Copy link
Owner

You can rebase now ;)

@mrled mrled force-pushed the include-libadwaita-macos-bundle branch from b41aa8e to f59f91d Compare February 27, 2025 16:21
@mrled
Copy link
Contributor Author

mrled commented Feb 27, 2025

There we go, it's passing now

@feschber
Copy link
Owner

Whats your status on this PR?

@mrled
Copy link
Contributor Author

mrled commented Mar 20, 2025

Just fixing a minor permissions thing and it's good to go if you want it :)

  • Testing this on a fresh VM with no homebrew lets the app launch and work normally
  • The permissions problem doesn't actually break anything, but it shows warnings when the quarantine is cleared with xattr -rd. My most recent commit fixes this.

@feschber
Copy link
Owner

Awesome! I will see if I can setup a vm again to at least test it myself once. It's kind of a pain on non-macos.

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