Closed Bug 454709 Opened 11 years ago Closed 11 years ago
CE Resource Compiling Needs Shunt Application
In order to compile resources within the SECURITY module, several pathnames have to be converted from MingW32 to Windows (i.e., "/c/...." to "c:/....") This means we need a resource compiler shunt tool (arm-wince-res.exe) to do the command line conversion as well as calling the correct SDK resource compiler.
Attachment #337997 - Flags: review?(doug.turner)
Comment on attachment 337997 [details] [diff] [review] Patch to add arm-wince-res.c source to VS8 and VS9 WinCE tools this looks fine, but the patch to wince/tools/Makefile didn't apply. is there some other patch needed?
Attachment #337997 - Flags: review?(doug.turner) → review-
Good observation - this patch depends upon bug 444485 - Make WinMobile Build work with Visual Studio 2008 (VS9). Seems that every bug I added with a depends upon field filled in was actually added to bugzilla with the depends upon field blanked out. Sorry for the confusion - and this attachment should apply cleanly once 444485's attachment is applied.
Depends on: 454485
Attachment #337997 - Flags: review- → review?(doug.turner)
Comment on attachment 337997 [details] [diff] [review] Patch to add arm-wince-res.c source to VS8 and VS9 WinCE tools This patch applies for me, as long as I apply the patch for bug 444485 first. 444485 is the bug for including support for Visual Studio 9, and this bug patch depends upon that previously-added VS9 support. I had originally added a "depends upon" 444485 when creating this bug, but somehow that field got lost. Reset "depends upon" field inside the associated bug, and now this patch can be re-tried. I am sorry that I made extra work for you, doug!
Comment on attachment 337997 [details] [diff] [review] Patch to add arm-wince-res.c source to VS8 and VS9 WinCE tools ted is already reviewing a bunch of this stuff. It looks okay to me; assuming things keep building. why a specific target for building the res tool?
Attachment #337997 - Flags: review?(doug.turner) → review?(ted.mielczarek)
Attachment #337997 - Flags: review?(ted.mielczarek) → review+
the res tool is needed because the path names have to be modified to convert ming to something that the vs tools understand.
changeset: 19522:e0b2a01975da tag: tip user: Doug Turner <email@example.com> date: Mon Sep 22 22:08:46 2008 +0100 summary: Bug 454709 - WinCE Resource Compiling Needs Shunt Application. patch by wolfe. r=dougt. npodb
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Assignee: nobody → wolfe
Component: General → Build Config
Product: Fennec → Core
QA Contact: general → build-config
You need to log in before you can comment on or make changes to this bug.