Closed Bug 1433517 Opened 2 years ago Closed 2 years ago

Fix generate of MOZ_MACBUNDLE_ID

Categories

(Firefox Build System :: General, enhancement)

enhancement
Not set

Tracking

(firefox60 fixed)

RESOLVED FIXED
mozilla60
Tracking Status
firefox60 --- fixed

People

(Reporter: arthur, Assigned: arthur)

Details

(Whiteboard: [tor 21723])

Attachments

(1 file)

The generation of MOZ_MACBUNDLE_ID does not work for Tor Browser without a patch. Here's Tor Browser's patch:
torpat.ch/21723
and here is the original ticket:
https://trac.torproject.org/21723
If your main problem for tor browser was with the space, this was recently fixed in bug 1431477.
(stripping more illegal character is a good thing, though, I'm all for taking a rebased version of your patch.
Thanks, glandium! Here's a rebase for review.
Attachment #8948472 - Flags: review?(mh+mozilla)
Attachment #8948472 - Flags: review?(mh+mozilla) → review+
Keywords: checkin-needed
Pushed by cbrindusan@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a7f88998e959
Prevent illegal characters in MOZ_MACBUNDLE_ID r=glandium
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/a7f88998e959
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
Product: Core → Firefox Build System
Assignee: nobody → arthuredelstein
You need to log in before you can comment on or make changes to this bug.