From mboxrd@z Thu Jan 1 00:00:00 1970 Subject: Re: [PATCH] Fix includes for userspace tools and libraries (was Re: [PATCH] Fix include semanage/handle.h for semanage_set_root() as used by semodule) From: Stephen Smalley To: Eric Paris Cc: Guido Trentalancia , Eric Paris , SELinux Mail List In-Reply-To: <4E6E8149.30702@redhat.com> References: <1315587716.2170.16.camel@vortex> <1315588656.2170.26.camel@vortex> <1315832253.17035.5.camel@moss-pluto> <1315859373.2223.19.camel@vortex> <4E6E8149.30702@redhat.com> Content-Type: text/plain; charset="UTF-8" Date: Tue, 13 Sep 2011 08:41:37 -0400 Message-ID: <1315917697.12522.1.camel@moss-pluto> Mime-Version: 1.0 Sender: owner-selinux@tycho.nsa.gov List-Id: selinux@tycho.nsa.gov On Mon, 2011-09-12 at 18:01 -0400, Eric Paris wrote: > On 09/12/2011 04:29 PM, Guido Trentalancia wrote: > > Hi Stephen. > > > > On Mon, 2011-09-12 at 08:57 -0400, Stephen Smalley wrote: > >> On Fri, 2011-09-09 at 13:31 -0400, Eric Paris wrote: > >>> I'm also really annoyed by this at times, but I don't know what the > >>> right answer is. > >>> > >>> We really treat it like we have 5+ separate projects which just all > >>> happen to live in the same git tree. Thus to build libselinux you > >>> must have already built and installed libsepol. And then to build > >>> libsemanage you need to have done that with both libsepol and > >>> libselinux. Then of course to get the builds and install right (on > >>> Fedora/RHEL) you have to know to use LIBDIR=/usr/lib64 SHLIBDIR=/lib64 > >>> for libsepol and libsepol, but SHLIBDIR=/usr/lib64 for libsemanage. > >>> Its a mess, but I don't know what to do that's better.... > >> > >> make DESTDIR=~/out at top-level should work fine - it will build and > >> install each library into a private tree (as specified by DESTDIR) and > >> then build and install the programs against those libraries. Without > >> needing to clobber the system ones. > > > > At the moment, it does not, until the Makefile(s) are modified or at > > least until the libraries are manually installed first. > > > > Try by yourself, but do not forget to remove > > $(PREFIX)/usr/include/{selinux,semanage,sepol} and > > $(PREFIX){/usr,}/lib/libse{linux,manage,pol}* first ! > > I recently added the patch which changed the global to be 'build' > instead of 'install', so sds might not realize that 'install' is no > longer the default, however I think Guido is right about the install > having a number of paces that it hard coded things... Well, that's a regression then - it used to work with make DESTDIR=~/out just fine. -- Stephen Smalley 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.