Bug List: (This bug is not in your last search results)   Show last search results      Search page      Enter new bug
Bug#: 24022
Alias:
Product:
Component:
Status: CLOSED
Resolution: REMIND
Assigned To: Gentoo Games <games@gentoo.org>
Hardware:
OS:
Version:
Priority:
Severity:
Reporter: Bryan Feir <jenora@sympatico.ca>
Add CC:
CC:
URL:
Summary:
Status Whiteboard:
Keywords:

Filename Description Type Creator Created Size Actions
Create a New Attachment (proposed patch, testcase, etc.) View All

Bug 24022 depends on: Show dependency tree
Bug 24022 blocks:
Votes: 0    Show votes for this bug    Vote for this bug

Additional Comments: (this is where you put emerge --info)


Not eligible to see or edit group visibility for this bug.




View Bug Activity   |   Format For Printing   |   XML   |   Clone This Bug


Description:   Opened: 2003-07-06 19:32 0000
The app-games/adonthell ebuild fails during the compile step trying to find a
'/lib/-lsandbox' library.

The error is created by the way the configure file attempts to find out what
libraries Python requires by parsing the ldd output.  Unfortunately, thanks to
the LD_PRELOAD variable being set to /lib/libsandbox.so, the sed script inside
the configure file turns that into /lib/-lsandbox and leaves that in the PY_LIBS
variable.

There is currently a patch in the adonthell/files directory that is an attempt
to fix this, however it does two things wrong:
 * It doesn't actually remove the sandbox entry, simply removes the /lib/ part
 * The patch is applied to the configure file and then autoconf is run in the
ebuild, thus overwriting the patch that was just applied.

Since the core of the problem is the LD_PRELOAD variable, it is simpler to just
not export that variable before running the ldd test.  Either fix the configure
file:

*** configure-orig      Sun Jan  5 17:10:50 2003
--- configure   Sun Jul  6 22:25:26 2003
***************
*** 12770,12776 ****
--- 12770,12778 ----
  
     if test x$haveldd != x ; then
  
+        export -n LD_PRELOAD
         py_deps=`ldd $PYPACKAGE | sed 's/\( *lib\([^\/]*\)\.so.*=.*$\)/-l\2/p; d'`
+        export LD_PRELOAD
         for py_lib in $py_deps ; do
           if test "$py_lib" != "-lm" && test "$py_lib" != "-lc" ; then
              PY_DEPS="$PY_DEPS $py_lib"

Or, more 'properly', fix the configure.in file and then run autoconf:

*** configure.in-orig   Sun Jan  5 17:01:24 2003
--- configure.in        Sun Jul  6 22:26:34 2003
***************
*** 216,222 ****
--- 216,224 ----
  
     if test x$haveldd != x ; then
         changequote(<<, >>)
+        export -n LD_PRELOAD
         py_deps=`ldd $PYPACKAGE | sed 's/\( *lib\([^\/]*\)\.so.*=.*$\)/-l\2/p; d'`
+        export LD_PRELOAD
         for py_lib in $py_deps ; do
           if test "$py_lib" != "-lm" && test "$py_lib" != "-lc" ; then
              PY_DEPS="$PY_DEPS $py_lib"

-- Bryan Feir

------- Comment #1 From SpanKY 2003-07-06 22:33:38 0000 -------

*** This bug has been marked as a duplicate of 17281 ***

------- Comment #2 From Mr. Bones. 2005-05-19 09:06:40 0000 -------
Closing to clean up after bugzilla upgrade.  reopen if closed in error. 
Thanks.

Bug List: (This bug is not in your last search results)   Show last search results      Search page      Enter new bug