From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Petazzoni Date: Tue, 13 Nov 2018 21:22:29 +0100 Subject: [Buildroot] [PATCH] libsemanage: define installation location of semanage.conf for host In-Reply-To: <20181109215730.30795-1-thomas.petazzoni@bootlin.com> References: <20181109215730.30795-1-thomas.petazzoni@bootlin.com> Message-ID: <20181113212229.3254c6f4@windsurf> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: buildroot@busybox.net Hello, On Fri, 9 Nov 2018 22:57:30 +0100, Thomas Petazzoni wrote: > When /etc/selinux/semanage.conf does not exist, libsemanage tries to > overwrite it. For the target package, it works fine because $(DESTDIR) > is taken into account. > > However, for the host package, $(DESTDIR) is empty, and the location > used for /etc/selinux/semanage.conf is not affected by $(PREFIX). This > causes host-libsemanage to try to install /etc/selinux/semanage.conf, > which obviously fails with: > > test -f /etc/selinux/semanage.conf || install -m 644 -D semanage.conf /etc/selinux/semanage.conf > install: cannot create directory '/etc/selinux': Permission denied > > To fix this, this commit passes DEFAULT_SEMANAGE_CONF_LOCATION in the > make options when building/installing host-libsemanage, providing a > path to semanage.conf that Buildroot can write to. > > Fixes: > > http://autobuild.buildroot.net/results/cd27e3c66274622d0c3dd5a601a36efb1bc45011/ > > Signed-off-by: Thomas Petazzoni > --- > package/libsemanage/libsemanage.mk | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) Applied to master, thanks. Thomas -- Thomas Petazzoni, CTO, Bootlin Embedded Linux and Kernel engineering https://bootlin.com