Open Bug 1396177 Opened 6 years ago Updated 11 months ago

Fix the beetmover-repackage regression caused by bug 1390461


(Firefox Build System :: General, defect)



(firefox57 affected)

Tracking Status
firefox57 --- affected


(Reporter: zbraniecki, Unassigned)



(3 files)

As part of bug 1390461 we extended l10n-check test to clobber the things it tested at the end.

It seems that in result of that, we started clobbering the langpack that is needed later for the beetmover-repackage (see bug 1390461 comment 49 for details).
Assignee: nobody → gandalf
This is a trivial fix that reverts the removal of the generated langpack.

Aki, is it possible for you to check if this will fix the beetmover-repackage jobs?

I can't see any other things that would cause it, but I'm also confused because l10n-check was meant to only generate x-test langpacks, so it should also clean up only x-test langpacks. On the other hand, while going through a plethora of platform issues with that patch I'm pretty sure I saw an attempt in l10n-check to test `` which makes it more probable that this patch will fix things, despite it not being supposed to cause the breakage.

Anyhoo, if it does fix things, I'd like to land it ASAP, as I don't believe we need any deeper review process, and we can discuss further improvements to the test next week.

Also, NI on Pike to make put it on his radar.
Flags: needinfo?(l10n)
Flags: needinfo?(aki)
Comment on attachment 8903873 [details]
Bug 1396177 - Do not clobber the langpack in l10n-check.

I think this could work. Might take me a few hours to test... we may want to just try it.
Attachment #8903873 - Flags: review?(aki) → review+
Pushed by
Do not clobber the langpack in l10n-check. r=aki is a linux nightly graph off a test push on the `date` project branch.
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
(In reply to Aki Sasaki [:aki] from comment #5)
> is a linux
> nightly graph off a test push on the `date` project branch.

It seems that the two jobs that were failing got completed here. Can you confirm that this is fixed now?
Looks like this fixes the problem. Thanks!
Flags: needinfo?(aki)
Thank you!

Pike - do you want me to file a follow-up to figure out how to remove the langpack in clobber without removing the langpack that the beetmover needs, or are you ok with the current status?

This means that we're uploading the web-extension variant of the langpack, and inspecting the target.langpack.xpi from the latest nightlies confirms that.

We'll need a real fix. MOZ_SIMPLE_PACKAGE_NAME= might need to move around a bit. Gonna try that.
Flags: needinfo?(l10n)
Resolution: FIXED → ---
(In reply to Axel Hecht [:Pike] from comment #11)
> I pushed something completely untested to
> jobs?repo=try&revision=1e29dab9647b9de9b624626425b8bae304aa0156

That broke, but looks green.
Trying in . BM-R is what broke last time. I backed out Zibi's patch and landed yours on date-branch; hopefully this works!
Comment on attachment 8904285 [details]
bug 1396177, unset MOZ_SIMPLE_PACKAGE_NAME for all of l10n-check,

Looks like this works!
Attachment #8904285 - Flags: review?(aki) → review+
Comment on attachment 8904284 [details]
Backed out changeset b01a7e57425b (bug 1396177) as we're uploading the wrong file to ftp
Attachment #8904284 - Flags: review+
Pushed by
Backed out changeset b01a7e57425b as we're uploading the wrong file to ftp r=aki
unset MOZ_SIMPLE_PACKAGE_NAME for all of l10n-check, r=aki
Closed: 6 years ago6 years ago
Resolution: --- → FIXED
Backed out for breaking Beta release automation per bug 1385227.
Resolution: FIXED → ---
Target Milestone: mozilla57 → ---
Product: Core → Firefox Build System
Assignee: gandalf → nobody
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.