Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 476318

Summary: media-video/mplayer should use --extra-cflags and --extra-ldflags
Product: Gentoo Linux Reporter: Nikoli <nikoli>
Component: Current packagesAssignee: Gentoo Media-video project <media-video>
Status: RESOLVED WORKSFORME    
Severity: normal    
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---

Description Nikoli 2013-07-09 20:48:10 UTC
Now ./configure prints this warning during build:
"MPlayer compilation will use the CPPFLAGS/CFLAGS/LDFLAGS/YASMFLAGS set by you,
but:

    *** ***  DO NOT REPORT BUGS IF IT DOES NOT COMPILE/WORK!  *** ***

It is strongly recommended to let MPlayer choose the correct *FLAGS!
To do so, remove *FLAGS from the environment an re-run configure.
You can use --extra-*flags to add custom flags if necessary."

To avoid it ebuild should not replace CFLAGS and LDFLAGS, but append instead, it is fixed in mplayer2 and mpv ebuilds:
	CFLAGS= LDFLAGS= ./configure \
		--cc="$(tc-getCC)" \
		--extra-cflags="${CFLAGS}" \
		--extra-ldflags="${LDFLAGS}" \
Comment 1 Alexis Ballier gentoo-dev 2013-07-29 20:16:29 UTC
x86_64-pc-linux-gnu-gcc -MMD -MP -Wundef  -Wstrict-prototypes -Wmissing-prototypes -Wdisabled-optimization -Wno-pointer-sign -Wdeclaration-after-statement -std=gnu99  -D_POSIX_C_SOURCE=200112 -D_XOPEN_SOURCE=600 -D_ISOC99_SOURCE -I. -Iffmpeg -march=native -mssse3 -msse4 -msse4.1 -msse4.2 -mavx -maes -O2 -pipe -ggdb3 -fno-tree-vectorize -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE64_SOURCE  -fpie -DPIC -D_REENTRANT -I/usr/include/   -D_REENTRANT  -I/usr/include -I/usr/include/freetype2 -I/usr/include/fribidi -I/usr/include/glib-2.0 -I/usr/lib64/glib-2.0/include  -I/usr/include -I/usr/include/libdvdread -c -o libmpcodecs/vf_harddup.o libmpcodecs/vf_harddup.c



seems it does the right thing, simply ignore the warning