Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 509286 - media-video/mplayer-1.2_pre20130729 - In file included from mixer.c:27:0: config.h:120:1: error: missing terminating " character
Summary: media-video/mplayer-1.2_pre20130729 - In file included from mixer.c:27:0: con...
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: Sparc Linux
: Normal normal (vote)
Assignee: media-video herd
Depends on:
Reported: 2014-05-01 19:28 UTC by Dustin
Modified: 2014-08-04 16:15 UTC (History)
2 users (show)

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

output of emerge --info (emergeinfo.txt,4.32 KB, text/plain)
2014-05-01 19:30 UTC, Dustin
build.log (build.log,85.97 KB, text/plain)
2014-05-02 16:39 UTC, Dustin
config.h (config.h,51.63 KB, text/x-chdr)
2014-05-02 17:53 UTC, Dustin

Note You need to log in before you can comment on or make changes to this bug.
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
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]
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]
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.