Re: [PATCH] libsepol: Add INCLUDEDIR to utils/Makefile

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 2017-05-04 at 16:22 +0200, Petr Lautrbach wrote:
> The patch is wrong, please disregard.
> 
> I'm not sure about the right fix in order not to break gentoo use
> case.
> I'd just revert fcb5d5c change in libsepol/utils/Makefile for now.

Can't you just specify CFLAGS to make, similar to what is done in the
top-level Makefile for building with DESTDIR set?

> 
> 
> 
> On 05/04/2017 04:08 PM, Petr Lautrbach wrote:
> > fcb5d5c removed ../include from CFLAGS from libsepol/utils/Makefile
> > so
> > that a build tool can't find sepol/sepol.h when libsepol is built
> > on a
> > system without sepol.h in standard paths.
> > 
> > Fixes:
> > chkcon.c:1:10: fatal error: sepol/sepol.h: No such file or
> > directory
> >  #include <sepol/sepol.h>
> > 
> > Signed-off-by: Petr Lautrbach <plautrba@xxxxxxxxxx>
> > ---
> >  libsepol/utils/Makefile | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/libsepol/utils/Makefile b/libsepol/utils/Makefile
> > index a13164e1..f5097be7 100644
> > --- a/libsepol/utils/Makefile
> > +++ b/libsepol/utils/Makefile
> > @@ -1,6 +1,7 @@
> >  # Installation directories.
> >  PREFIX ?= $(DESTDIR)/usr
> >  BINDIR ?= $(PREFIX)/bin
> > +INCLUDEDIR ?= $(PREFIX)/include
> >  
> >  CFLAGS ?= -Wall -Werror
> >  LDFLAGS += -L../src
> > 
> 
> 



[Index of Archives]     [Selinux Refpolicy]     [Linux SGX]     [Fedora Users]     [Fedora Desktop]     [Yosemite Photos]     [Yosemite Camping]     [Yosemite Campsites]     [KDE Users]     [Gnome Users]

  Powered by Linux