583039 - Add a build support for MeegoTouch

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Steffen Imhof]

Attachment: Patch to add --enable-meegotouch switch

For upcoming Meego devices there is a new graphic toolkit called MeegoTouch (see http://meego.gitorious.org/meegotouch/)

This library is built on top of Qt so the attached patch adds a new --enable-meegotouch switch which adds the MeegoTouch libraries and include paths to the Qt related variables.

The MOZ_ENABLE_MEEGOTOUCH #define can then be used to mark the MeegoTouch specific code.

Comment 1

[(not currently active) Ted Mielczarek]

I'm not a fan of adding new configure options. Is this something that can be detected based on the existing options, like --with-maemo-version ?

Comment 2

[Steffen Imhof]

In theory it is still possible to build a pure Qt version of xulrunner even if you are on Maemo6/MeeGo.

So I think these are two options should really be separate.

Comment 3

[(not currently active) Ted Mielczarek]

I am trying to reduce the number of configure options, because they are an attractive nuisance for people building Mozilla, and they cause headaches when they don't work as expected. If the only reason to ever use this is on Maemo6/MeeGo, then it should just be the default there, and not an option. In theory it's possible to do a lot of things. If we're going to ship our builds with this enabled, it should be on by default.

Comment 4

[Steffen Imhof]

Yeah, that makes sense (it's not like I did not get tangled up in the various options before :-) ).

So would you be ok with a patch that enables it per default on Maemo6/Meego and and still having an option to disable it explicitly?

Comment 5

[Doug Turner (:dougt)]

Comment on attachment 461301 [details]
Patch to add --enable-meegotouch switch

how about just using MAEMO 6 for now?

Comment 6

[(not currently active) Ted Mielczarek]

Yeah, --with-maemo-version=6 should just turn this on.

Comment 7

[Steffen Imhof]

Ok, I still think that these are not logically connected, but I see the point about keeping things simple. I'll update the patch soon.

But it is ok to keep an (extra) MOZ_ENABLE_MEEGOTOUCH to guard the MeegoTouch specific code, right?

Comment 8

[Steffen Imhof]

Attachment: New patch that couples Maemo version 6 and the usage of MeegoTouch

Now the configure script enables MeegoTouch whenever the Maemo version is greater than 5.

Comment 9

[Steffen Imhof]

Attachment: MeegoTouch/Maemo6 coupled patch with MOZ_ENABLE_MEEGOTOUCH

Sorry, the last version of the patch was missing MOZ_ENABLE_MEEGOTOUCH which is used in dependent patches.

Comment 10

[Steffen Imhof]

Attachment: Updated patch with a new name for the #define

I heard that Jeremias and Doug agreed in a private chat on using MOZ_PLATFORM_MEEGO instead of USE_MEEGOTOUCH, so here it is.
(though I still think of MeegoTouch as a toolkit not as a platform)

Comment 11

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

Comment on attachment 462095 [details] [diff] [review]
Updated patch with a new name for the #define

As long as maemo > 5 requiring meegotouchcore is ok this is fine.  r=me

Comment 12

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

Comment on attachment 462095 [details] [diff] [review]
Updated patch with a new name for the #define

>+     AC_DEFINE(MOZ_PLATFORM_MEEGO)
MOZ_ENABLE_MEEGOTOUCH?

Comment 13

[(not currently active) Ted Mielczarek]

No, I think he wants MOZ_PLATFORM_MEEGO to mirror MOZ_PLATFORM_MAEMO.

Comment 14

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

(In reply to comment #13)
> No, I think he wants MOZ_PLATFORM_MEEGO to mirror MOZ_PLATFORM_MAEMO.

Ah, ok.

Comment 15

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

Comment on attachment 462095 [details] [diff] [review]
Updated patch with a new name for the #define

If MOZ_PLATFORM_MEEGO is intentional then that should replace MOZ_ENABLE_MEEGOTOUCH.  I don't really care what it's named as long as the makefile define and the c define are consistent.

Comment 16

[Doug Turner (:dougt)]

http://hg.mozilla.org/mozilla-central/rev/d87b40f23fa7