Closed Bug 1811416 Opened 2 years ago Closed 2 years ago

Fail early if a localized build is attempted with artifact builds enabled

Categories

(Firefox Build System :: General, enhancement)

enhancement

Tracking

(firefox112 fixed)

RESOLVED FIXED
112 Branch
Tracking Status
firefox112 --- fixed

People

(Reporter: ahochheiden, Assigned: ahochheiden)

References

Details

Attachments

(1 file)

The docs for localized builds mention that artifact builds are not supported, but if you miss that note, you will eventually hit this error:

 0:04.24 make[4]: *** No rule to make target `l10n'.  Stop.
 0:04.24 make[3]: *** [l10n-ar] Error 2
 0:04.24 make[2]: *** [l10n-ar] Error 2
 0:04.24 make[1]: *** [langpack-ar] Error 2
 0:04.24 make: *** [langpack-ar] Error 2

Which does not make it obvious that the problem is artifact builds being enabled. We should fail early and inform the user of the problem.

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:ahochheiden, could you have a look please?
If you still have some work to do, you can add an action "Plan Changes" in Phabricator.
For more information, please visit auto_nag documentation.

Flags: needinfo?(mh+mozilla)
Flags: needinfo?(ahochheiden)

I'll land it after the soft code freeze is lifted.

Flags: needinfo?(mh+mozilla)
Flags: needinfo?(ahochheiden)
Pushed by ahochheiden@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9c7577bdd6e3
Fail early if a Localized Build is attempted with Artifact Builds enabled r=firefox-build-system-reviewers,glandium
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 112 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: