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

From: [email-masked] (GNU bug Tracking System)
Date: Wed, 14 Jul 2010 14:47:02 +0000

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 [email-masked].

-- 
6628: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=6628
GNU Bug Tracking System
Contact [email-masked] with problems

Subject: Re: bug#6628: [PATCH] omit $(POW_LIB) when linking, as this is no longer needed
From: Pádraig Brady <[email-masked]>
To: Paul Eggert <[email-masked]>
Cc: [email-masked]
Date: Wed, 14 Jul 2010 15:45:09 +0100
List-Id: <debbugs-submit.debbugs.gnu.org>

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 <[email-masked]>
> 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.

Subject: [PATCH] omit $(POW_LIB) when linking, as this is no longer needed
From: Paul Eggert <[email-masked]>
To: Bug Coreutils <[email-masked]>
Date: Tue, 13 Jul 2010 16:32:17 -0700
List-Id: <debbugs-submit.debbugs.gnu.org>

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 <[email-masked]>
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