Closed Bug 690560 Opened 11 years ago Closed 11 years ago

make repack-l10n-all doesn't work correctly on Mac universal builds.


(Calendar :: Build Config, defect)

Not set


(Not tracked)



(Reporter: standard8, Assigned: standard8)




(1 file)

The repack-l10n-all target actually tries to repack in the i386 directory rather than the universal one. We need to find a way to fix that.
Blocks: 673089
Attached patch Possible fixSplinter Review
This seems to fix the repack-l10n-all target, I'm not yet convinced it doesn't break normal l10n repacks.
Comment on attachment 564677 [details] [diff] [review]
Possible fix

Ok, this works for the release builds (I did 1.0rc1 Mac with this patch), and I'm pretty sure it'll work for L10n repacks as I've changed all the targets in that file and so it'll all just switch to the universal directory if it is a universal build.
Attachment #564677 - Flags: review?(philipp)
Comment on attachment 564677 [details] [diff] [review]
Possible fix

Looks good, r=philipp
Attachment #564677 - Flags: review?(philipp) → review+
Assignee: nobody → mbanner
Checked in:

Leaving open, as I want to remember to check the nightly l10n builds once they cycle today.
It turns out that I had broken l10n builds, I've pushed a bustage fix:
Depends on: 696122
You need to log in before you can comment on or make changes to this bug.