680792 - Remove support for Irix from our build system

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Comment 1

[Matheus Kerschbaum]

Attachment: patch

Comment 2

[Boris Zbarsky [:bzbarsky]]

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?

Comment 3

[Matheus Kerschbaum]

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 4

[(not currently active) Ted Mielczarek]

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.

Comment 5

[Matheus Kerschbaum]

Attachment: 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.

Comment 6

[Ed Morley [:emorley]]

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

Be gone crufty code! :-)

Comment 7

[Kyle Huey (Exited; not receiving bugmail, old account, do not use)]

http://hg.mozilla.org/mozilla-central/rev/a80fe3a6586f