Closed Bug 866487 Opened 7 years ago Closed 7 years ago

MOZ_UPDATER always set to true in AppConstants.java

Categories

(Firefox for Android :: General, defect)

ARM
Android
defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 23

People

(Reporter: daichen314, Assigned: daichen314)

References

Details

Attachments

(1 file)

User Agent: Mozilla/5.0 (Windows NT 5.1) AppleWebKit/537.20 (KHTML, like Gecko) Chrome/25.0.1337.0 Safari/537.20

Steps to reproduce:

set MOZ_UPDATER= in mobile/android/branding/unofficial/configure.sh


Actual results:

MOZ_UPDATER set to true in AppConstants.java


Expected results:

MOZ_UPDATER set to false in AppConstants.java

it should add ifdef XXX in android/base/Makefile.in before "-DMOZ_UPDATER=$(MOZ_UPDATER)"
Hi Chen, since you already found the cause of the issue, would you like to contribute a patch to fix it?
Blocks: 786380
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Windows XP → Android
Hardware: x86 → ARM
I made a patch for android
Attachment #744100 - Attachment is patch: true
Attachment #744100 - Flags: review?(snorp)
Attachment #744100 - Flags: review?(snorp) → review+
Assignee: nobody → daichen314
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/4c4810ef2092
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 23
You need to log in before you can comment on or make changes to this bug.