Paul Eggert wrote:
> On 11/15/20 10:01 AM, Tom G. Christensen wrote:
> > Undefined                       first referenced
> >   symbol                             in file
> > clock_gettime                       ../gllib/libgnu.a(tempname.o)
> > ld: fatal: symbol referencing errors. No output written to test-getumask
> > collect2: error: ld returned 1 exit status
> > make[3]: *** [test-getumask] Error 1
> > 
> > This issue looks to be generic. It fails similarly on CentOS 6.
> 
> I fixed this issue by installing the attached. Thanks for reporting it.

Thanks. Fixing the LDFLAGS for the tests fixes the immediate error,
but consumers of Gnulib modules should also know what they should add
in their prog_LDFLAGS. So, here are two patches to update the 'Link'
sections in the module descriptions. It propagates like this:

  tempname <-- mkdtemp, mkostemp, mkostemps, mkstemp, mkstemps
           <-- supersede
           <-- tmpfile <-- tmpfile-safer
           <-- getumask

Bruno


2020-11-16  Bruno Haible  <br...@clisp.org>

        getumask: Document link dependencies.
        * modules/getumask (Link): New section.

2020-11-16  Bruno Haible  <br...@clisp.org>

        Update link dependencies in modules after 2020-09-09 change.
        * modules/tempname (Link): Add $(LIB_CLOCK_GETTIME).
        * modules/mkdtemp (Link): Likewise.
        * modules/mkostemp (Link): Likewise.
        * modules/mkostemps (Link): Likewise.
        * modules/mkstemp (Link): Likewise.
        * modules/mkstemps (Link): Likewise.
        * modules/supersede (Link): Likewise.
        * modules/tmpfile (Link): Likewise.
        * modules/tmpfile-safer (Link): Likewise.

>From 1bb0df49196cc9ffaf5ccd5278a1c3e4387a8eef Mon Sep 17 00:00:00 2001
From: Bruno Haible <br...@clisp.org>
Date: Mon, 16 Nov 2020 16:35:31 +0100
Subject: [PATCH 1/2] Update link dependencies in modules after 2020-09-09
 change.

* modules/tempname (Link): Add $(LIB_CLOCK_GETTIME).
* modules/mkdtemp (Link): Likewise.
* modules/mkostemp (Link): Likewise.
* modules/mkostemps (Link): Likewise.
* modules/mkstemp (Link): Likewise.
* modules/mkstemps (Link): Likewise.
* modules/supersede (Link): Likewise.
* modules/tmpfile (Link): Likewise.
* modules/tmpfile-safer (Link): Likewise.
---
 ChangeLog             | 13 +++++++++++++
 modules/mkdtemp       |  1 +
 modules/mkostemp      |  1 +
 modules/mkostemps     |  1 +
 modules/mkstemp       |  1 +
 modules/mkstemps      |  1 +
 modules/supersede     |  1 +
 modules/tempname      |  1 +
 modules/tmpfile       |  1 +
 modules/tmpfile-safer |  1 +
 10 files changed, 22 insertions(+)

diff --git a/ChangeLog b/ChangeLog
index 8c4342f..92a8add 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,16 @@
+2020-11-16  Bruno Haible  <br...@clisp.org>
+
+	Update link dependencies in modules after 2020-09-09 change.
+	* modules/tempname (Link): Add $(LIB_CLOCK_GETTIME).
+	* modules/mkdtemp (Link): Likewise.
+	* modules/mkostemp (Link): Likewise.
+	* modules/mkostemps (Link): Likewise.
+	* modules/mkstemp (Link): Likewise.
+	* modules/mkstemps (Link): Likewise.
+	* modules/supersede (Link): Likewise.
+	* modules/tmpfile (Link): Likewise.
+	* modules/tmpfile-safer (Link): Likewise.
+
 2020-11-15  Paul Eggert  <egg...@cs.ucla.edu>
 
 	getumask-tests: port to Solaris 10 etc.
diff --git a/modules/mkdtemp b/modules/mkdtemp
index 13b2d5a..d4603bf 100644
--- a/modules/mkdtemp
+++ b/modules/mkdtemp
@@ -25,6 +25,7 @@ Include:
 
 Link:
 $(LIB_GETRANDOM)
+$(LIB_CLOCK_GETTIME)
 
 License:
 LGPLv2+
diff --git a/modules/mkostemp b/modules/mkostemp
index 460e842..8bfd9d5 100644
--- a/modules/mkostemp
+++ b/modules/mkostemp
@@ -28,6 +28,7 @@ Include:
 
 Link:
 $(LIB_GETRANDOM)
+$(LIB_CLOCK_GETTIME)
 
 License:
 LGPLv2+
diff --git a/modules/mkostemps b/modules/mkostemps
index bb9fe71..0f67ab4 100644
--- a/modules/mkostemps
+++ b/modules/mkostemps
@@ -27,6 +27,7 @@ Include:
 
 Link:
 $(LIB_GETRANDOM)
+$(LIB_CLOCK_GETTIME)
 
 License:
 LGPLv2+
diff --git a/modules/mkstemp b/modules/mkstemp
index d985715..ba0ef94 100644
--- a/modules/mkstemp
+++ b/modules/mkstemp
@@ -26,6 +26,7 @@ Include:
 
 Link:
 $(LIB_GETRANDOM)
+$(LIB_CLOCK_GETTIME)
 
 License:
 LGPLv2+
diff --git a/modules/mkstemps b/modules/mkstemps
index 1e5f097..364cfa8 100644
--- a/modules/mkstemps
+++ b/modules/mkstemps
@@ -26,6 +26,7 @@ Include:
 
 Link:
 $(LIB_GETRANDOM)
+$(LIB_CLOCK_GETTIME)
 
 License:
 LGPLv2+
diff --git a/modules/supersede b/modules/supersede
index b01ca52..38f52fa 100644
--- a/modules/supersede
+++ b/modules/supersede
@@ -34,6 +34,7 @@ Include:
 
 Link:
 $(LIB_GETRANDOM)
+$(LIB_CLOCK_GETTIME)
 
 License:
 GPL
diff --git a/modules/tempname b/modules/tempname
index 385c65a..27b0d3d 100644
--- a/modules/tempname
+++ b/modules/tempname
@@ -33,6 +33,7 @@ Include:
 
 Link:
 $(LIB_GETRANDOM)
+$(LIB_CLOCK_GETTIME)
 
 License:
 LGPLv2+
diff --git a/modules/tmpfile b/modules/tmpfile
index 20b88c8..fc71e2d 100644
--- a/modules/tmpfile
+++ b/modules/tmpfile
@@ -28,6 +28,7 @@ Include:
 
 Link:
 $(LIB_GETRANDOM)
+$(LIB_CLOCK_GETTIME)
 
 License:
 LGPL
diff --git a/modules/tmpfile-safer b/modules/tmpfile-safer
index 02e49d4..d385cdb 100644
--- a/modules/tmpfile-safer
+++ b/modules/tmpfile-safer
@@ -22,6 +22,7 @@ Include:
 
 Link:
 $(LIB_GETRANDOM)
+$(LIB_CLOCK_GETTIME)
 
 License:
 GPL
-- 
2.7.4

>From f1b7489ca2b67e4c68ab2a8e0591ef42562166e7 Mon Sep 17 00:00:00 2001
From: Bruno Haible <br...@clisp.org>
Date: Mon, 16 Nov 2020 16:37:44 +0100
Subject: [PATCH 2/2] getumask: Document link dependencies.

* modules/getumask (Link): New section.
---
 ChangeLog        | 5 +++++
 modules/getumask | 4 ++++
 2 files changed, 9 insertions(+)

diff --git a/ChangeLog b/ChangeLog
index 92a8add..c0c0466 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,10 @@
 2020-11-16  Bruno Haible  <br...@clisp.org>
 
+	getumask: Document link dependencies.
+	* modules/getumask (Link): New section.
+
+2020-11-16  Bruno Haible  <br...@clisp.org>
+
 	Update link dependencies in modules after 2020-09-09 change.
 	* modules/tempname (Link): Add $(LIB_CLOCK_GETTIME).
 	* modules/mkdtemp (Link): Likewise.
diff --git a/modules/getumask b/modules/getumask
index f7784da..61d71d0 100644
--- a/modules/getumask
+++ b/modules/getumask
@@ -26,6 +26,10 @@ Makefile.am:
 Include:
 <sys/stat.h>
 
+Link:
+$(LIB_GETRANDOM)
+$(LIB_CLOCK_GETTIME)
+
 License:
 GPL
 
-- 
2.7.4

Reply via email to