Closed Bug 796927 Opened 9 years ago Closed 9 years ago

xpcom does not compile on armv6 (sb2) for raspberry pi

Categories

(Core :: XPCOM, defect)

ARM
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla18

People

(Reporter: romaxa, Assigned: romaxa)

Details

Attachments

(2 files)

arm-unknown-linux-gnueabi-g++ -o xptcinvoke_arm.o -c  -fvisibility=hidden -DMOZ_GLUE_IN_PROGRAM -DMOZILLA_INTERNAL_API -DEXPORT_XPTC_API -D_IMPL_NS_COM -I/xpcom/reflect/xptcall/src/md/unix/../.. -I/xpcom/reflect/xptcall/src/md/unix/../../../../xptinfo/src  -I/xpcom/reflect/xptcall/src/md/unix -I. -I../../../../../../dist/include  -I/obj-build-b2g-rsppi/dist/include/nspr -I/obj-build-b2g-rsppi/dist/include/nss      -fPIC -I/usr/include/arm-linux-gnueabihf -I/opt/vc/include -pedantic -Wall -Wpointer-arith -Woverloaded-virtual -Werror=return-type -Wtype-limits -Wempty-body -Wno-ctor-dtor-privacy -Wno-overlength-strings -Wno-invalid-offsetof -Wno-variadic-macros -Wno-long-long -march=armv6zk -mfpu=vfp -mfloat-abi=hard -fno-exceptions -fno-strict-aliasing -ffunction-sections -fdata-sections -fno-exceptions -std=gnu++0x -pthread -pipe  -DNDEBUG -DTRIMMED -gdwarf-2 -O3 -g -mapcs-frame -fno-omit-frame-pointer  -O2  -I/usr/include/arm-linux-gnueabihf -I/opt/vc/include -DMOZILLA_CLIENT -include ../../../../../../mozilla-config.h -MD -MF .deps/xptcinvoke_arm.o.pp  /xpcom/reflect/xptcall/src/md/unix/xptcinvoke_arm.cpp
/xpcom/reflect/xptcall/src/md/unix/xptcinvoke_arm.cpp: In function 'nsresult NS_InvokeByIndex_P(nsISupports*, uint32_t, uint32_t, nsXPTCVariant*)':
/xpcom/reflect/xptcall/src/md/unix/xptcinvoke_arm.cpp:404:10: error: invalid conversion from 'uint32_t {aka unsigned int}' to 'nsresult {aka tag_nsresult}' [-fpermissive]
make[1]: *** [xptcinvoke_arm.o] Error 1
simple cast of result to (nsresult) on return works ok.
Assignee: nobody → romaxa
Status: NEW → ASSIGNED
Attachment #668318 - Flags: review?(joe)
I guess it should be ok
Attachment #668320 - Flags: review?(siarhei.siamashka)
Comment on attachment 668318 [details] [diff] [review]
use nsresult instead of uint32_t

Review of attachment 668318 [details] [diff] [review]:
-----------------------------------------------------------------

I think you uploaded the wrong patch. The name doesn't match the contents.
Attachment #668318 - Flags: review?(joe)
Comment on attachment 668318 [details] [diff] [review]
use nsresult instead of uint32_t

Ah, figured it out, patches are reversed.
Attachment #668318 - Attachment description: gfx add cstring include for memcpy → use nsresult instead of uint32_t
Attachment #668318 - Flags: review?(siarhei.siamashka)
Attachment #668320 - Attachment description: use nsresult instead of uint32_t → gfx add cstring include for memcpy
Attachment #668320 - Flags: review?(siarhei.siamashka) → review+
Comment on attachment 668318 [details] [diff] [review]
use nsresult instead of uint32_t

Review of attachment 668318 [details] [diff] [review]:
-----------------------------------------------------------------

r+ (not sure if/how I can change the "review?" attachment flag)
Comment on attachment 668318 [details] [diff] [review]
use nsresult instead of uint32_t

You definitely can change the review state; it's in the attachment details tab. But I'll do it for you :)
Attachment #668318 - Flags: review?(siarhei.siamashka) → review+
(In reply to Joe Drew (:JOEDREW! \o/) from comment #7)
> You definitely can change the review state; it's in the attachment details
> tab.

I suspect that I'm just a victim of https://bugzilla.mozilla.org/show_bug.cgi?id=294021#c1 or something similar.
Why the devil don't you have editbugs?! I'll fix that.
https://hg.mozilla.org/mozilla-central/rev/0bea829361c3
https://hg.mozilla.org/mozilla-central/rev/13fd1d18cdf4
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla18
You need to log in before you can comment on or make changes to this bug.