Yesterday, Ondrej noticed that the 'supersede' module does not mention
$(LIB_ACL).

But these .o files from the 'acl-permissions' module do need -lacl:

acl-internal.o
                 U acl_entries
                 U acl_free
                 U acl_get_entry
                 U acl_get_tag_type

get-permissions.o
                 U acl_errno_valid
                 U acl_get_fd
                 U acl_get_file

set-permissions.o
                 U acl_access_nontrivial
                 U acl_default_nontrivial
                 U acl_delete_def_file
                 U acl_errno_valid
                 U acl_free
                 U acl_from_mode
                 U acl_set_fd
                 U acl_set_file

And since the modules copy-file, supersede depend on 'acl-permissions',
they need -lacl as well.


2023-01-05  Bruno Haible  <br...@clisp.org>

        copy-file, supersede: Fix link-time requirements.
        * modules/copy-file (Link): Add $(LIB_ACL).
        * modules/supersede (Link): Likewise.

diff --git a/modules/copy-file b/modules/copy-file
index 6140c72fa1..fa4a403af1 100644
--- a/modules/copy-file
+++ b/modules/copy-file
@@ -36,6 +36,7 @@ Include:
 "copy-file.h"
 
 Link:
+$(LIB_ACL)
 $(LIB_MBRTOWC)
 
 License:
diff --git a/modules/supersede b/modules/supersede
index 70f3fdc482..5e458536a2 100644
--- a/modules/supersede
+++ b/modules/supersede
@@ -35,6 +35,7 @@ Include:
 "supersede.h"
 
 Link:
+$(LIB_ACL)
 $(LIB_GETRANDOM)
 $(LIB_CLOCK_GETTIME)
 $(LIBTHREAD)




Reply via email to