Closed Bug 759115 Opened 8 years ago Closed 8 years ago

Bump minimum Android SDK API level we require to 14

Categories

(Firefox Build System :: General, defect)

All
Android
defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
mozilla16

People

(Reporter: glandium, Assigned: glandium)

References

Details

Attachments

(1 file, 2 obsolete files)

When building with SDK 13, which is currently allowed, one gets the following build failure:

res/values-xlarge/themes.xml:17: error: Error: No resource found that matches the given name: attr 'android:actionBarDivider'.
res/values-sw600dp/themes.xml:17: error: Error: No resource found that matches the given name: attr 'android:actionBarDivider'.
make[6]: *** [R.java] Error 1
Assignee: nobody → mh+mozilla
Attachment #627704 - Flags: review?(ted.mielczarek)
Comment on attachment 627704 [details] [diff] [review]
Bump minimum Android SDK API level we require to 14

Review of attachment 627704 [details] [diff] [review]:
-----------------------------------------------------------------

This doesn't change our runtime compat requirements, just the version of the SDK we require, right?
Attachment #627704 - Flags: review?(ted.mielczarek) → review+
(In reply to Ted Mielczarek [:ted] from comment #2)
> This doesn't change our runtime compat requirements, just the version of the
> SDK we require, right?

Indeed.
https://hg.mozilla.org/mozilla-central/rev/d8e75ef4e5ad
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla15
Backed out, we use android-13 for android XUL (and it's fine).
https://hg.mozilla.org/mozilla-central/rev/65fa5cb6f79c
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Mostly, this moves stuff around, and just adds a check on the target application to set the minimum android sdk api level accordingly.
https://tbpl.mozilla.org/?noignore=1&tree=Try&rev=7c9f4fa9c5a1
Attachment #629715 - Flags: review?(ted.mielczarek)
Attachment #627704 - Attachment is obsolete: true
Attachment #629715 - Attachment is obsolete: true
Attachment #629715 - Flags: review?(ted.mielczarek)
Attachment #627704 - Flags: review+
Attachment #629733 - Flags: review?(ted.mielczarek) → review+
https://hg.mozilla.org/mozilla-central/rev/a430a3d30163
Status: REOPENED → RESOLVED
Closed: 8 years ago8 years ago
Resolution: --- → FIXED
Typo?

diff --git a/js/src/aclocal.m4 b/js/src/aclocal.m4
--- a/js/src/aclocal.m4
+++ b/js/src/aclocal.m4
@@ -17,6 +17,6 @@ builtin(include, build/autoconf/frameptr
 builtin(include, build/autoconf/compiler-opts.m4)dnl
 builtin(include, build/autoconf/expandlibs.m4)dnl
 builtin(include, build/autoconf/arch.m4)dnl
-builtin(include, build/autoconf/android.m4)dn
+builtin(include, build/autoconf/android.m4)dnl
 
 MOZ_PROG_CHECKMSYS()
Depends on: 765808
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.