360139 – =net-libs/libpri-1.4.12_beta2 does not respect CFLAGS="-O"

Bug 360139 - =net-libs/libpri-1.4.12_beta2 does not respect CFLAGS="-O"

Summary: =net-libs/libpri-1.4.12_beta2 does not respect CFLAGS="-O"

Status:	RESOLVED FIXED

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	New packages (show other bugs)
Hardware:	All Linux

Importance:	Normal minor (vote)
Assignee:	Tony Vroon (RETIRED)

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2011-03-23 13:13 UTC by Agostino Sarubbo
Modified:	2011-05-17 08:11 UTC (History)
CC List:	1 user (show)

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

Attachments
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 Agostino Sarubbo gentoo-dev

2011-03-23 13:13:57 UTC

$summary

small example:


gcc -march=native -O2 -Wall -Wstrict-prototypes -Wmissing-prototypes -g -fPIC   -O2 -MD -MT rose_qsig_aoc.lo -MF .rose_qsig_aoc.lo.d -MP -c -o rose_qsig_aoc.lo rose_qsig_aoc.c


-march=native -O2 are my cflags, but buildsystem appends another -O2 that overwrites my optimization.
In this case, there is no problem, but if I use -O? it is replaced


thanks

(*) sort keywords when you modify the ebuild

Comment 1 Agostino Sarubbo gentoo-dev

2011-05-13 21:49:58 UTC

i forgot that should be dropped also -g

Comment 2 Tony Vroon (RETIRED) gentoo-dev

2011-05-17 08:11:15 UTC

+*libpri-1.4.12_beta3 (17 May 2011)
+
+  17 May 2011; Tony Vroon <chainsaw@gentoo.org> +libpri-1.4.12_beta3.ebuild,
+  +files/libpri-1.4.12-multilib.patch,
+  +files/libpri-1.4.12-respect-cflags.patch,
+  +files/libpri-1.4.12-respect-ldflags.patch:
+  New beta; patchset rediffed by Stefan Knoblich in bug #346573. Consolidated
+  Werror removal and CFLAGS patches; remove -O2 and -g as per Agostino "ago"
+  Sarubbo in bug #360139. Contains important PTP fixes with regard to
+  downing/restarting links.