The default bug view has changed. See this FAQ.

Remove build/hcc and build/hcpp

RESOLVED FIXED in mozilla9

Status

()

Core
Build Config
RESOLVED FIXED
6 years ago
6 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

(2 attachments, 1 obsolete attachment)

Comment hidden (empty)
(Assignee)

Comment 1

6 years ago
Created attachment 556439 [details] [diff] [review]
patch

From a quick look in MXR it appears that Unixware depends on this workaround. Are we ok with dropping support for that platform?
Attachment #556439 - Flags: review?(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.
Attachment #556439 - Flags: review?(ted.mielczarek) → review+
(Assignee)

Updated

6 years ago
Blocks: 682920
(Assignee)

Comment 3

6 years ago
Created attachment 556630 [details] [diff] [review]
patch for checkin
Attachment #556439 - Attachment is obsolete: true
Attachment #556630 - Flags: review+
Attachment #556630 - Flags: checkin?(bmo)
http://hg.mozilla.org/projects/build-system/rev/080f856c88a7
Whiteboard: fixed-in-bs
Attachment #556630 - Flags: checkin?(bmo) → checkin+
Target Milestone: --- → mozilla9
(Assignee)

Comment 5

6 years ago
Created attachment 556656 [details] [diff] [review]
bustage fix

Maybe one day I'll actually remember to change js/src before burning the tree.
Attachment #556656 - Flags: checkin?(bmo)
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 :-)
Attachment #556656 - Flags: checkin?(bmo) → checkin+
http://hg.mozilla.org/mozilla-central/rev/080f856c88a7
http://hg.mozilla.org/mozilla-central/rev/b26059bb002e
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.