682751 - Remove build/hcc and build/hcpp

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Comment 1

[Matheus Kerschbaum]

Attachment: patch

From a quick look in MXR it appears that Unixware depends on this workaround. Are we ok with dropping support for that platform?

Comment 2

[(not currently active) Ted Mielczarek]

Comment on attachment 556439 [details] [diff] [review]
patch

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

If UnixWare really relies on this then yes, it's fine to drop support. (I'd bet that's probably an ancient workaround anyway.)

::: configure.in
@@ +946,5 @@
>  esac
>  
>  dnl Test breaks icc on OS/2 && MSVC
>  if test "$CC" != "icc" -a -z "$_WIN32_MSVC"; then
>      AC_PROG_CC_C_O

Just remove this whole block. It's not worth testing for compilers that can't support -c -o at all.

Comment 3

[Matheus Kerschbaum]

Attachment: patch for checkin

Comment 4

[Ed Morley [:emorley]]

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

Comment 5

[Matheus Kerschbaum]

Attachment: bustage fix

Maybe one day I'll actually remember to change js/src before burning the tree.

Comment 6

[Ed Morley [:emorley]]

Comment on attachment 556656 [details] [diff] [review]
bustage fix

Yeah well I could have spotted it or built locally before pushing too!

I don't know if you have a build environment set up already, but if you could build locally as a quick sanity check, it will make things quicker when I check patches in. Thanks :-)

Comment 7

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

http://hg.mozilla.org/mozilla-central/rev/080f856c88a7
http://hg.mozilla.org/mozilla-central/rev/b26059bb002e