Skip to content

Add firefox linux binaries for rust_wasm_bindgen_test #3320

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

Merged
merged 1 commit into from
Mar 10, 2025

Conversation

UebelAndre
Copy link
Collaborator

@UebelAndre UebelAndre commented Mar 10, 2025

The firefox binaries are only available for linux since there is no easy way to extract a .dmg for macos and the .exe that firefox distributes for the relevant platforms.

Additionally, the @rules_rust_wasm_bindgen//:test_browser setting moved to @rules_rust_wasm_bindgen//settings:test_browser

@UebelAndre UebelAndre force-pushed the wasm-bindgen branch 2 times, most recently from 74c4d27 to b70ab85 Compare March 10, 2025 19:48
@UebelAndre UebelAndre marked this pull request as ready for review March 10, 2025 19:50
@UebelAndre UebelAndre requested a review from illicitonion March 10, 2025 19:50
@UebelAndre UebelAndre enabled auto-merge March 10, 2025 19:50
@UebelAndre
Copy link
Collaborator Author

Without this the Bazel Central Registry CI would have failed since it runs bazel test //... and would have hit the issue mentioned here:

@UebelAndre UebelAndre disabled auto-merge March 10, 2025 19:53
@UebelAndre UebelAndre enabled auto-merge March 10, 2025 19:53
@UebelAndre UebelAndre force-pushed the wasm-bindgen branch 2 times, most recently from f7bdec2 to 34e1a32 Compare March 10, 2025 20:04
Copy link
Collaborator

@illicitonion illicitonion left a comment

Choose a reason for hiding this comment

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

When I have a moment I'll try to get macOS working for this, but I'm not sure when I'll have a moment :) Definitely feel free to if you have any time/inclination!

@UebelAndre
Copy link
Collaborator Author

UebelAndre commented Mar 10, 2025

When I have a moment I'll try to get macOS working for this, but I'm not sure when I'll have a moment :) Definitely feel free to if you have any time/inclination!

I mean, I developed this on a Mac. If you extract firefox from the dmg (e.g. from https://ftp.mozilla.org/pub/firefox/releases/136.0/mac/en-US/) then this works just fine. Ideally there would be a rule that could do this extraction or Mozilla would host the apps in a tar file somewhere 🙏

@UebelAndre UebelAndre added this pull request to the merge queue Mar 10, 2025
Merged via the queue into bazelbuild:main with commit 4d023c1 Mar 10, 2025
3 checks passed
@UebelAndre UebelAndre deleted the wasm-bindgen branch March 10, 2025 23:51
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