Petri Hintukainen pushed to branch master at VideoLAN / libaacs


Commits:
ddd90a20 by Joshua Root at 2020-07-05T11:57:04+02:00
Fix include flags order

AM_CPPFLAGS is the correct variable to use for preprocessor flags. Importantly, 
it is placed on the compiler command line before CPPFLAGS, whereas AM_CFLAGS 
comes after CPPFLAGS. So before this change, it was possible for incorrect 
headers in external locations to be used instead of the internal ones specified 
in SET_INCLUDES.
- - - - -


1 changed file:

- Makefile.am


Changes:

=====================================
Makefile.am
=====================================
@@ -5,7 +5,8 @@ EXTRA_DIST=bootstrap COPYING KEYDB.cfg README.txt ChangeLog
 SET_FEATURES = @SET_FEATURES@
 SET_INCLUDES = -I$(top_srcdir)/src -I$(top_builddir)/src/libaacs
 
-AM_CFLAGS = -std=c99 $(SET_FEATURES) $(SET_INCLUDES) $(LIBGCRYPT_CFLAGS) 
$(GPG_ERROR_CFLAGS)
+AM_CFLAGS = -std=c99 $(LIBGCRYPT_CFLAGS) $(GPG_ERROR_CFLAGS)
+AM_CPPFLAGS = $(SET_FEATURES) $(SET_INCLUDES)
 AM_YFLAGS = -d -p libaacs_yy
 
 lib_LTLIBRARIES = libaacs.la



View it on GitLab: 
https://code.videolan.org/videolan/libaacs/-/commit/ddd90a2072188ee86657490d70a593fdb29f5570

-- 
View it on GitLab: 
https://code.videolan.org/videolan/libaacs/-/commit/ddd90a2072188ee86657490d70a593fdb29f5570
You're receiving this email because of your account on code.videolan.org.


_______________________________________________
libaacs-devel mailing list
libaacs-devel@videolan.org
https://mailman.videolan.org/listinfo/libaacs-devel

Reply via email to