FYI: 353-gary-fix-nonrecursive-libobj-dist

2008-02-01 Thread Gary V. Vaughan 
Applied to HEAD as an obvious bugfix.

  Index: ChangeLog
  from  Gary V. Vaughan  [EMAIL PROTECTED]
  
* libltdl/Makefile.inc (EXTRA_DIST): Now that we're not using
Autoconf's AC_SOURCES macro, Automake can't trace our extra
source files argz, lt__dirent.c and lt__strl.c.  List them
manually here to make sure they are distributed.
* NEWS: Updated.
Reported by Bob Friesenhahn  [EMAIL PROTECTED]
  
  Index: NEWS
  ===
  RCS file: /sources/libtool/libtool/NEWS,v
  retrieving revision 1.216
  diff -u -u -r1.216 NEWS
  --- NEWS 31 Jan 2008 16:21:18 - 1.216
  +++ NEWS 1 Feb 2008 19:03:08 -
  @@ -1,5 +1,12 @@
   NEWS - list of user-visible changes between releases of GNU Libtool
   
  +New in 2.1c: 2012-??-??; CVS version 2.1c, Libtool team:
  +
  +* Bug fixes:
  +
  +  - argz.c, lt__dirent.c and lt__strl.c are correctly distributed with
  +parent projects using nonrecursive libltdl.
  +
   New in 2.1b: 2008-02-01; CVS version 2.1a, Libtool team:
   
   * Important incompatible changes and obsoleted features:
  Index: libltdl/Makefile.inc
  ===
  RCS file: /sources/libtool/libtool/libltdl/Makefile.inc,v
  retrieving revision 1.9
  diff -u -u -r1.9 Makefile.inc
  --- libltdl/Makefile.inc 31 Jan 2008 16:21:19 - 1.9
  +++ libltdl/Makefile.inc 1 Feb 2008 19:03:08 -
  @@ -70,6 +70,9 @@
  libltdl/ltdl.h \
  libltdl/slist.c
   
  +EXTRA_DIST  += libltdl/lt__dirent.c \
  +   libltdl/lt__strl.c
  +
   libltdl_libltdl_la_CPPFLAGS  = -DLTDLOPEN=$(LTDLOPEN) $(AM_CPPFLAGS)
   libltdl_libltdl_la_LDFLAGS   = $(AM_LDFLAGS) $(LTDL_VERSION_INFO) 
$(LT_DLPREOPEN)
   libltdl_libltdl_la_LIBADD= $(LTLIBOBJS)
  @@ -136,7 +139,8 @@
   ## --- ##
   
   BUILT_SOURCES+= libltdl/$(ARGZ_H)
  -EXTRA_DIST   += libltdl/argz_.h
  +EXTRA_DIST   += libltdl/argz_.h \
  +libltdl/argz.c
   
   # We need the following in order to create an argz.h when the system
   # doesn't have one that works with the given compiler.
-- 
  ())_.  Email me: [EMAIL PROTECTED]
  ( '/   Read my blog: http://blog.azazil.net
  / )= ...and my book: http://sources.redhat.com/autobook
`(_~)_ 
_
This patch notification generated by vcsapply version 1.0
http://savannah.gnu.org/projects/cvs-utils


pgpdL20nmVxp0.pgp
Description: PGP signature

Re: [libtool 2.1b] testsuite: 16 54 55 56 failed

2008-02-01 Thread Ralf Wildenhues 
Hello Andreas,

* Andreas Schwab wrote on Fri, Feb 01, 2008 at 03:43:25PM CET:
 16. convenience.at:69: testing ...
 ./convenience.at:70: { test -n $CXX  test X$CXX != Xno; } || (exit 77)
 libtool: compile:  g++ -c main1.cpp  -o .libs/main1.o
 /usr/src/packages/BUILD/libtool-2.1b/libtool: line 821: g++: command not found
 libtool: compile:  g++ -c a1.cpp  -o .libs/a1.o

Thanks for the bug report.  Fixed as follows.

The fact that AC_PROG_CXX sets CXX to g++ as a default seems to me like
a bug in Autoconf.  The code used gcc as a default up to commit
  d9409a1c3c06a015a0fc829989cb1a93c1599cd7
which it had ever since AC_PROG_CXX was added 1994 in commit
  8202df5faf642cde4e5b1fa02cd9cc4411a7a477

I'll propose to remove that in Autoconf.

Cheers,
Ralf

2008-02-01  Ralf Wildenhues  [EMAIL PROTECTED]

* tests/testsuite.at (LT_AT_TAG): For the CXX tag, if $CXX is
g++, check that `g++ -v' works; AC_PROG_CXX may have set that
as default value even if it found no C++ compiler.
Report by Andreas Schwab.

Index: tests/testsuite.at
===
RCS file: /cvsroot/libtool/libtool/tests/testsuite.at,v
retrieving revision 1.50
diff -u -r1.50 testsuite.at
--- tests/testsuite.at  25 Mar 2007 12:12:43 -  1.50
+++ tests/testsuite.at  1 Feb 2008 19:05:24 -
@@ -1,6 +1,7 @@
 # Process with autom4te to create an -*- Autotest -*- test suite.
 #
-#   Copyright (C) 2004, 2005, 2006, 2007 Free Software Foundation, Inc.
+#   Copyright (C) 2004, 2005, 2006, 2007, 2008 Free Software
+#   Foundation, Inc.
 #   Written by Gary V. Vaughan, 2004
 #
 #   This file is part of GNU Libtool.
@@ -187,6 +188,8 @@
 m4_define([LT_AT_TAG],
 [AT_KEYWORDS([$1])
 AT_CHECK([{ test -n [$]$1  test X[$]$1 != Xno; } || (exit 77)])
+m4_if([$1], [CXX],
+  [AT_CHECK([test X$CXX != Xg++ || (g++ -v /dev/null 21) || (exit 77)])])
 ])

Re: multilib patch

2008-02-01 Thread Ralf Wildenhues 
Hi Peter,

* Peter O'Gorman wrote on Thu, Jan 31, 2008 at 08:00:59AM CET:
 
 I installed amd64 debian in a VM and with this patch I get:
 /usr/lib/gcc/x86_64-linux/gnu/4.1.2 /usr/lib /lib
 
 I do not know if it is best to leave in the symlink and the original dir
 ( i.e. do cd $dir  pwd as well as cd $dir  pwd -P), which on debian
 would give something like:
 /usr/lib/gcc/x86_64-linux/gnu/4.1.2 /usr/lib64 /usr/lib /lib64 /lib

With this patch I get for -m32:
sys_lib_search_path_spec=/usr/lib/gcc/x86_64-linux-gnu/4.0.3/../../..  
/usr/lib/gcc/x86_64-linux-gnu/4.0.3/32 /usr/lib/gcc/x86_64-linux-gnu/4.0.3 
/usr/lib32 /lib32

The first entry is wrong, as it's equal to /usr/lib which is 64bit.

For -m64 I get:
sys_lib_search_path_spec=/usr/lib/gcc/x86_64-linux-gnu/4.0.3 /usr/lib /lib

I have not yet tried to understand the logic of the patch.

Cheers,
Ralf

Re: multilib patch

2008-02-01 Thread Peter O'Gorman 
Ralf Wildenhues wrote:
 
 I have not yet tried to understand the logic of the patch.


Sorry. I have dropped it. libstdc++ should still be found because of the
compiler_search_path changes.

Peter
-- 
Peter O'Gorman
http://pogma.com