From: James Carter <jwcart2@tycho.nsa.gov>
To: "Justin P. Mattock" <justinmattock@gmail.com>
Cc: Joshua Brindle <jbrindle@tresys.com>, SELinux <selinux@tycho.nsa.gov>
Subject: Re: Error building the userspace tools
Date: Thu, 07 Jan 2010 16:58:27 -0500 [thread overview]
Message-ID: <1262901507.26831.11.camel@localhost> (raw)
In-Reply-To: <4B44FE57.9050102@gmail.com>
On Wed, 2010-01-06 at 13:19 -0800, Justin P. Mattock wrote:
> On 01/06/10 12:37, James Carter wrote:
> > I don't know what the correct fix for this is, but when I make the
> > userspace tools with "make LIBDIR=/usr/lib64 SHLIBDIR=/lib64", I need
> > the patch below for everything to compile.
> >
> > What I don't know is:
> > 1) Why is this only required when I am specifying lib64?
> > 2) Since it is libsemanage that is using libustr and libbz2, why does
> > semodule and setsebool need to specify them?
> >
> > This occurs on Fedora 12. Is the problem due to a difference in the way
> > Fedora is building the userspace tools?
> >
> >
> >
> > diff --git a/policycoreutils/semodule/Makefile b/policycoreutils/semodule/Makefile
> > index c96a286..feba570 100644
> > --- a/policycoreutils/semodule/Makefile
> > +++ b/policycoreutils/semodule/Makefile
> > @@ -7,7 +7,7 @@ LIBDIR ?= ${PREFIX}/lib
> >
> > CFLAGS ?= -Werror -Wall -W
> > override CFLAGS += -I$(INCLUDEDIR)
> > -LDLIBS = -lsepol -lselinux -lsemanage -L$(LIBDIR)
> > +LDLIBS = -lustr -lbz2 -lsepol -lselinux -lsemanage -L$(LIBDIR)
> > SEMODULE_OBJS = semodule.o
> >
> > all: semodule
> > diff --git a/policycoreutils/setsebool/Makefile b/policycoreutils/setsebool/Makefile
> > index 556b780..214abc3 100644
> > --- a/policycoreutils/setsebool/Makefile
> > +++ b/policycoreutils/setsebool/Makefile
> > @@ -7,7 +7,7 @@ LIBDIR ?= ${PREFIX}/lib
> >
> > CFLAGS ?= -Werror -Wall -W
> > override CFLAGS += -I$(INCLUDEDIR)
> > -LDLIBS = -lsepol -lselinux -lsemanage -L$(LIBDIR)
> > +LDLIBS = -lustr -lbz2 -lsepol -lselinux -lsemanage -L$(LIBDIR)
> > SETSEBOOL_OBJS = setsebool.o
> >
> > all: setsebool
>
> I'm wondering if you
> commit 55648 is the reason
> for the error your getting.
>
> from over here your make commands
> look as it should.
>
Looking at it again. The problem is that when I specify LIBDIR and
SHLIBDIR, semodule and setsebool are linked against the static
libsemanage library.
An additional problem is that the creation of the link to
libsemanage.so.1 fails.
This patch seems to fix these two problems
diff --git a/libsemanage/src/Makefile b/libsemanage/src/Makefile
index 67afc60..137af40 100644
--- a/libsemanage/src/Makefile
+++ b/libsemanage/src/Makefile
@@ -127,7 +127,7 @@ install: all
test -d $(LIBDIR)/pkgconfig || install -m 755 -d $(LIBDIR)/pkgconfig
install -m 644 $(LIBPC) $(LIBDIR)/pkgconfig
test -f $(DEFAULT_SEMANAGE_CONF_LOCATION) || install -m 644 -D semanage.conf $(DEFAULT_SEMANAGE_CONF_LOCATION)
- cd $(LIBDIR) && ln -sf $(LIBSO) $(TARGET)
+ cd $(SHLIBDIR) && ln -sf $(LIBSO) $(TARGET)
install-pywrap: pywrap
test -d $(PYLIBDIR)/site-packages || install -m 755 -d $(PYLIBDIR)/site-packages
diff --git a/policycoreutils/semodule/Makefile b/policycoreutils/semodule/Makefile
index c96a286..1ab1f57 100644
--- a/policycoreutils/semodule/Makefile
+++ b/policycoreutils/semodule/Makefile
@@ -4,10 +4,11 @@ INCLUDEDIR ?= $(PREFIX)/include
SBINDIR ?= $(PREFIX)/sbin
MANDIR = $(PREFIX)/share/man
LIBDIR ?= ${PREFIX}/lib
+SHLIBDIR ?= $(PREFIX)/lib
CFLAGS ?= -Werror -Wall -W
override CFLAGS += -I$(INCLUDEDIR)
-LDLIBS = -lsepol -lselinux -lsemanage -L$(LIBDIR)
+LDLIBS = -lsepol -lselinux -lsemanage -L$(SHLIBDIR)
SEMODULE_OBJS = semodule.o
all: semodule
diff --git a/policycoreutils/setsebool/Makefile b/policycoreutils/setsebool/Makefile
index 556b780..17f96d9 100644
--- a/policycoreutils/setsebool/Makefile
+++ b/policycoreutils/setsebool/Makefile
@@ -4,10 +4,11 @@ INCLUDEDIR ?= $(PREFIX)/include
SBINDIR ?= $(PREFIX)/sbin
MANDIR = $(PREFIX)/share/man
LIBDIR ?= ${PREFIX}/lib
+SHLIBDIR ?= $(PREFIX)/lib
CFLAGS ?= -Werror -Wall -W
override CFLAGS += -I$(INCLUDEDIR)
-LDLIBS = -lsepol -lselinux -lsemanage -L$(LIBDIR)
+LDLIBS = -lsepol -lselinux -lsemanage -L$(SHLIBDIR)
SETSEBOOL_OBJS = setsebool.o
all: setsebool
--
James Carter <jwcart2@tycho.nsa.gov>
National Security Agency
--
This message was distributed to subscribers of the selinux mailing list.
If you no longer wish to subscribe, send mail to majordomo@tycho.nsa.gov with
the words "unsubscribe selinux" without quotes as the message.
next prev parent reply other threads:[~2010-01-07 21:58 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-01-06 20:37 Error building the userspace tools James Carter
2010-01-06 21:19 ` Justin P. Mattock
2010-01-07 21:58 ` James Carter [this message]
2010-01-08 3:08 ` Justin P. Mattock
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1262901507.26831.11.camel@localhost \
--to=jwcart2@tycho.nsa.gov \
--cc=jbrindle@tresys.com \
--cc=justinmattock@gmail.com \
--cc=selinux@tycho.nsa.gov \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.