1255813 - Remove build system support for Solaris, HPUX and AIX

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Comment 1

[Mike Hommey [:glandium]]

Attachment: MozReview Request: Bug 1255813 - Remove build system support for Solaris, HPUX and AIX

Review commit: https://reviewboard.mozilla.org/r/39515/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/39515/

Comment 2

[(not currently active) Ted Mielczarek]

Comment on attachment 8729574 [details]
MozReview Request: Bug 1255813 - Remove build system support for Solaris, HPUX and AIX

https://reviewboard.mozilla.org/r/39515/#review36203

I'm not going to miss any of this junk.

::: build/autoconf/icu.m4:102
(Diff revision 1)
>                  MOZ_ICU_DBG_SUFFIX=
>                  if test -n "$MOZ_DEBUG" -a -z "$MOZ_NO_DEBUG_RTL"; then
>                      MOZ_ICU_DBG_SUFFIX=d
>                  fi
>                  ;;
> -            Darwin|Linux|DragonFly|FreeBSD|NetBSD|OpenBSD|GNU/kFreeBSD|SunOS|Android)
> +            Darwin|Linux|DragonFly|FreeBSD|NetBSD|OpenBSD|GNU/kFreeBSD|Android)

Can you leave this hunk out just to avoid the conflict with my in-flight ICU patches? Most of this file is going away with them anyway.

::: old-configure.in
(Diff revision 1)
>      if test "$LIBRUNPATH"; then
>          DSO_LDOPTS="-R$LIBRUNPATH $DSO_LDOPTS"
>      fi
>      ;;
>  
> -*-solaris*)

I can't say I care *too* much here, but it feels like we could support the Solaris-with-GCC case without too much effort, as most of the junk you're removing is SUNpro support.

Then again, maybe it'd be easier to just add it back in when everything else is in configure.py.

Comment 3

[Mike Hommey [:glandium]]

(In reply to Ted Mielczarek [:ted.mielczarek] from comment #2)
> Then again, maybe it'd be easier to just add it back in when everything else
> is in configure.py.

That was my thought.

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/a960fa1bfffd

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/a960fa1bfffd

Comment 6

[Jan Beich]

Martin, would this affect Solaris-like support in PkgSrc? Which compiler it's using by default?

Comment 7

[Konstantin Andreev]

Just encountered that Solaris support was removed from Mozilla about year ago.

This is ... so sudden and grievous.

How and why did this happen ? Was there any public discussion ?

Comment 8

[Mike Hommey [:glandium]]

(In reply to Konstantin Andreev from comment #7)
> Just encountered that Solaris support was removed from Mozilla about year
> ago.
> 
> This is ... so sudden and grievous.
> 
> How and why did this happen ? Was there any public discussion ?

https://groups.google.com/d/msg/mozilla.dev.platform/9EI8D2jNiEs/Jr0pi4DgBgAJ

Comment 9

[Konstantin Andreev]

(In reply to Konstantin Andreev from comment #7)
> Just encountered that Solaris support was removed from Mozilla about year ago.
> How and why did this happen ? Was there any public discussion ?

(In reply to Mike Hommey [:glandium] from comment #8)
> https://groups.google.com/d/msg/mozilla.dev.platform/9EI8D2jNiEs/Jr0pi4DgBgAJ

Solaris has gone with autoconf. Point taken.

I looked further into the mozilla.dev.platform to clarify how the autoconf was decommissioned, but no luck.

Was there any public discussion about autoconf decommissioning?

Comment 10

[Nathan Froyd [:froydnj]]

(In reply to Konstantin Andreev from comment #9)
> I looked further into the mozilla.dev.platform to clarify how the autoconf
> was decommissioned, but no luck.
> 
> Was there any public discussion about autoconf decommissioning?

Yes, four or five years ago now.  I don't have a link to a formal "we are getting rid of autoconf", but searching for terms like `moz.build` in mozilla.dev.platform may yield positive results.