From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from goalie.tycho.ncsc.mil (goalie [144.51.242.250]) by tarius.tycho.ncsc.mil (8.14.4/8.14.4) with ESMTP id t19H7I9b025871 for ; Mon, 9 Feb 2015 12:07:18 -0500 From: Paul Moore To: Stephen Smalley Subject: Re: [PATCH] selinux-testsuite: fix RHEL detection Date: Mon, 09 Feb 2015 12:07:12 -0500 Message-ID: <2227049.nfgmPr3pCp@sifl> In-Reply-To: References: <20150206223424.13392.21940.stgit@localhost> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Cc: selinux List-Id: "Security-Enhanced Linux \(SELinux\) mailing list" List-Post: List-Help: On Monday, February 09, 2015 11:13:02 AM Stephen Smalley wrote: > Doesn't RHEL have variants other than Server? Good point. I'll look into the other variants. > On Fri, Feb 6, 2015 at 5:34 PM, Paul Moore wrote: > > The redhat-release package is not consistently named across RHEL > > releases, use the /etc/redhat-release instead to detect the RHEL > > version. > > > > Signed-off-by: Paul Moore > > --- > > > > tests/Makefile | 9 +++++---- > > 1 file changed, 5 insertions(+), 4 deletions(-) > > > > diff --git a/tests/Makefile b/tests/Makefile > > index 3838e14..7c27787 100644 > > --- a/tests/Makefile > > +++ b/tests/Makefile > > @@ -1,18 +1,19 @@ > > -REDHAT_RELEASE=$(shell rpm -q redhat-release) > > +RHEL_VER=$(shell cat /etc/redhat-release) > > +RHEL_VER_PREFIX=Red Hat Enterprise Linux Server release > > > > SUBDIRS_COMMON:=domain_trans entrypoint execshare exectrace > > execute_no_trans fdreceive inherit link mkdir msg open ptrace readlink > > relabel rename rxdir sem setattr setnice shm sigkill stat sysctl > > task_create task_setnice task_setscheduler task_getscheduler task_getsid > > task_getpgid task_setpgid wait file ioctl capable_file capable_net > > capable_sys > > > > SUBDIRS:= $(SUBDIRS_COMMON) dyntrans dyntrace bounds nnp > > > > -ifeq (redhat-release-4, $(findstring redhat-release-4, > > $(REDHAT_RELEASE))) > > +ifeq ($(RHEL_VER_PREFIX)4, $(findstring $(RHEL_VER_PREFIX)4, > > $(RHEL_VER))) > > > > SUBDIRS:=$(SUBDIRS_COMMON) > > > > endif > > > > -ifeq (redhat-release-5, $(findstring redhat-release-5, > > $(REDHAT_RELEASE))) > > +ifeq ($(RHEL_VER_PREFIX)5, $(findstring $(RHEL_VER_PREFIX)5, > > $(RHEL_VER))) > > > > SUBDIRS:=$(SUBDIRS_COMMON) dyntrace dyntrans > > > > endif > > > > -ifeq (redhat-release-6, $(findstring redhat-release-6, > > $(REDHAT_RELEASE))) > > +ifeq ($(RHEL_VER_PREFIX)6, $(findstring $(RHEL_VER_PREFIX)6, > > $(RHEL_VER))) > > > > SUBDIRS:=$(SUBDIRS_COMMON) dyntrace dyntrans bounds > > > > endif > > > > _______________________________________________ > > Selinux mailing list > > Selinux@tycho.nsa.gov > > To unsubscribe, send email to Selinux-leave@tycho.nsa.gov. > > To get help, send an email containing "help" to > > Selinux-request@tycho.nsa.gov. -- paul moore security @ redhat