| Summary: | mozconfig.eclass changes -Os to -O2 | ||
|---|---|---|---|
| Product: | Gentoo Linux | Reporter: | Rafael Ávila de Espíndola <rafael.espindola> |
| Component: | New packages | Assignee: | Mozilla Gentoo Team <mozilla> |
| Status: | RESOLVED INVALID | ||
| Severity: | normal | CC: | jfarrell, simon.strandman |
| Priority: | High | ||
| Version: | 2005.0 | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Package list: | Runtime testing required: | --- | |
*** Bug 107584 has been marked as a duplicate of this bug. *** *** Bug 67670 has been marked as a duplicate of this bug. *** Now using mozcoreconf and mozconfig-2 now which fall back to mozilla defaults <- Per Anarchy (In reply to comment #3) > Now using mozcoreconf and mozconfig-2 now which fall back to mozilla defaults > <- Per Anarchy > No, it *doesn't* fall back to Mozilla's defaults. The official releases are compiled with "-Os". Ebuild authors need to stop override user-specified parameters, otherwise the utility of portage is reduced. |
mozconfig.eclass tries to fix the -O option, but in doing so it changes -O2 to -Os. Since Os is more a less the same as O2, I thin that a god fix would be to add elif is-flag -Os; then mozconfig_annotate "from CFLAGS" --enable-optimize=-Os just before the else. This way -O2 is the default, -O1 and -O0 are used when needed and -Os is used only if it is the CFLAGS.