Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 29028 - When building smpeg-xmms there is a problem with the smpeg-xmms configure script...
Summary: When building smpeg-xmms there is a problem with the smpeg-xmms configure scr...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: x86 Linux
: High normal (vote)
Assignee: Gentoo Sound Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-09-17 21:37 UTC by Donald E. Curtis
Modified: 2003-09-20 03:16 UTC (History)
0 users

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


Attachments
Patch to Fix Configure Error (smpeg-xmms_0.3.5.patch,1.20 KB, patch)
2003-09-20 01:07 UTC, Donald E. Curtis
Details | Diff
Updated Ebuild to use the Patch (smpeg-xmms-0.3.5.ebuild,895 bytes, text/plain)
2003-09-20 01:35 UTC, Donald E. Curtis
Details
Context Patch (smpeg-xmms_context.patch,1.41 KB, patch)
2003-09-20 03:14 UTC, Donald E. Curtis
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Donald E. Curtis 2003-09-17 21:37:36 UTC
When i compiled smpeg-xmms 0.3.5 i got an error that the conftest.c would
compile but not run.  in fact i found that it didn't compile at all.  the
configure script generates this conftest.c file and when echoing the code it was
putting in hard line breaks that was messing up the printf's in the testing
code.  ANYWAYS, here is the patch to fix the problem.  I dunno if you can go
ahead and throw this in as a patch even though its smpeg-xmms's problem.

--- configure   2003-09-18 04:15:46.000000000 +0000
+++ configure.new       2003-09-18 04:17:29.000000000 +0000
@@ -2701,16 +2701,11 @@
     }
   else
     {
-      printf("\n*** 'smpeg-config --version' returned %d.%d.%d, but the 
-minimum version\n", $smpeg_major_version, $smpeg_minor_version, 
-$smpeg_micro_version);
-      printf("*** of SMPEG required is %d.%d.%d. If smpeg-config is 
-correct, then it is\n", major, minor, micro);
+      printf("\n*** 'smpeg-config --version' returned %d.%d.%d, but the minimum
version\n", $smpeg_major_version, $smpeg_min
or_version, $smpeg_micro_version);
+      printf("*** of SMPEG required is %d.%d.%d. If smpeg-config is correct,
then it is\n", major, minor, micro);
       printf("*** best to upgrade to the required version.\n");
-      printf("*** If smpeg-config was wrong, set the environment variable 
-SMPEG_CONFIG\n");
-      printf("*** to point to the correct copy of smpeg-config, and remove 
-the file\n");
+      printf("*** If smpeg-config was wrong, set the environment variable
SMPEG_CONFIG\n");
+      printf("*** to point to the correct copy of smpeg-config, and remove the
file\n");
       printf("*** config.cache before re-running configure\n");
       return 1;
     }



Reproducible: Always
Steps to Reproduce:
1. emerge smpeg-xmms

Actual Results:  
checking for SMPEG - version >= 0.4.4... no
*** Could not run SMPEG test program, checking why...
*** The test program compiled, but did not run. This 
usually means
*** that the run-time linker is not finding SMPEG or 
finding the wrong
*** version of SMPEG. If it is not finding SMPEG, you'll 
need to set your
*** LD_LIBRARY_PATH environment variable, or edit 
/etc/ld.so.conf to point
*** to the installed location  Also, make sure you have run 
ldconfig if that
*** is required on your system
***
*** If you have an old version installed, it is best to 
remove it, although
*** you may also be able to get things to work by modifying 
LD_LIBRARY_PATH
configure: error: *** SMPEG >= 0.4.4 not installed - please install first ***

!!! ERROR: media-video/smpeg-xmms-0.3.5 failed.
!!! Function econf, Line 324, Exitcode 1
!!! econf failed



Expected Results:  
checking for smpeg-config... /usr/bin/smpeg-config
checking for SMPEG - version >= 0.4.4... yes

(after patching)

Portage 2.0.49-r4 (default-x86-1.4, gcc-3.3.1, glibc-2.3.2-r1, 2.4.20-gentoo-r7)
=================================================================
System uname: 2.4.20-gentoo-r7 i686 Intel(R) Pentium(R) 4 CPU 2.53GHz
ACCEPT_KEYWORDS="x86 ~x86"
AUTOCLEAN="yes"
CFLAGS="-O3 -march=i686 -funroll-loops -pipe"
CHOST="i686-pc-linux-gnu"
COMPILER="gcc3"
CONFIG_PROTECT="/etc /var/qmail/control /usr/kde/2/share/config
/usr/kde/3/share/config /usr/X11R6/lib/X11/xkb /usr/kde/3.1/share/config
/usr/share/texmf/tex/generic/config/ /usr/share/texmf/tex/platex/config/
/usr/share/config"
CONFIG_PROTECT_MASK="/etc/gconf /etc/env.d"
CXXFLAGS="-O2 -mcpu=i686 -pipe"
DISTDIR="/usr/portage/distfiles"
FEATURES="sandbox ccache autoaddcvs"
GENTOO_MIRRORS="http://gentoo.oregonstate.edu
http://distro.ibiblio.org/pub/Linux/distributions/gentoo"
MAKEOPTS="-j2"
PKGDIR="/usr/portage/packages"
PORTAGE_TMPDIR="/var/tmp"
PORTDIR="/usr/portage"
PORTDIR_OVERLAY="/usr/local/portage"
SYNC="rsync://rsync.gentoo.org/gentoo-portage"
USE="x86 oss apm avi crypt cups encode foomaticdb gif imlib jpeg libg++ libwww
mad mikmod motif mpeg ncurses nls pdflib png quicktime slang spell truetype xml2
xmms xv zlib gdbm berkdb readline arts svga java sdl gpm tcpd pam ssl perl
python oggvorbis gtk qt opengl X gtk2 -gnome -kde alsa xinerama mmx"
Comment 1 J. Ellis (RETIRED) gentoo-dev 2003-09-20 00:45:28 UTC
Can you add this as an attachment to the bug? i'll try and add it after you update. Thanks.
Comment 2 Donald E. Curtis 2003-09-20 01:07:01 UTC
Created attachment 18015 [details, diff]
Patch to Fix Configure Error
Comment 3 J. Ellis (RETIRED) gentoo-dev 2003-09-20 01:17:25 UTC
Nevermind, i created the patch and have commited. Providing context diff is probably better.

Commited as -r1, ~x86 masked. Let me know if there are any issues.

Thanks for the submission.
Comment 4 Donald E. Curtis 2003-09-20 01:35:34 UTC
Created attachment 18016 [details]
Updated Ebuild to use the Patch
Comment 5 Donald E. Curtis 2003-09-20 03:14:00 UTC
Created attachment 18017 [details, diff]
Context Patch
Comment 6 Donald E. Curtis 2003-09-20 03:16:04 UTC
The patch in the CVS tree is incorrect for some reason or another but i have uploaded a context version of the patch.   The current CVS patch leaves one line unchanged that needs to be changed.