Re: [PATCH 4/4] libsepol,libsemanage: write file name in flex output

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

 



On 11/28/2016 04:40 PM, Nicolas Iooss wrote:
> Tools like lcov (for code coverage) does not like files named
> "<stdout>". For example it reports errors like:
> 
>     genhtml: ERROR: cannot read
>     /usr/src/selinux/libsemanage/src/<stdout>
> 
> When using flex -o option, the output file name gets written in the
> generated C code, which solves this issue.
> 
> Signed-off-by: Nicolas Iooss <nicolas.iooss@xxxxxxx>

Thanks, applied all four.

> ---
>  libsemanage/src/Makefile | 2 +-
>  libsepol/src/Makefile    | 3 ++-
>  2 files changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/libsemanage/src/Makefile b/libsemanage/src/Makefile
> index ef25902bb019..df5809645a00 100644
> --- a/libsemanage/src/Makefile
> +++ b/libsemanage/src/Makefile
> @@ -101,7 +101,7 @@ semanageswig_python_exception.i: ../include/semanage/semanage.h
>  	bash -e exception.sh > $@ || (rm -f $@ ; false)
>  
>  conf-scan.c: conf-scan.l conf-parse.h
> -	$(LEX) $(LFLAGS) -t $< > $@
> +	$(LEX) $(LFLAGS) -o $@ $<
>  
>  conf-parse.c: conf-parse.y
>  	$(YACC) $(YFLAGS) -o $@ $<
> diff --git a/libsepol/src/Makefile b/libsepol/src/Makefile
> index 72724580eb98..497cd5675bd9 100644
> --- a/libsepol/src/Makefile
> +++ b/libsepol/src/Makefile
> @@ -67,7 +67,8 @@ $(CILDIR)/src/cil_lexer.lo: $(CILDIR)/src/cil_lexer.c
>  	$(CC) $(filter-out -Werror, $(CFLAGS)) -fPIC -DSHARED -c -o $@ $<
>  
>  $(CILDIR)/src/cil_lexer.c: $(CILDIR)/src/cil_lexer.l
> -	$(LEX) -t $< > $@
> +	$(LEX) -o $@ $<
> +
>  endif
>  
>  %.o:  %.c 
> 

_______________________________________________
Selinux mailing list
Selinux@xxxxxxxxxxxxx
To unsubscribe, send email to Selinux-leave@xxxxxxxxxxxxx.
To get help, send an email containing "help" to Selinux-request@xxxxxxxxxxxxx.



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

  Powered by Linux