Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 266260 - media-sound/musepack-tools-1.15v does not compile with -O2
Summary: media-sound/musepack-tools-1.15v does not compile with -O2
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Gentoo Sound Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-04-15 14:24 UTC by Nick Malakhov
Modified: 2009-05-12 20:52 UTC (History)
1 user (show)

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


Attachments
emerge -p media-sound/musepack-tools --info (emerge-info,10.62 KB, text/plain)
2009-04-20 06:23 UTC, Nick Malakhov
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Nick Malakhov 2009-04-15 14:24:40 UTC
I had a strange incident today. On my core2duo 
emerge =media-sound/musepack-tools-1.15v
was working for two hours. The last compile time was 8 seconds
I solve this problem by setting 

CFLAGS="-O1-pipe -march=native -msse4.1 -mmmx -fomit-frame-pointer -w"

while all my system built with '-O2' flag. 

So, I think, there should be some verification, like in openoffice, and portage should disable -O2 flag for this package.


cc --version
gcc (Gentoo 4.3.2-r3 p1.6, pie-10.1.5) 4.3.2
Copyright (C) 2008 Free Software Foundation, Inc.
Comment 1 Robert Spencer 2009-04-15 18:30:32 UTC
I can confirm that this package does not compile with either -O2 or -Os but does compile with -O1. This package did compile with -O2 when I was using GCC 4.1.2 and when I switched to GCC 4.3.2 the package failure occurred.

relevant emerge --info

Portage 2.1.6.7 (default/linux/x86/2008.0, gcc-4.3.2, glibc-2.8_p20080602-r1, 2.6.29-gentoo i686)
=================================================================
System uname: Linux-2.6.29-gentoo-i686-AMD_Athlon-tm-_64_Processor_3700+-with-glibc2.0
Timestamp of tree: Wed, 15 Apr 2009 00:00:01 +0000
app-shells/bash:     3.2_p39
dev-java/java-config: 1.3.7-r1, 2.1.7
dev-lang/python:     2.4.4-r13, 2.5.2-r7
dev-python/pycrypto: 2.0.1-r6
dev-util/cmake:      2.6.2-r1
sys-apps/baselayout: 1.12.11.1
sys-apps/sandbox:    1.2.18.1-r2
sys-devel/autoconf:  2.13, 2.63
sys-devel/automake:  1.5, 1.7.9-r1, 1.8.5-r3, 1.9.6-r2, 1.10.2
sys-devel/binutils:  2.18-r3
sys-devel/gcc-config: 1.4.0-r4
sys-devel/libtool:   1.5.26
virtual/os-headers:  2.6.27-r2
ACCEPT_KEYWORDS="x86"
CBUILD="i686-pc-linux-gnu"
CFLAGS="-O2 -march=native -pipe -fomit-frame-pointer"
CHOST="i686-pc-linux-gnu"
CONFIG_PROTECT="/etc /var/lib/hsqldb"
CONFIG_PROTECT_MASK="/etc/ca-certificates.conf /etc/env.d /etc/env.d/java/ /etc/fonts/fonts.conf /etc/gconf /etc/revdep-rebuild /etc/terminfo /etc/udev/rules.d"
CXXFLAGS="-O2 -march=native -pipe -fomit-frame-pointer"
DISTDIR="/usr/portage/distfiles"
FEATURES="distlocks fixpackages parallel-fetch protect-owned sandbox sfperms strict unmerge-orphans userfetch"
GENTOO_MIRRORS="http://gentoo.mirrors.tds.net/gentoo "
LDFLAGS="-Wl,-O1"
LINGUAS="en_US"
MAKEOPTS="-j2"



I fixed by adding a custom cflags file by following the example in this wiki: http://www.gentoo-wiki.info/CflagsExceptions

-Bob
Comment 2 Lars Wendler (Polynomial-C) (RETIRED) gentoo-dev 2009-04-18 15:06:44 UTC
Please attach the build.log file of the failed emerge to this bug.

Nick, you forgot to post your "emerge --info".
Comment 3 Nick Malakhov 2009-04-20 06:22:47 UTC
Actually, there is nothing useful in build.log:

>>> Unpacking source...
>>> Unpacking mpcsv7-src-1.15v.tar.bz2 to /var/tmp/portage/media-sound/musepack-tools-1.15v-r1/work
 * Applying musepack-tools-1.15v-Makefile.patch ...
  [ ok ]
 * Applying musepack-tools-1.15v-gcc4.patch ...
  [ ok ]
 * Applying musepack-tools-1.15v-execstack.patch ...
  [ ok ]
 * Applying musepack-tools-1.15v-fbsd.patch ...
  [ ok ]
>>> Source unpacked in /var/tmp/portage/media-sound/musepack-tools-1.15v-r1/work
>>> Compiling source in /var/tmp/portage/media-sound/musepack-tools-1.15v-r1/work/sv7 ...
make -j4 -s CC=gcc mppenc mppdec replaygain
cpu_feat.nas:142: warning: label alone on a line without a colon might be in error

*** Compile sources with ***

gcc -O2 -pipe -march=native -msse4.1 -mmmx -fomit-frame-pointer -w -I/var/tmp/portage/media-sound/musepack-tools-1.15v-r1/work/sv7  -DCVD_FASTLOG -DFAST_MATH    -DMPPDEC_VERSION="1.15v" -DMPPENC_VERSION="1.15v" -I/usr/include -L/usr/lib -fno-strict-aliasing -fno-gcse -fno-finite-math-only -fno-unsafe-math-optimizations -o <<EXE>> <<SRC>> -lm


*** Execute binary with ***

./<<EXE>>



then compilation hangs and top shows this in a first line:

 9679 root      20   0  2428  776  456 R   97  0.0   0:15.96 config


Comment 4 Nick Malakhov 2009-04-20 06:23:30 UTC
Created attachment 188937 [details]
emerge -p media-sound/musepack-tools --info
Comment 5 Samuli Suominen (RETIRED) gentoo-dev 2009-05-12 07:09:27 UTC

*** This bug has been marked as a duplicate of bug 167534 ***
Comment 6 Nick Malakhov 2009-05-12 07:40:08 UTC
Seems like adding -j1 didn't help



I am wathing this for 20 minutes:

>>> Emerging (1 of 1) media-sound/musepack-tools-1.15v-r1
 * mpcsv7-src-1.15v.tar.bz2 RMD160 SHA1 SHA256 size ;-) ...                                                                                                                 [ ok ]
 * checking ebuild checksums ;-) ...                                                                                                                                        [ ok ]
 * checking auxfile checksums ;-) ...                                                                                                                                       [ ok ]
 * checking miscfile checksums ;-) ...                                                                                                                                      [ ok ]
>>> Unpacking source...
>>> Unpacking mpcsv7-src-1.15v.tar.bz2 to /var/tmp/portage/media-sound/musepack-tools-1.15v-r1/work
tar: Нулевой блок в 5019
 * Applying musepack-tools-1.15v-Makefile.patch ...                                                                                                                          [ ok ]
 * Applying musepack-tools-1.15v-gcc4.patch ...                                                                                                                              [ ok ]
 * Applying musepack-tools-1.15v-execstack.patch ...                                                                                                                         [ ok ]
 * Applying musepack-tools-1.15v-fbsd.patch ...                                                                                                                              [ ok ]
>>> Source unpacked in /var/tmp/portage/media-sound/musepack-tools-1.15v-r1/work
>>> Compiling source in /var/tmp/portage/media-sound/musepack-tools-1.15v-r1/work/sv7 ...
make -j4 -s -j1 CC=gcc mppenc mppdec replaygain

*** Compile sources with ***

gcc -O2 -pipe -march=native -msse4.1 -mmmx -fomit-frame-pointer -w -I/var/tmp/portage/media-sound/musepack-tools-1.15v-r1/work/sv7  -DCVD_FASTLOG -DFAST_MATH    -DMPPDEC_VERSION="1.15v" -DMPPENC_VERSION="1.15v" -I/usr/include -L/usr/lib -fno-strict-aliasing -fno-gcse -fno-finite-math-only -fno-unsafe-math-optimizations -o <<EXE>> <<SRC>> -lm


*** Execute binary with ***

./<<EXE>>
Comment 7 Samuli Suominen (RETIRED) gentoo-dev 2009-05-12 07:45:22 UTC
(In reply to comment #6)
> Seems like adding -j1 didn't help
> 
> 
> 
> I am wathing this for 20 minutes:
> 
> >>> Emerging (1 of 1) media-sound/musepack-tools-1.15v-r1
>  * mpcsv7-src-1.15v.tar.bz2 RMD160 SHA1 SHA256 size ;-) ...                    
>                                                                                
>             [ ok ]
>  * checking ebuild checksums ;-) ...                                           
>                                                                                
>             [ ok ]
>  * checking auxfile checksums ;-) ...                                          
>                                                                                
>             [ ok ]
>  * checking miscfile checksums ;-) ...                                         
>                                                                                
>             [ ok ]
> >>> Unpacking source...
> >>> Unpacking mpcsv7-src-1.15v.tar.bz2 to /var/tmp/portage/media-sound/musepack-tools-1.15v-r1/work
> tar: Нулевой блок в 5019
>  * Applying musepack-tools-1.15v-Makefile.patch ...                            
>                                                                                
>              [ ok ]
>  * Applying musepack-tools-1.15v-gcc4.patch ...                                
>                                                                                
>              [ ok ]
>  * Applying musepack-tools-1.15v-execstack.patch ...                           
>                                                                                
>              [ ok ]
>  * Applying musepack-tools-1.15v-fbsd.patch ...                                
>                                                                                
>              [ ok ]
> >>> Source unpacked in /var/tmp/portage/media-sound/musepack-tools-1.15v-r1/work
> >>> Compiling source in /var/tmp/portage/media-sound/musepack-tools-1.15v-r1/work/sv7 ...
> make -j4 -s -j1 CC=gcc mppenc mppdec replaygain
> 
> *** Compile sources with ***
> 
> gcc -O2 -pipe -march=native -msse4.1 -mmmx -fomit-frame-pointer -w
> -I/var/tmp/portage/media-sound/musepack-tools-1.15v-r1/work/sv7  -DCVD_FASTLOG
> -DFAST_MATH    -DMPPDEC_VERSION="1.15v" -DMPPENC_VERSION="1.15v" -I/usr/include
> -L/usr/lib -fno-strict-aliasing -fno-gcse -fno-finite-math-only
> -fno-unsafe-math-optimizations -o <<EXE>> <<SRC>> -lm
> 
> 
> *** Execute binary with ***
> 
> ./<<EXE>>
> 

Sigh, see: make -j4

Besides =media-sound/musepack-tools-1.15v-r1 does this forcing of -j1 for you, use that.

And this is anyway a duplicate of 167534, please continue there if there's something new.

*** This bug has been marked as a duplicate of bug 167534 ***
Comment 8 Samuli Suominen (RETIRED) gentoo-dev 2009-05-12 07:46:40 UTC
Oh, I'm so blind, you were saying -O2 not -j2, sorry!
Comment 9 Samuli Suominen (RETIRED) gentoo-dev 2009-05-12 07:47:22 UTC
So it does work with -O1?
Comment 10 Samuli Suominen (RETIRED) gentoo-dev 2009-05-12 08:13:34 UTC
+  12 May 2009; Samuli Suominen <ssuominen@gentoo.org>
+  musepack-tools-1.15v-r1.ebuild:
+  Use replace-flags for -O1 wrt #266260, thanks to Nick Malakhov.
+
Comment 11 Nick Malakhov 2009-05-12 08:48:31 UTC
Thanks, it now compiles fine, but with following message:


 * QA Notice: Files built without respecting LDFLAGS have been detected
 *  Please include the following list of files in your report:         
 * /usr/bin/mppdec                                                     
 * /usr/bin/mppenc                                                     
 * /usr/bin/replaygain  


Should I reopen this bug or create another?
Comment 12 Samuli Suominen (RETIRED) gentoo-dev 2009-05-12 20:52:25 UTC
(In reply to comment #11)
> Thanks, it now compiles fine, but with following message:
> 
> 
>  * QA Notice: Files built without respecting LDFLAGS have been detected
>  *  Please include the following list of files in your report:         
>  * /usr/bin/mppdec                                                     
>  * /usr/bin/mppenc                                                     
>  * /usr/bin/replaygain  
> 
> 
> Should I reopen this bug or create another?
> 

Please open a new one. There's actually a new musepack-tools from upstream available, I'm working on getting it in tree so I doesn't really matter if
this version isn't entirely polished.. but sure, open a new bug to keep
track of the problem.