From adf5ea13347aee4227e1253a5fcb68d9a3dcfacd Mon Sep 17 00:00:00 2001 From: Simon Wilkinson Date: Sat, 11 Sep 2010 12:43:35 +0100 Subject: [PATCH] Linux: Move keyring includes where they're needed We don't need the keyring headers in every file, so reduce namespace pollution by just including them in osi_groups.c, which is the only place that uses them. Reviewed-on: http://gerrit.openafs.org/2746 Reviewed-by: Derrick Brashear Tested-by: Derrick Brashear (cherry picked from commit 33ce7a9308c8a71fbfcea82a935e32521031f48d) Change-Id: Ia943179a23c856ef9d37d1774192389cfe38612a Reviewed-on: http://gerrit.openafs.org/4067 Tested-by: BuildBot Reviewed-by: Derrick Brashear --- src/afs/LINUX/osi_groups.c | 11 +++++++++++ src/afs/sysincludes.h | 10 ---------- 2 files changed, 11 insertions(+), 10 deletions(-) diff --git a/src/afs/LINUX/osi_groups.c b/src/afs/LINUX/osi_groups.c index c3515ad1a..baaff78cc 100644 --- a/src/afs/LINUX/osi_groups.c +++ b/src/afs/LINUX/osi_groups.c @@ -28,6 +28,17 @@ #include +#if defined(LINUX_KEYRING_SUPPORT) +# include +# include +# if defined(HAVE_LINUX_KEY_TYPE_H) +# include +# endif +# ifndef KEY_ALLOC_IN_QUOTA +# define KEY_ALLOC_IN_QUOTA 1 +# endif +#endif + #ifdef AFS_LINUX26_ONEGROUP_ENV # define NUMPAGGROUPS 1 diff --git a/src/afs/sysincludes.h b/src/afs/sysincludes.h index 56a1daa5c..f7abc2e46 100644 --- a/src/afs/sysincludes.h +++ b/src/afs/sysincludes.h @@ -135,16 +135,6 @@ # include # endif # include -# if defined(LINUX_KEYRING_SUPPORT) -# include -# include -# if defined(HAVE_LINUX_KEY_TYPE_H) -# include -# endif -# ifndef KEY_ALLOC_IN_QUOTA -# define KEY_ALLOC_IN_QUOTA 1 -# endif -# endif # endif /* Avoid conflicts with coda overloading AFS type namespace. Must precede * inclusion of uaccess.h. -- 2.39.5