458088 - add WinCE default configuration settings to configure.in

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[John Wolfe (:wolfe)]

Attachment: v.1 patch

Two configuration settings which should be turned on by default for each WinCE build, so they do not have to appear in a mozconfig file.

ac_add_options --disable-vista-sdk-requirements

and 

ac_add_options --enable-default-toolkit=cairo-windows

Comment 1

[John Wolfe (:wolfe)]

Attachment: v.2 patch - cleaner than before

Cleaner way than v.1 patch to set default configurations for WinCE builds.

Comment 2

[Benjamin Smedberg]

cairo-windows is correct, but I'm not so sure about the vista-sdk-requirements blanket. vista-sdk-requirements covers a lot more ground than just parental-controls, see http://mxr.mozilla.org/mozilla-central/search?string=vista_sdk_requirements

Comment 3

[(not currently active) Ted Mielczarek]

The Windows Mobile builds use their own SDK, so they don't have the Vista definitions (at least this is what Doug tells me). I suggested that we either #ifdef the relevant bits of code on WINCE, or flip the right defines in configure to make it happen. I don't know the actual scope of what needs to be disabled, though.

Comment 6

[(not currently active) Ted Mielczarek]

Comment on attachment 341306 [details] [diff] [review]
v.2 patch - cleaner than before

I'd prefer a patch that doesn't add $_PLATFORM_MOZ_DISABLE_VISTA_SDK_REQUIREMENTS, but it's not that big of a deal.

Comment 8

[John Wolfe (:wolfe)]

Attachment: v.2.1 patch -- updated v.2 patch which applies cleanly

Nothing changed in this patch, except the surrounding context -- this new attachment just applies the changes cleanly to the current configure.in file.

As I understand it, the r+ given to patch v2.0 should still apply.

Comment 9

[John Wolfe (:wolfe)]

Comment on attachment 366964 [details] [diff] [review]
v.2.1 patch -- updated v.2 patch which applies cleanly

carried forward the r+ted from v.2 patch.

Comment 10

[Dão Gottwald [:dao]]

http://hg.mozilla.org/mozilla-central/rev/2492fd7690c8

Comment 11

[Serge Gautherie (:sgautherie)]

Bug 506493 comment 39:
{
Justin Wood (:Callek)      2010-04-04 21:38:23 PDT

I doubt this would work; given order of things, [_PLATFORM_MOZ_DISABLE_VISTA_SDK_REQUIREMENTS] is
AFTER where its used.
}

I noticed too but didn't care.
As Callek seems to care, could you confirm that/how it works?
(My guess would be that the var is missing |"| around itself where used!?)

Comment 12

[Justin Wood (:Callek)]

(In reply to comment #11)
> Bug 506493 comment 39:
> {
> Justin Wood (:Callek)      2010-04-04 21:38:23 PDT
> 
> I doubt this would work; given order of things,
> [_PLATFORM_MOZ_DISABLE_VISTA_SDK_REQUIREMENTS] is
> AFTER where its used.
> }
> 
> I noticed too but didn't care.
> As Callek seems to care, could you confirm that/how it works?
> (My guess would be that the var is missing |"| around itself where used!?)


Actually the m-c order seemed right; so this doesn't belong as a comment here... I'll poke at it in our side's bug.

Comment 13

[Justin Wood (:Callek)]

(In reply to comment #12)
> Actually the m-c order seemed right; so this doesn't belong as a comment
> here... I'll poke at it in our side's bug.

Slight correction; *this* patch was right, but something in m-c seems to have changed the order here... I'm investigating and will comment in our-sides bug and the "problem" bug.

Comment 14

[Justin Wood (:Callek)]

Ok...

Dao/Wolfe, the problem is, the PATCH was right; but it bitrotted by the time it was landed...

This is currently not doing the right thing because of that. Do we want to reopen, use a new bug, or backout the change?

Comment 15

[:Gavin Sharp [email: [e-mail]]]

Reopen if you're going to back out the patch, use a new bug otherwise.

Comment 16

[Serge Gautherie (:sgautherie)]

I'm going to back this out (tomorrow), unless comment 11 is answered.

Comment 17

[Brad Lassey [:blassey] (use needinfo?)]

I'm not sure what you want here? that configure variable isn't even used anymore, this patch is a year and a half old, things have changed.

Comment 18

[Serge Gautherie (:sgautherie)]

(In reply to comment #17)

Ah ... deprecated but not unused/removed:
I filed bug 557958: let's see that one first.

Comment 19

[Justin Wood (:Callek)]

Based on last comment, lets just keep this fixed; and no need to do a "real" backout since it landed so long ago; we can however remove the offending winCE var/setting in trunk if we so desire (in a new bug).