259061 – net-libs/webkit-gtk forces -O2

Bug 259061 - net-libs/webkit-gtk forces -O2

Summary: net-libs/webkit-gtk forces -O2

Status:	RESOLVED FIXED

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	[OLD] Library (show other bugs)
Hardware:	All Linux

Importance:	High normal (vote)
Assignee:	Markus Ullmann (RETIRED)

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2009-02-15 06:35 UTC by Ryan Hill (RETIRED)
Modified:	2009-05-30 21:13 UTC (History)
CC List:	1 user (show)

See Also:
Package list:
Runtime testing required:	---

Attachments
webkit-gtk-0_p40220-CFLAGS.patch (webkit-gtk-0_p40220-CFLAGS.patch,592 bytes, patch) 2009-02-15 06:35 UTC, Ryan Hill (RETIRED)	Details \| Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Ryan Hill (RETIRED) gentoo-dev

2009-02-15 06:35:03 UTC

by default webkit adds -O2 after the user's CFLAGS, so any other -O setting will be overridden.

eg.
-O3 -march=core2 -fomit-frame-pointer -pipe -O2 -MT WebCore/page/libWebCore_la-WorkerNavigator.lo -MD -MP -MF WebCore/page/.deps/libWebCore_la-WorkerNavigator.Tpo -c -o WebCore/page/libWebCore_la-WorkerNavigator.lo `test -f 'WebCore/page/WorkerNavigator.cpp' || echo './'`WebCore/page/WorkerNavigator.cpp

adding --disable-optimizations to econf forces -O0, so to fix this we'll have to patch configure.ac.

Comment 1 Ryan Hill (RETIRED) gentoo-dev

2009-02-15 06:35:38 UTC

Created attachment 182097 [details, diff]
webkit-gtk-0_p40220-CFLAGS.patch

Comment 2 Nirbheek Chauhan (RETIRED) gentoo-dev

2009-05-23 19:58:51 UTC

This has been fixed in upstream trunk, will go into 1.1.8

Comment 3 Nirbheek Chauhan (RETIRED) gentoo-dev

2009-05-30 21:13:25 UTC

Wasn't fixed upstream, bug fixed in the ebuild for now