[PATCH] qacl: split into qcopy-acl and qset-acl

[Paul Eggert]

Emacs needs the former, but not the latter.
* modules/acl-permissions: New file, containing most of the old qacl.
* modules/file-has-acl (Depends-on): Depend on acl-permissions, not qacl.
* modules/qacl: Now merely depends on qcopy-acl and qset-acl.
* modules/qcopy-acl, modules/qset-acl: New files.
* MODULES.html.sh (File system functions):
Mention the new modules, and mention qacl while we're at it.
---
 ChangeLog               | 11 +++++++++++
 MODULES.html.sh         |  4 ++++
 modules/acl-permissions | 37 +++++++++++++++++++++++++++++++++++++
 modules/file-has-acl    |  2 +-
 modules/qacl            | 23 ++++-------------------
 modules/qcopy-acl       | 25 +++++++++++++++++++++++++
 modules/qset-acl        | 25 +++++++++++++++++++++++++
 7 files changed, 107 insertions(+), 20 deletions(-)
 create mode 100644 modules/acl-permissions
 create mode 100644 modules/qcopy-acl
 create mode 100644 modules/qset-acl

diff --git a/ChangeLog b/ChangeLog
index 6e43e66..517d439 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,14 @@
+2015-05-27  Paul Eggert  <address@hidden>
+
+       qacl: split into qcopy-acl and qset-acl
+       Emacs needs the former, but not the latter.
+       * modules/acl-permissions: New file, containing most of the old qacl.
+       * modules/file-has-acl (Depends-on): Depend on acl-permissions, not 
qacl.
+       * modules/qacl: Now merely depends on qcopy-acl and qset-acl.
+       * modules/qcopy-acl, modules/qset-acl: New files.
+       * MODULES.html.sh (File system functions):
+       Mention the new modules, and mention qacl while we're at it.
+
 2015-05-27  Glenn Morris  <address@hidden>
 
        gitlog-to-changelog: new option --ignore-line
diff --git a/MODULES.html.sh b/MODULES.html.sh
index 0019b22..7835e9d 100755
--- a/MODULES.html.sh
+++ b/MODULES.html.sh
@@ -2614,6 +2614,7 @@ func_all_modules ()
 
   func_begin_table
   func_module acl
+  func_module acl-permissions
   func_module areadlink
   func_module areadlink-with-size
   func_module areadlinkat
@@ -2654,6 +2655,9 @@ func_all_modules ()
   func_module openat
   func_module openat-die
   func_module pathmax
+  func_module qacl
+  func_module qcopy-acl
+  func_module qset-acl
   func_module read-file
   func_module readlinkat
   func_module same
diff --git a/modules/acl-permissions b/modules/acl-permissions
new file mode 100644
index 0000000..49b91ff
--- /dev/null
+++ b/modules/acl-permissions
@@ -0,0 +1,37 @@
+Description:
+Access control lists of files.  (Unportable.)
+
+Files:
+lib/acl.h
+lib/acl-internal.h
+lib/acl-errno-valid.c
+lib/acl_entries.c
+lib/acl-internal.c
+lib/get-permissions.c
+lib/set-permissions.c
+m4/acl.m4
+
+Depends-on:
+extern-inline
+fstat
+stdbool
+sys_stat
+
+configure.ac:
+gl_FUNC_ACL
+
+Makefile.am:
+lib_SOURCES += acl-errno-valid.c acl-internal.c \
+  get-permissions.c set-permissions.c
+
+Include:
+"acl.h"
+
+Link:
+$(LIB_ACL)
+
+License:
+GPL
+
+Maintainer:
+Paul Eggert, Jim Meyering
diff --git a/modules/file-has-acl b/modules/file-has-acl
index b602dc5..7b9b4f5 100644
--- a/modules/file-has-acl
+++ b/modules/file-has-acl
@@ -5,7 +5,7 @@ Files:
 lib/file-has-acl.c
 
 Depends-on:
-qacl  [test $gl_need_lib_has_acl]
+acl-permissions  [test $gl_need_lib_has_acl]
 
 configure.ac:
 gl_FILE_HAS_ACL
diff --git a/modules/qacl b/modules/qacl
index a0fb2ad..20c49ac 100644
--- a/modules/qacl
+++ b/modules/qacl
@@ -1,36 +1,21 @@
 Description:
 Access control lists of files.  (Unportable.)
+This is a placeholder module for backwards compatibility.
+New programs should use either qcopy-acl or qset-acl or both.
 
 Files:
-lib/acl.h
-lib/acl-internal.h
-lib/acl-errno-valid.c
-lib/acl_entries.c
-lib/acl-internal.c
-lib/get-permissions.c
-lib/qcopy-acl.c
-lib/qset-acl.c
-lib/set-permissions.c
-m4/acl.m4
 
 Depends-on:
-extern-inline
-fstat
-stdbool
-sys_stat
+qcopy-acl
+qset-acl
 
 configure.ac:
-gl_FUNC_ACL
 
 Makefile.am:
-lib_SOURCES += acl-errno-valid.c acl-internal.c qcopy-acl.c qset-acl.c \
-       get-permissions.c set-permissions.c
 
 Include:
-"acl.h"
 
 Link:
-$(LIB_ACL)
 
 License:
 GPL
diff --git a/modules/qcopy-acl b/modules/qcopy-acl
new file mode 100644
index 0000000..c0e5b6a
--- /dev/null
+++ b/modules/qcopy-acl
@@ -0,0 +1,25 @@
+Description:
+Copy access control list from one file to another.  (Unportable.)
+
+Files:
+lib/qcopy-acl.c
+
+Depends-on:
+acl-permissions
+
+configure.ac:
+
+Makefile.am:
+lib_SOURCES += qcopy-acl.c
+
+Include:
+"acl.h"
+
+Link:
+$(LIB_ACL)
+
+License:
+GPL
+
+Maintainer:
+Paul Eggert, Jim Meyering
diff --git a/modules/qset-acl b/modules/qset-acl
new file mode 100644
index 0000000..27fbc44
--- /dev/null
+++ b/modules/qset-acl
@@ -0,0 +1,25 @@
+Description:
+Set access control list of a file by specifying a mode.  (Unportable.)
+
+Files:
+lib/qset-acl.c
+
+Depends-on:
+acl-permissions
+
+configure.ac:
+
+Makefile.am:
+lib_SOURCES += qset-acl.c
+
+Include:
+"acl.h"
+
+Link:
+$(LIB_ACL)
+
+License:
+GPL
+
+Maintainer:
+Paul Eggert, Jim Meyering
-- 
2.1.0