Armen [:armenzg] Reporter
	Description • 14 years ago

These locales won't be shipped for Firefox 4 let's get rid of them.

	Armen [:armenzg] Reporter
	Comment 3 • 14 years ago

Comment on attachment 526993 [details] [diff] [review]
Update verify config changes

http://hg.mozilla.org/build/tools/rev/fb335deb699b

	Armen [:armenzg] Reporter
	Comment 4 • 14 years ago

Comment on attachment 526992 [details] [diff] [review]
Patcher configs

coop or rail, could you please land this for me before generating the MUs?

	Chris Cooper [:coop] (he/him)
	Comment 5 • 14 years ago

Comment on attachment 526992 [details] [diff] [review]
Patcher configs

Checking in moz191-branch-major-update-patcher2.cfg;
/cvsroot/mozilla/tools/patcher-configs/moz191-branch-major-update-patcher2.cfg,v  <--  moz191-branch-major-update-patcher2.cfg
new revision: 1.32; previous revision: 1.31
done
Checking in moz192-branch-major-update-patcher2.cfg;
/cvsroot/mozilla/tools/patcher-configs/moz192-branch-major-update-patcher2.cfg,v  <--  moz192-branch-major-update-patcher2.cfg
new revision: 1.5; previous revision: 1.4
done

	Armen [:armenzg] Reporter
	Comment 6 • 14 years ago

This is not my are of expertise.
Putting it back into the pool.

	bhearsum@mozilla.com (:bhearsum)
	Comment 7 • 14 years ago

(In reply to comment #6)
> This is not my are of expertise.
> Putting it back into the pool.

Based on the patches being checked in, I'm not sure what is left to be done here, can you spell it out?

	Rail Aliiev [:rail] Assignee
	Comment 8 • 14 years ago

If I understand correctly, the bumper will recreate the locale list next time. MU bumper doesn't use old-shipped-locales, IIRC.

	bhearsum@mozilla.com (:bhearsum)
	Comment 9 • 14 years ago

(In reply to comment #8)
> If I understand correctly, the bumper will recreate the locale list next
> time. MU bumper doesn't use old-shipped-locales, IIRC.

Hmm, it looks to me like it does:
perl ../tools/release/update-verify-bump.pl -o linux -p firefox -r Firefox --old-version=3.5.19 --old-app-version=3.5.19 --old-long-version=3.5.19 -v 4.0.1 --app-version=4.0.1 --long-version=4.0.1 -n 1 -a https://aus2.mozilla.org -s stage-old.mozilla.org -c ../tools/release/updates/moz191-firefox-linux-major.cfg -d /pub/mozilla.org/firefox/nightly/3.5.19-candidates/build2/ -l old-shipped-locales --pretty-candidates-dir --major

which is actually the problem, I think. This is actually describing the same issue from bug 384065 which was WONTFIX'ed because of cost/benefit of fixing the existing patcher scripts. Even in the AUS3 world we'll have to deal with the update verify bumper though.

One way to do this would be to have the bump script look at shipped-locales and old-shipped-locales, and only use locales listed in both.

	Rail Aliiev [:rail] Assignee
	Comment 10 • 14 years ago

(In reply to comment #9)
> One way to do this would be to have the bump script look at shipped-locales
> and old-shipped-locales, and only use locales listed in both.

... and use this only for MUs. Usually oldVersion is stable enough to add/remove locales,so the possibility to hit bug 384065 is very low.

	bhearsum@mozilla.com (:bhearsum)
	Comment 11 • 14 years ago

(In reply to comment #10)
> (In reply to comment #9)
> > One way to do this would be to have the bump script look at shipped-locales
> > and old-shipped-locales, and only use locales listed in both.
> 
> ... and use this only for MUs. Usually oldVersion is stable enough to
> add/remove locales,so the possibility to hit bug 384065 is very low.

Is there any reason we *can't* look at both of them for regular minor updates? It seems like there's no downside to doing so.

	Rail Aliiev [:rail] Assignee
	Comment 12 • 14 years ago

Betas may have a lot of add/removes, it's very rare when we drop/add locales for stable releases.

	bhearsum@mozilla.com (:bhearsum)
	Comment 13 • 14 years ago

(In reply to comment #12)
> Betas may have a lot of add/removes, it's very rare when we drop/add locales
> for stable releases.

Yeah, I guess what I'm getting at is that we _can_ use the same code path for minor and major updates, even though it won't make a difference for minor updates, generally. Unless there's some reason it won't work for minor updates, why not use it for both?

	Rail Aliiev [:rail] Assignee
	Comment 14 • 14 years ago

Yeah, we can use the same code. 

Adding something like --all-locales which ignores old-shipped-locales and bumps configs the same way we have now may be an option and can be used for betas.

	Rail Aliiev [:rail] Assignee
	Comment 15 • 13 years ago

Fixed by bug 651481