Hello Tomasz

On Fri, 01 Oct 2021 00:04:51 +0200 Tomasz Wolak <tomas.wo...@gmail.com> wrote:
> Package: libyaml-cpp-dev
> Version: 0.6.3-9
> Severity: normal
> X-Debbugs-Cc: tomas.wo...@gmail.com
> 
> One of the package config files for CMake:
> 
> /usr/lib/x86_64-linux-gnu/cmake/yaml-cpp/yaml-cpp-config.cmake
> 
> contains an incorrect path to the library's header files:
> 'include' instead of '/usr/include/yaml-cpp'.
> 
> I have managed to hot-fix this by correcting debian patches:
> 
> 1. install-cmake-dev-files.patch - the line (for CMakeLists.txt):
>  set(INCLUDE_INSTALL_ROOT_DIR include)
> was changed to:
> -set(INCLUDE_INSTALL_ROOT_DIR include)
> +set(INCLUDE_INSTALL_ROOT_DIR /usr/include)
> 
> 2. fix-pkg-config.patch, where I changed the line:
> +set(YAML_CPP_INCLUDE_DIR "@INCLUDE_INSTALL_ROOT_DIR@")
> to the following:
> +set(YAML_CPP_INCLUDE_DIR "@INCLUDE_INSTALL_DIR@")
> 
> After changing this, the rebuilt package contains the correct
> path to the header files (/usr/include/yaml-cpp) in its cmake
> configuration.
> 
> (I am not sure if this is the best way for fixing this but it seems
> reasonable. It should be reviewed and tested, of course.
> I am not sure how to provide you a patch to a Debian patch...
> that's why such descriptive form).
> 
> (Btw. version 0.6.3-10 seems to have the same problem).
> 
> 


can you please check if 0.7.0 in debian/experimental fixes this issue?
The cmake scripts have been reworked, and I can't see anymore the variable not 
correctly evaluated.

In case please let me know if the bug is fixed

thanks

Gianfranco

> -- System Information:
> Debian Release: 11.0
>   APT prefers stable-updates
>   APT policy: (500, 'stable-updates'), (500, 'stable-security'), (500, 
> 'oldstable-updates'), (500, 'stable'), (500, 'oldstable')
> Architecture: amd64 (x86_64)
> Foreign Architectures: i386
> 
> Kernel: Linux 5.10.0-8-amd64 (SMP w/4 CPU threads)
> Kernel taint flags: TAINT_PROPRIETARY_MODULE, TAINT_OOT_MODULE, 
> TAINT_UNSIGNED_MODULE
> Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8), 
> LANGUAGE=en_US:en
> Shell: /bin/sh linked to /bin/dash
> Init: systemd (via /run/systemd/system)
> LSM: AppArmor: enabled
> 
> Versions of packages libyaml-cpp-dev depends on:
> ii  libyaml-cpp0.6  0.6.3-9
> 
> libyaml-cpp-dev recommends no packages.
> 
> libyaml-cpp-dev suggests no packages.
> 
> -- no debconf information
> 

Reply via email to