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)
Tracking
()
RESOLVED
FIXED
mozilla18
People
(Reporter: romaxa, Assigned: romaxa)
Details
Attachments
(2 files)
1.06 KB,
patch
|
joe
:
review+
|
Details | Diff | Splinter Review |
681 bytes,
patch
|
joe
:
review+
|
Details | Diff | Splinter Review |
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
Assignee | ||
Comment 1•9 years ago
|
||
simple cast of result to (nsresult) on return works ok.
Assignee | ||
Comment 2•9 years ago
|
||
Assignee | ||
Comment 3•9 years ago
|
||
I guess it should be ok
Attachment #668320 -
Flags: review?(siarhei.siamashka)
Comment 4•9 years ago
|
||
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 5•9 years ago
|
||
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)
Updated•9 years ago
|
Attachment #668320 -
Attachment description: use nsresult instead of uint32_t → gfx add cstring include for memcpy
Attachment #668320 -
Flags: review?(siarhei.siamashka) → review+
Comment 6•9 years ago
|
||
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 7•9 years ago
|
||
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+
Comment 8•9 years ago
|
||
(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.
Comment 9•9 years ago
|
||
Why the devil don't you have editbugs?! I'll fix that.
Assignee | ||
Comment 10•9 years ago
|
||
remote: https://hg.mozilla.org/integration/mozilla-inbound/rev/0bea829361c3 remote: https://hg.mozilla.org/integration/mozilla-inbound/rev/13fd1d18cdf4
Comment 11•9 years ago
|
||
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.
Description
•