Skip to content

[rust] Use PKG again for Firefox macOS downloads#17702

Merged
AutomatedTester merged 1 commit into
trunkfrom
worktree-revert-firefox-pkg
Jun 22, 2026
Merged

[rust] Use PKG again for Firefox macOS downloads#17702
AutomatedTester merged 1 commit into
trunkfrom
worktree-revert-firefox-pkg

Conversation

@AutomatedTester

Copy link
Copy Markdown
Member

Description

Restores the version-based PKG/DMG artifact selection for Firefox macOS downloads in Selenium Manager.

In #17687 a workaround was added that forced DMG for all macOS Firefox downloads, because PKG extraction was failing with a cpio bad magic value error on newer Firefox builds (Mozilla changed the internal pkg format to use pbzx compression).

That root cause was fixed in #17691 (Support pbzx-compressed macOS .pkg payloads), so the workaround is no longer needed. This reverts the firefox.rs change to once again use DMG only for Firefox < 68 and PKG for 68+.

Motivation and Context

Removes a now-unnecessary workaround now that PKG extraction works again.

Types of changes

  • Cleanup (formatting, renaming)

🤖 Generated with Claude Code

The DMG-only workaround was added because PKG extraction failed with a
cpio "bad magic value" error on newer Firefox builds. That root cause was
fixed in #17691 (support for pbzx-compressed macOS .pkg payloads), so the
version-based PKG/DMG selection can be restored.
@selenium-ci selenium-ci added C-rust Rust code is mostly Selenium Manager B-manager Selenium Manager labels Jun 22, 2026
@AutomatedTester AutomatedTester merged commit 329b994 into trunk Jun 22, 2026
66 checks passed
@AutomatedTester AutomatedTester deleted the worktree-revert-firefox-pkg branch June 22, 2026 13:45
This was referenced Jul 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

B-manager Selenium Manager C-rust Rust code is mostly Selenium Manager

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants