GNU bug report logs - #6628
[PATCH] omit $(POW_LIB) when linking, as this is no longer needed

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Paul Eggert <eggert <at> CS.UCLA.EDU>
Subject: bug#6628: closed (Re: bug#6628: [PATCH] omit $(POW_LIB) when
 linking,	as this is no longer needed)
Date: Wed, 14 Jul 2010 14:47:02 +0000

[Message part 1 (text/plain, inline)]

Your bug report

#6628: [PATCH] omit $(POW_LIB) when linking, as this is no longer needed

which was filed against the coreutils package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to 6628 <at> debbugs.gnu.org.

-- 
6628: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=6628
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: Pádraig Brady <P <at> draigBrady.com>
To: Paul Eggert <eggert <at> CS.UCLA.EDU>
Cc: 6628-done <at> debbugs.gnu.org
Subject: Re: bug#6628: [PATCH] omit $(POW_LIB) when linking,	as this is no
	longer needed
Date: Wed, 14 Jul 2010 15:45:09 +0100

On 14/07/10 00:32, Paul Eggert wrote:
> Here's an obvious patch to simplify src/Makefile.am now that
> gnulib strtod no longer needs libm.  Is it OK to install this?
> 
>>From 018e0adb679ac8ba713de00c1afb686395ce13fc Mon Sep 17 00:00:00 2001
> From: Paul R. Eggert <eggert <at> cs.ucla.edu>
> Date: Tue, 13 Jul 2010 16:29:37 -0700
> Subject: [PATCH] omit $(POW_LIB) when linking, as this is no longer needed

That looks good.
I've just updated to the latest gnulib,
so I went ahead and pushed that.

cheers,
Pádraig.

[Message part 3 (message/rfc822, inline)]

From: Paul Eggert <eggert <at> CS.UCLA.EDU>
To: Bug Coreutils <bug-coreutils <at> gnu.org>
Subject: [PATCH] omit $(POW_LIB) when linking, as this is no longer needed
Date: Tue, 13 Jul 2010 16:32:17 -0700

Here's an obvious patch to simplify src/Makefile.am now that
gnulib strtod no longer needs libm.  Is it OK to install this?

From 018e0adb679ac8ba713de00c1afb686395ce13fc Mon Sep 17 00:00:00 2001
From: Paul R. Eggert <eggert <at> cs.ucla.edu>
Date: Tue, 13 Jul 2010 16:29:37 -0700
Subject: [PATCH] omit $(POW_LIB) when linking, as this is no longer needed

* src/Makefile.am (printf_LDADD, seq_LDADD, sleep_LDADD, sort_LDADD):
(tail_LDADD, uptime_LDADD): Omit $(POW_LIB), as it's no longer
needed due to recent gnulib changes, where the strtod module no
longer uses the pow function.  strtold needs pow only because it's
sometimes aliased to strtod.  See
http://lists.gnu.org/archive/html/bug-gnulib/2010-07/msg00076.html
---
 src/Makefile.am |    8 --------
 1 files changed, 0 insertions(+), 8 deletions(-)

diff --git a/src/Makefile.am b/src/Makefile.am
index 2e0e320..d87fbb5 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -343,14 +343,6 @@ ls_LDADD += $(LIB_CAP)
 dd_LDADD += $(LIB_FDATASYNC)
 shred_LDADD += $(LIB_FDATASYNC)
 
-# for strtod, strtold
-printf_LDADD += $(POW_LIB)
-seq_LDADD += $(POW_LIB)
-sleep_LDADD += $(POW_LIB)
-sort_LDADD += $(POW_LIB)
-tail_LDADD += $(POW_LIB)
-uptime_LDADD += $(POW_LIB)
-
 # for xnanosleep
 sleep_LDADD += $(LIB_NANOSLEEP)
 sort_LDADD += $(LIB_NANOSLEEP)
-- 
1.7.1

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.