Remove support for Irix from our build system

RESOLVED FIXED in mozilla9

Status

()

Core
Build Config
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: Matheus Kerschbaum, Assigned: Matheus Kerschbaum)

Tracking

Trunk
mozilla9
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: fixed-in-bs)

Attachments

(1 attachment, 1 obsolete attachment)

18.70 KB, patch
Matheus Kerschbaum
: review+
Details | Diff | Splinter Review
Comment hidden (empty)
(Assignee)

Comment 1

6 years ago
Created attachment 554750 [details] [diff] [review]
patch
Attachment #554750 - Flags: review?(ted.mielczarek)
(Assignee)

Updated

6 years ago
Blocks: 680793
Is there no Gecko being run on Irix, then?  Or will it still be possible to try to build on Irix, just without any of the special-casing?
(Assignee)

Comment 3

6 years ago
It won't be possible to build Firefox under Irix/mips after this patch lands, but my understanding is that the platform is dead so there's no reason to keep this code around.
Comment on attachment 554750 [details] [diff] [review]
patch

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

::: build/hcpp
@@ -96,1 @@
>  			;;

I think this whole hcpp wrapper can probably die. AFAICT from configure, it's only for compilers that don't support "-c -o" at the same time, which is pretty old.

::: xpcom/reflect/xptcall/src/md/unix/Makefile.in
@@ -291,5 @@
> -#
> -ifeq ($(OS_ARCH),IRIX)
> -ifneq ($(basename $(OS_RELEASE)),5)
> -CPPSRCS		:= xptcinvoke_irix.cpp xptcstubs_irix.cpp
> -ASFILES		:= xptcinvoke_asm_irix.s xptcstubs_asm_irix.s

I'd guess you should hg rm these files while you're here.
Attachment #554750 - Flags: review?(ted.mielczarek) → review+
(Assignee)

Comment 5

6 years ago
Created attachment 555881 [details] [diff] [review]
patch for checkin

Updated to tip.

(In reply to Ted Mielczarek [:ted, :luser] from comment #4)
> I think this whole hcpp wrapper can probably die. AFAICT from configure,
> it's only for compilers that don't support "-c -o" at the same time, which
> is pretty old.

I'll remove hcpp in a follow-up bug.

> I'd guess you should hg rm these files while you're here.

Bug 680793.
Attachment #554750 - Attachment is obsolete: true
Attachment #555881 - Flags: review+
(Assignee)

Updated

6 years ago
Keywords: checkin-needed

Comment 6

6 years ago
http://hg.mozilla.org/projects/build-system/rev/a80fe3a6586f

Be gone crufty code! :-)
Keywords: checkin-needed
Whiteboard: fixed-in-bs

Updated

6 years ago
Target Milestone: --- → mozilla9
http://hg.mozilla.org/mozilla-central/rev/a80fe3a6586f
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
(Assignee)

Updated

6 years ago
Blocks: 682751

Updated

5 years ago
Blocks: 807488
You need to log in before you can comment on or make changes to this bug.