xforms and schemaval need to be static for Minimo

RESOLVED FIXED

Status

Core Graveyard
XForms
RESOLVED FIXED
12 years ago
10 months ago

People

(Reporter: aaronr, Assigned: aaronr)

Tracking

({fixed1.8.0.2, fixed1.8.1})

Trunk
x86
All
fixed1.8.0.2, fixed1.8.1

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

2.34 KB, patch
Doron Rosenberg (IBM)
: review+
smaug
: review+
Details | Diff | Splinter Review
(Assignee)

Description

12 years ago
We need to make sure that XForms and Schemavalidation are static for Minimo builds.  dougt says that Minimo probably won't support components aside from possibly components written in JS or only using xpcom glue interfaces.  xpcom is not available as a .dll in Minimo and nspr and JS soon won't be, either.
(Assignee)

Comment 1

12 years ago
Created attachment 204732 [details] [diff] [review]
patch

forcing xforms to be static.  Had to include all the MOZ_XUL_APP stuff under an ifndef MINIMO test.  Otherwise rules in rules.mk will force components (like XForms and schemavalidation) to be shared libraries (.dlls) if they xul apps.
Attachment #204732 - Flags: review?(doronr)
(Assignee)

Updated

12 years ago
Attachment #204732 - Flags: review?(smaug)

Comment 2

12 years ago
Comment on attachment 204732 [details] [diff] [review]
patch

I'm not too familiar with Minimo, but if this makes things to work r=me :)
Attachment #204732 - Flags: review?(smaug) → review+

Updated

12 years ago
Attachment #204732 - Flags: review?(doronr) → review+

Comment 3

12 years ago
checked into trunk
Whiteboard: xf-to-branch
(Assignee)

Comment 4

11 years ago
checked into MOZILLA_1_8_BRANCH via bug 323691.  Leaving open for now until it gets into 1.8.0
Status: NEW → ASSIGNED

Updated

11 years ago
Whiteboard: xf-to-branch

Updated

11 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
Keywords: fixed1.8.0.2
Resolution: --- → FIXED
(Assignee)

Comment 5

11 years ago
verified fixed in MOZILLA_1_8_BRANCH
Keywords: fixed1.8.1
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.