Closed Bug 1527011 Opened 7 years ago Closed 7 years ago

artifact build for win10/aarch64 fails

Categories

(Firefox Build System :: General, defect)

defect
Not set
normal

Tracking

(firefox67 fixed)

RESOLVED FIXED
mozilla67
Tracking Status
firefox67 --- fixed

People

(Reporter: egao, Assigned: gbrown)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

A non-artifact build can be executed and completes as expected: example

However if the --artifact value is passed, such as ./mach try fuzzy --query "windows10-aarch64/opt --artifact, the build fails with the following error:

Whoops!
You are trying to pass a shortname for --custom-build-variant-cfg. 
However, I need to know the bits to find the appropriate filename. You can tell me by passing:
    "--bits" or a config filename via "--config" with "32" or "64" in it. 
In either case, these option arguments must come before --custom-build-variant.

Treeherder: https://treeherder.mozilla.org/#/jobs?repo=try&selectedJob=227294692&revision=29e3f3796a4e650d9cebed586f7faf16caa190ef

Taskcluster: https://tools.taskcluster.net/groups/ae_5jTeaR_6_FNjT0Cm3iA/tasks/PaSg0uFVRmyDad9qeKIyOw/runs/0/logs/public%2Flogs%2Flive.log

Flags: needinfo?(nfroyd)

I know a little about artifact builds and can have a look. I notice that there seems to be no win/aarch64 allowance in https://searchfox.org/mozilla-central/rev/00c0d068ece99717bea7475f7dc07e61f7f35984/python/mozbuild/mozbuild/artifacts.py, for instance. Has anyone tried to make this work before? Any known blockers?

(In reply to Geoff Brown [:gbrown] from comment #1)

I know a little about artifact builds and can have a look. I notice that there seems to be no win/aarch64 allowance in https://searchfox.org/mozilla-central/rev/00c0d068ece99717bea7475f7dc07e61f7f35984/python/mozbuild/mozbuild/artifacts.py, for instance. Has anyone tried to make this work before? Any known blockers?

There's no real reason we shouldn't do this; we just have to do all the artifact-alternative verbiage. Bug 1525069 suggests we want to support this; no reason to not do so in automation.

Assignee: nobody → gbrown
Flags: needinfo?(nfroyd)
Comment on attachment 9043407 [details] [diff] [review] support artifact builds on win64-aarch64 Review of attachment 9043407 [details] [diff] [review]: ----------------------------------------------------------------- I can't really vet the mozconfigs, but if it's green for you it's sensible to me! ::: browser/config/mozconfigs/win64-aarch64/artifact @@ +1,3 @@ > +. "$topsrcdir/build/mozconfig.artifact.automation" > + > +# Needed to set SourceRepository in application.ini (used by Talos) This might be true, but it's more that `MOZILLA_OFFICIAL` is the "we're building in Mozilla's automation" flag, and should be true for every build in TC. @@ +9,5 @@ > +. "$topsrcdir/build/win64-aarch64/mozconfig.vs-latest" > +. "$topsrcdir/build/mozconfig.common.override" > + > +. "$topsrcdir/build/mozconfig.artifact" > +ac_add_options --enable-artifact-build-symbols Huh, I wonder why this isn't in `mozconfig.artifact`. Another day.
Attachment #9043407 - Flags: review?(nalexander) → review+

(In reply to Nick Alexander :nalexander [he/him] from comment #6)

::: browser/config/mozconfigs/win64-aarch64/artifact
@@ +1,3 @@

+. "$topsrcdir/build/mozconfig.artifact.automation"
+
+# Needed to set SourceRepository in application.ini (used by Talos)

This might be true, but it's more that MOZILLA_OFFICIAL is the "we're
building in Mozilla's automation" flag, and should be true for every build
in TC.

It's not my wording: copied from (for instance) https://searchfox.org/mozilla-central/rev/cb7faaf6b4ad2528390186f1ce64618dea71031e/browser/config/mozconfigs/win64/artifact#3

I'll keep it, for consistency.

Pushed by gbrown@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/636d2c00234d Support win64-aarch64 artifact builds; r=nalexander
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: