Closed Bug 397308 Opened 18 years ago Closed 16 years ago

js-ctypes fails on Linux x86-64 with gcc

Categories

(Core :: js-ctypes, defect)

x86
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: asrail, Assigned: asrail)

Details

Attachments

(1 file, 1 obsolete file)

gcc -o sysv.o -c /home/asrail/mozbuilds/mozilla/extensions/js-ctypes/js-ctypes/src/linux/sysv.s /home/asrail/mozbuilds/mozilla/extensions/js-ctypes/js-ctypes/src/linux/sysv.s: Assembler messages: /home/asrail/mozbuilds/mozilla/extensions/js-ctypes/js-ctypes/src/linux/sysv.s:42: Error: suffix or operands invalid for `push' /home/asrail/mozbuilds/mozilla/extensions/js-ctypes/js-ctypes/src/linux/sysv.s:53: Error: suffix or operands invalid for `push' /home/asrail/mozbuilds/mozilla/extensions/js-ctypes/js-ctypes/src/linux/sysv.s:54: Error: suffix or operands invalid for `push' /home/asrail/mozbuilds/mozilla/extensions/js-ctypes/js-ctypes/src/linux/sysv.s:127: Error: suffix or operands invalid for `pop' I've seen the ifndef on the start of the file, but I've seen that it won't help that much.
I.e., the assembler won't process the instruction... since no part of the file should be touched while on x86-64, the file shouldn't be processed anyway.
Attached patch Proposed patch (obsolete) — Splinter Review
Assignee: nobody → asrail
Status: NEW → ASSIGNED
Attachment #282064 - Flags: review?
Comment on attachment 282064 [details] [diff] [review] Proposed patch Canceling review for now, since I'm not sure it will work on a 64 bits system, anyway.
Attachment #282064 - Attachment is obsolete: true
Attachment #282064 - Flags: review?
Attached patch WIPSplinter Review
Product: Other Applications → Core
Version: Trunk → unspecified
Fixed by landing in mozilla-central.
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: