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 packages	Assignee:	Gentoo Media-video project <media-video>
Status:	RESOLVED OBSOLETE
Severity:	normal	CC:	dustin.glidden, matoro_bugzilla_gentoo, sparc
Priority:	Normal
Version:	unspecified
Hardware:	Sparc
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---
Bug Depends on:
Bug Blocks:	618550
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.

Comment 9 matoro archtester

2022-05-01 18:41:26 UTC

Current version of =media-video/mplayer-1.4-r1 builds fine for me with LTO on sparc, this version has been out of tree for a while.  Is this still relevant?