Closed Bug 1305960 Opened 8 years ago Closed 5 years ago

Don't link to stdc++compat for C-only libraries

Categories

(Firefox Build System :: General, defect)

Product:
Firefox Build System
Component:
General
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(firefox52 wontfix)

Status:
RESOLVED DUPLICATE of bug 1377971
Tracking Flags:
Tracking Status
firefox52 --- wontfix

People

(Reporter: ted, Unassigned)

References

Details

bug 1256642 adds some logic to the build system to set `LIB_IS_C_ONLY` in the recursive make backend when linking libraries consisting solely of C sources. Unfortunately our `Binary` template which is the base for all linkable things in moz.build forces linking everything against `stdc++compat` when --enable-stdcxx-compat is used (as in automation): https://dxr.mozilla.org/mozilla-central/rev/d6ee82b9a74155b6bfd544166f036fc572ae8c56/build/templates.mozbuild#12 We'd need to do something smarter here to make the patch in bug 1256642 do the right thing in this scenario.
Blocks: 1272626
Too late for firefox 52, mass-wontfix.
status-firefox52: affected → wontfix
Product: Core → Firefox Build System

I think we fixed this with bug 1377971.

Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.