Closed Bug 600957 Opened 9 years ago Closed 9 years ago
Have a way to provide mobile-browser-specific resources
Some Android code (notifications, for example) requires images to be compiled into resources. We need a way to control that with easier customization.
Added a new MOZ_ANDROID_RESOURCES_DIRECTORY variable to mobile confvars.sh. It will be checked and used in the makefiles.
Assignee: nobody → alexp
Status: NEW → ASSIGNED
Alert service implementation for Android will be looking for MOZ_ANDROID_DRAWABLES variable, which could be provided in mobile/confvars.sh.
Why does this need to be a configure variable? From your usage in bug 601050, it looks like you could just do: ifeq (Android,$(OS_TARGET)) ANDROID_DRAWABLES = ... endif in that Makefile. I don't see the need to put it in configure.
(In reply to comment #4) > I don't see the need to put it in configure. The idea is to make this variable available in the platform code. Here's where it's checked and used: http://mxr.mozilla.org/mozilla-central/source/embedding/android/Makefile.in#167 The variable simply defined in m-b/confvars.sh is not visible in that Makefile unless it's propagated through the configure. Well, I might be missing something here, and would be glad to know if this could be done some other way.
That code is all getting heavily mangled in bug 567873 anyway.
(In reply to comment #6) > That code is all getting heavily mangled in bug 567873 anyway. Yes, though I believe it's supposed to use the same approach for this bit. The app should be able somehow to define a list of drawable resources, which the packaging Makefile could pick up.
Hm. This is why we have packaging manifests, isn't it? mwu: is there a better way to do this? This just feels wrong.
(In reply to comment #8) > Hm. This is why we have packaging manifests, isn't it? > > mwu: is there a better way to do this? This just feels wrong. Sure. We can take the approach from https://bugzilla.mozilla.org/show_bug.cgi?id=526333 and add a platform specific manifest.
Yeah, let's do that.
Comment on attachment 480287 [details] [diff] [review] Config changes for a new variable Going to r- this for now, then.
Attachment #480287 - Flags: review?(ted.mielczarek) → review-
Comment on attachment 490229 [details] [diff] [review] patch I haven't be thrilled by this approach. There is a weird, undocumented connection between the platform installer and the app installer. I guess this will work until something better comes along.
Attachment #490229 - Flags: review?(mark.finkle) → review+
Comment on attachment 490228 [details] [diff] [review] patch This is terrible but I don't have time right now to make something better.
Attachment #490228 - Flags: review?(mwu) → review+
pushed http://hg.mozilla.org/mobile-browser/rev/4733aec2fa63 and http://hg.mozilla.org/mozilla-central/rev/f03f50294d9c
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.