commit e42fb487839b242371b0150ab5b0b89c2d232976
author Thu Nguyen <thu@os.amperecomputing.com> Tue Oct 15 14:43:11 2024 +0000
committer Ed Tanous <ed@tanous.net> Wed Nov 13 18:03:29 2024 +0000
tree 8669013853aec03840c9e36089ac1c6989fd39fb
parent de7dd0618ea55c4e24a8dfbc70f339967c5cd474

Change include style to use system includes

The libcper header files in `libcper` are installed to
`usr/include/libcper`. Use that system includes in `libcper` source
instead of using the project includes.

Change-Id: I596edc2c754dae4829844f535a1e34caa246fb43
Signed-off-by: Thu Nguyen <thu@os.amperecomputing.com>

meson.build

diff --git a/meson.build b/meson.build
index acfb2d0..fb4a399 100644
--- a/meson.build
+++ b/meson.build
@@ -85,7 +85,9 @@
     'json-schema.c',
 )
 
-libcper_include = ['.']
+libcper_include = ['include']
+libcper_include_dir = include_directories(libcper_include, is_system: true)
+subdir('include')
 
 libcper_parse = library(
     'cper-parse',
@@ -93,14 +95,14 @@
     section_sources,
     edk_sources,
     version: meson.project_version(),
-    include_directories: include_directories(libcper_include),
+    include_directories: libcper_include_dir,
     c_args: '-Wno-address-of-packed-member',
     dependencies: [json_c_dep],
     install: true,
     install_dir: get_option('libdir'),
 )
 libcper_parse_dep = declare_dependency(
-    include_directories: include_directories(libcper_include),
+    include_directories: libcper_include_dir,
     link_with: libcper_parse,
 )
 
@@ -115,13 +117,13 @@
     libcper_generate_sources,
     generator_section_sources,
     version: meson.project_version(),
-    include_directories: include_directories(libcper_include),
+    include_directories: libcper_include_dir,
     dependencies: [libcper_parse_dep, json_c_dep],
     install: true,
     install_dir: get_option('libdir'),
 )
 libcper_generate_dep = declare_dependency(
-    include_directories: include_directories(libcper_include),
+    include_directories: libcper_include_dir,
     link_with: libcper_generate,
 )
 
@@ -132,25 +134,11 @@
     description: 'C bindings for parsing CPER',
 )
 
-install_headers(
-    'cper-parse.h',
-    'cper-parse-str.h',
-    'cper-utils.h',
-    'common-utils.h',
-    'Cper.h',
-    'Cper.c',
-    'BaseTypes.h',
-    subdir: 'libcper',
-    preserve_path : true
-)
-
-install_headers('generator/cper-generate.h', subdir: 'libcper/generator')
-
 if get_option('utility').allowed()
     executable(
         'cper-convert',
         'cli-app/cper-convert.c',
-        include_directories: include_directories(libcper_include),
+        include_directories: libcper_include_dir,
         dependencies: [libcper_parse_dep, json_c_dep],
         install: true,
         install_dir: get_option('bindir'),
@@ -160,7 +148,7 @@
         'cper-generate',
         'generator/cper-generate-cli.c',
         edk_sources,
-        include_directories: include_directories(libcper_include),
+        include_directories: libcper_include_dir,
         dependencies: [libcper_generate_dep],
         install: true,
         install_dir: get_option('bindir'),