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

Bug 509286

Summary: media-video/mplayer-1.2_pre20130729 - In file included from mixer.c:27:0: config.h:120:1: error: missing terminating " character
Product: Gentoo Linux Reporter: Dustin <dustin.glidden>
Component: Current packagesAssignee: media-video herd <media-video>
Status: UNCONFIRMED ---    
Severity: normal CC: dustin.glidden, sparc
Priority: Normal    
Version: unspecified   
Hardware: Sparc   
OS: Linux   
Whiteboard:
Package list:
Runtime testing required: ---
Attachments: output of emerge --info
build.log
config.h

Description Dustin 2014-05-01 19:28:42 UTC
On sparc mplayer fails to build on sparc with the following errors thrown repeatedly:
./config.h:119:23: warning: missing terminating " character [enabled by default]
./config.h:120:1: warning: missing terminating " character [enabled by default]
./config.h:120:1: error: missing terminating " character

Looking at the generated config.h on lines 119-120 it appears that the quote is not ended properly on line 119 and instead there is a space, then newline, and then the quote on its own line.  Fixing this manually fixes the problem.



Reproducible: Always

Steps to Reproduce:
1.emerge mplayer on sparc
2.
3.
Comment 1 Dustin 2014-05-01 19:30:38 UTC
Created attachment 376162 [details]
output of emerge --info
Comment 2 Jeroen Roovers (RETIRED) gentoo-dev 2014-05-02 00:48:13 UTC
Please attach the entire build log to this bug report.
Comment 3 Dustin 2014-05-02 16:39:10 UTC
Created attachment 376254 [details]
build.log
Comment 4 Jeroen Roovers (RETIRED) gentoo-dev 2014-05-02 17:41:50 UTC
It's probably useful to attach that config.h too.
Comment 5 Dustin 2014-05-02 17:53:16 UTC
Created attachment 376256 [details]
config.h
Comment 6 Stefano Takekawa 2014-05-06 11:22:38 UTC
I experienced the same problem on amd64. I was using lto also when compiling mplayer and you also did, if I understand well. I solved the problem compiling w/out lto.
Comment 7 Dustin 2014-05-06 17:40:50 UTC
You're absolutely right, it was lto.  I didn't even consider it because it was so early in the build process.  Whatever causes this should be fixed as once manually correcting the config.h mplayer compiles fine with lto.
Comment 8 dE 2014-08-04 16:15:35 UTC
I confirm this. I'm using LTO.

But this's note related to linking.