Skip to content

Upload build artifacts#263

Merged
tmbrbr merged 12 commits into
mainfrom
actions-publish-artifacts
Mar 18, 2025
Merged

Upload build artifacts#263
tmbrbr merged 12 commits into
mainfrom
actions-publish-artifacts

Conversation

@tmbrbr

@tmbrbr tmbrbr commented Feb 27, 2025

Copy link
Copy Markdown
Contributor

Uploading artifacts from the browser builds for the recently added Github actions.

@tmbrbr

tmbrbr commented Feb 28, 2025

Copy link
Copy Markdown
Contributor Author

At the moment the binaries are being uploaded successfully for all jobs except on the windows runners. In this case the builds are failing due to de-sync of MSCV on the github runner and the clang version used by mozbuild. (See here for a similar issue.

We can't just update the clang version in the image because Firefox downloads its own version. This will hopefully be fixed once the Firefox version is updated (e.g. in #262).

@tmbrbr tmbrbr self-assigned this Feb 28, 2025
@tmbrbr

tmbrbr commented Mar 17, 2025

Copy link
Copy Markdown
Contributor Author

Now that #262 is merged, this should hopefully work now, I'll rebase.

@tmbrbr tmbrbr force-pushed the actions-publish-artifacts branch from 949746a to 8bc6cb6 Compare March 17, 2025 14:27
@tmbrbr tmbrbr marked this pull request as ready for review March 17, 2025 14:28
@tmbrbr tmbrbr requested a review from leeN March 17, 2025 14:28

@leeN leeN left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@tmbrbr

tmbrbr commented Mar 18, 2025

Copy link
Copy Markdown
Contributor Author

Seems to be working after the update!

@tmbrbr tmbrbr merged commit 179c5f1 into main Mar 18, 2025
@tmbrbr tmbrbr deleted the actions-publish-artifacts branch February 9, 2026 13:28
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