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

firefox: build from source tarball instead of hg #40036

Merged
merged 1 commit into from
Jan 21, 2025

Conversation

OddBloke
Copy link
Member

@OddBloke OddBloke commented Jan 20, 2025

The Mercurial _RELEASE tag seems to be created some time after the release has been made public on ftp.mozilla.org. release-monitoring.org uses ftp.mozilla.org as its source of truth, which generally means that we detect the new version and attempt to build the release before the tag has been published, resulting in build failures.

Switching to the release tarball should avoid manual intervention for every Firefox release, thereby decreasing the latency from upstream release to availability in Wolfi. It has the additional advantage of future-proofing us against the planned migration to GitHub for development (though once that's occurred, we should examine whether consuming from GitHub directly would be appropriate). (wget is probably also faster than hg clone, though I haven't confirmed this.)

@octo-sts octo-sts bot added the bincapz/pass bincapz/pass Bincapz (aka. malcontent) scan didn't detect any CRITICALs on the scanned packages. label Jan 20, 2025
Copy link
Member

@EyeCantCU EyeCantCU left a comment

Choose a reason for hiding this comment

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

This looks like a significant improvement over cloning the source with mercurial. May need to bump the epoch here though (elastic never ran)

firefox.yaml Outdated Show resolved Hide resolved
@OddBloke OddBloke force-pushed the oddbloke/firefox branch 2 times, most recently from 5612041 to 2d5388a Compare January 21, 2025 17:38

This comment was marked as resolved.

@octo-sts octo-sts bot added the ai/skip-comment Stop AI from commenting on PR label Jan 21, 2025
The Mercurial `_RELEASE` tag seems to be created some time after the
release has been made public on ftp.mozilla.org. release-monitoring.org
uses ftp.mozilla.org as its source of truth, which generally means that
we detect the new version and attempt to build the release before the
tag has been published, resulting in build failures.

Switching to the release tarball should avoid manual intervention for
every Firefox release, thereby decreasing the latency from upstream
release to availability in Wolfi.  It has the additional advantage of
future-proofing us against the planned migration to GitHub for
development (though once that's occurred, we should examine whether
consuming from GitHub directly would be appropriate).  (`wget` is
probably also faster than `hg clone`, though I haven't confirmed this.)
@OddBloke OddBloke marked this pull request as ready for review January 21, 2025 19:10
@OddBloke OddBloke requested a review from EyeCantCU January 21, 2025 19:10
@OddBloke OddBloke enabled auto-merge (squash) January 21, 2025 19:15
@OddBloke OddBloke merged commit 62af0a6 into wolfi-dev:main Jan 21, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ai/skip-comment Stop AI from commenting on PR bincapz/pass bincapz/pass Bincapz (aka. malcontent) scan didn't detect any CRITICALs on the scanned packages.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants