Configuration Changes to allow WinCE resource and assembly compiling

RESOLVED FIXED

Status

()

Core
Build Config
RESOLVED FIXED
10 years ago
9 years ago

People

(Reporter: wolfe, Assigned: wolfe)

Tracking

({mobile})

Trunk
ARM
Windows Mobile 6 Professional
mobile
Points:
---
Bug Flags:
blocking-xul-fennec1.0 +
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Comment hidden (empty)
(Assignee)

Comment 1

10 years ago
Created attachment 338090 [details] [diff] [review]
Configure.in changes to allow mozconfig specification of AS and RC

This patch is located in a WINCE-only section of the configure.in file.

This patch puts any mozconfig-specified AS back into place (AS forcibly set to $(CC) on line 940 of configure.in).

This patch also sets RC to "rc.exe" ONLY if no RC has already been set.  This change allows a mozconfig-specified RC, while still leaving the old behavior if RC has not been specified in a mozconfig file.
Attachment #338090 - Flags: review?(doug.turner)
(Assignee)

Comment 2

10 years ago
Darn Firefox browser - I hit a back arrow that somehow got converted into a return.  Sorry for the extra emails.
OS: Windows XP → Windows Mobile 6 Professional
Hardware: PC → PocketPC

Updated

10 years ago
Attachment #338090 - Flags: review?(ted.mielczarek)
Attachment #338090 - Flags: review?(doug.turner)
Attachment #338090 - Flags: review?

Comment 3

10 years ago
Comment on attachment 338090 [details] [diff] [review]
Configure.in changes to allow mozconfig specification of AS and RC

ted should look at this.

I am not sure that this is the best place for this set, but if it isn't ted will know!!

Updated

10 years ago
Assignee: nobody → wolfe
Flags: blocking-fennec1.0+
Target Milestone: --- → Fennec A2
Attachment #338090 - Flags: review?(ted.mielczarek) → review+
Comment on attachment 338090 [details] [diff] [review]
Configure.in changes to allow mozconfig specification of AS and RC

-    RC=rc.exe
+    if test -n "$RC"; then 
+        RC=rc.exe  
+    fi

You want "test -z" here.
(Assignee)

Comment 5

10 years ago
Created attachment 338660 [details] [diff] [review]
Configure.in changes to allow mozconfig specification of AS and RC - using -z instead of -n
[Checkin: Comment 7]

Ted and Doug - just replaced the '-n' test flag with '-z', as per ted's comment on the previous attached patch file.
Attachment #338090 - Attachment is obsolete: true
Attachment #338660 - Flags: review?(ted.mielczarek)
Attachment #338090 - Flags: review?
Attachment #338660 - Flags: review?(ted.mielczarek) → review+
(Assignee)

Comment 6

10 years ago
Added mobile and checkin-needed flags to this bug.
Keywords: checkin-needed, mobile

Updated

10 years ago
Blocks: 432792
Comment on attachment 338660 [details] [diff] [review]
Configure.in changes to allow mozconfig specification of AS and RC - using -z instead of -n
[Checkin: Comment 7]

http://hg.mozilla.org/mozilla-central/rev/9ee685755478
Attachment #338660 - Attachment description: Configure.in changes to allow mozconfig specification of AS and RC - using -z instead of -n → Configure.in changes to allow mozconfig specification of AS and RC - using -z instead of -n [Checkin: Comment 7]
(In reply to comment #7)
> (From update of attachment 338660 [details] [diff] [review])

Note that (this) patch should have had Linux EOL, not Windows EOL.
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Flags: in-testsuite-
Keywords: checkin-needed
Resolution: --- → FIXED
Component: General → Build Config
Product: Fennec → Core
QA Contact: general → build-config
Target Milestone: Fennec A2 → ---
You need to log in before you can comment on or make changes to this bug.