From 1c187d797fc5bddcd29ce3e01d911ad36153ba15 Mon Sep 17 00:00:00 2001 From: Nicolas Iooss <nicolas.iooss@m4x.org> Date: Mon, 28 Nov 2016 22:40:17 +0100 Subject: [PATCH] checkpolicy: remove -lfl from LDLIBS When building checkpolicy/test, the linker reports the following error: cc dispol.o -lfl /usr/src/selinux/DESTDIR/usr/lib/libsepol.a -L/usr/src/selinux/DESTDIR/usr/lib -o dispol /usr/lib/gcc/x86_64-pc-linux-gnu/6.2.1/../../../../lib/libfl.so: undefined reference to `yylex' collect2: error: ld returned 1 exit status According to flex documentation (https://github.com/westes/flex/blob/master/doc/flex.texi), -lfl is used to provide an implementation for yywrap(). However every flex file now uses "%option noyywrap", which makes -lfl no longer mandatory. Remove this option from checkpolicy Makefiles. Signed-off-by: Nicolas Iooss <nicolas.iooss@m4x.org> --- checkpolicy/Makefile | 2 +- checkpolicy/test/Makefile | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/checkpolicy/Makefile b/checkpolicy/Makefile index 53a30745..875cf008 100644 --- a/checkpolicy/Makefile +++ b/checkpolicy/Makefile @@ -20,7 +20,7 @@ CHECKOBJS = y.tab.o lex.yy.o queue.o module_compiler.o parse_util.o \ CHECKPOLOBJS = $(CHECKOBJS) checkpolicy.o CHECKMODOBJS = $(CHECKOBJS) checkmodule.o -LDLIBS=$(LIBDIR)/libsepol.a -lfl +LDLIBS=$(LIBDIR)/libsepol.a GENERATED=lex.yy.c y.tab.c y.tab.h diff --git a/checkpolicy/test/Makefile b/checkpolicy/test/Makefile index c2367e19..e7bd717a 100644 --- a/checkpolicy/test/Makefile +++ b/checkpolicy/test/Makefile @@ -9,7 +9,7 @@ INCLUDEDIR ?= $(PREFIX)/include CFLAGS ?= -g -Wall -W -Werror -O2 -pipe override CFLAGS += -I$(INCLUDEDIR) -LDLIBS=-lfl $(LIBDIR)/libsepol.a -L$(LIBDIR) +LDLIBS=$(LIBDIR)/libsepol.a -L$(LIBDIR) all: dispol dismod