]> andersk Git - libyaml.git/commitdiff
build: do not install config.h
authorMarty E. Plummer <hanetzer@protonmail.com>
Tue, 9 Jan 2018 05:55:04 +0000 (23:55 -0600)
committerIngy döt Net <ingy@ingy.net>
Thu, 19 Jul 2018 21:49:53 +0000 (14:49 -0700)
'config.h' is meant to be a convenience header to be #included at build
time, but not installed. Installing it can cause a host of problems for
various other projects (for instance, attempting to build u-boot from
source while another project's 'config.h' exists in the compiler search
path will cause build failures similar to:
https://github.com/pepe2k/u-boot_mod/issues/148

Further, I've changed '#include <config.h>' to '#include "config.h"',
which should constrain the search path to the current build directories,
so if another package with a bugged build has this file installed, it
will not cause yaml to miscompile/fail.

If you have a file `/usr/include/config.h` on your filesystem, query
your package manager to find out what package owns it, and file a bug
report against it with your distro maintainers.

Signed-off-by: Marty E. Plummer <hanetzer@protonmail.com>
CMakeLists.txt
configure.ac
src/yaml_private.h

index 5e85aeffe125bf4a3a32f7202e46b0a82ac5e491..1abdd772b2aa7866aeea505f7ebc1a4b14b440d8 100644 (file)
@@ -83,7 +83,6 @@ target_include_directories(yaml PUBLIC
 install(
   FILES
     include/yaml.h
-    ${config_h}
   DESTINATION include COMPONENT Development
   )
 
index 637447fef64f03e8f10be1421e3f77283ba644f9..8139c77c9cac84765e077d1e68ccfc7832e4683a 100644 (file)
@@ -26,7 +26,7 @@ m4_define([YAML_AGE], 0)
 AC_PREREQ(2.59)
 AC_INIT([yaml], [YAML_MAJOR.YAML_MINOR.YAML_PATCH], [YAML_BUGS])
 AC_CONFIG_AUX_DIR([config])
-AC_CONFIG_HEADERS([config.h])
+AC_CONFIG_HEADERS([include/config.h])
 AM_INIT_AUTOMAKE([1.9 foreign])
 
 # Define macro variables for the package version numbers.
index d2971b8c1b5f0db16d6eedda220019ac36c0af52..b3351c41655c2b37306269565ac8a97c934defd0 100644 (file)
@@ -1,5 +1,5 @@
 #if HAVE_CONFIG_H
-#include <config.h>
+#include "config.h"
 #endif
 
 #include <yaml.h>
This page took 0.069082 seconds and 5 git commands to generate.