Don't install the internal header grp-merge.h
authorZack Weinberg <zackw@panix.com>
Mon, 18 Jul 2016 12:33:21 +0000 (09:33 -0300)
committerAdhemerval Zanella <adhemerval.zanella@linaro.org>
Mon, 18 Jul 2016 12:33:21 +0000 (09:33 -0300)
grp-merge.h was introduced in Stephen Gallagher's patch adding the
"group merging" feature to NSS.  It declares two functions, __copy_grp
and __merge_grp, both of which are tagged 'internal_function', which
means that nobody can even compile the contents of the header without
access to libc-symbols.h, which is not installed.  (Also, these
functions are GLIBC_PRIVATE exports from libc.so.)  Hence I believe
grp-merge.h should not be installed either.

This really needs to be in 2.24, so that no released version of the
library installs this header.

I hope that what I did to the ChangeLog diff will allow it to be
applied without hassle.

* grp/Makefile: Don't install the internal header grp-merge.h.

ChangeLog
grp/Makefile

index 12a13cb..94b023c 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2016-07-18  Zack Weinberg  <zackw@panix.com>
+
+       * grp/Makefile: Don't install the internal header grp-merge.h.
+
 2016-07-18  Szabolcs Nagy  <szabolcs.nagy@arm.com>
 
        * sysdeps/aarch64/libm-test-ulps: Regenerated.
index b4d52e2..3807bfa 100644 (file)
@@ -22,7 +22,7 @@ subdir        := grp
 
 include ../Makeconfig
 
-headers := grp.h grp-merge.h
+headers := grp.h
 
 routines := fgetgrent initgroups setgroups \
            getgrent getgrgid getgrnam putgrent \