Add support for absolute CMAKE_INSTALL_*DIR

[otegami]

Problem

The Current CMakeLists incorrectly configures libdir and includedir when CMAKE_INSTALL_LIBDIR and CMAKE_INSTALL_INCLUDEDIR are absolute paths. And then it leads to incorrect paths in the generated pkg-config file.
This misconfiguration prevents the compiler from finding necessary header files and library files.

How to reproduce

% git switch -c c_master
% cmake -S . -B ../msgpack-c.build -DCMAKE_INSTALL_PREFIX=/tmp/local
% cmake --build ../msgpack-c.build
% cmake --install ../msgpack-c.build
% export PKG_CONFIG_PATH=/tmp/local/lib/pkgconfig:$PKG_CONFIG_PATH
% gcc -o simple_c example/simple_c.c $(pkg-config --cflags --libs msgpack-c)
In file included from include/msgpack/object.h:13,
                 from include/msgpack.h:17,
                 from example/simple_c.c:1:
include/msgpack/zone.h:13:10: fatal error: sysdep.h: No such file or directory
   13 | #include "sysdep.h"
      |          ^~~~~~~~~~
compilation terminated.`

Expected

% gcc -o simple_c example/simple_c.c $(pkg-config --cflags --libs msgpack-c)
% ./simple_c
93 01 c3 a7 65 78 61 6d 70 6c 65
[1, true, "example"]

Solution

Adjust the CMakeLists.txt to correctly handle both relative and absolute paths for library and include directories. Update the msgpack-c.pc.in file to use proper variables that respect the absolute path settings.

[otegami]

exec_prefix は利用されていないかつ、どちらもCMAKE_INSTALL_PREFIXを参照ているので必要ないと判断しました。

[kou]

https://github.com/otegami/msgpack-c/pull/1/files#diff-1e7de1ae2d059d21e1dd75d5812d5a34b0222cef273b7c3a2af62eb747f9d20aR34 で参照しているのでこれだと壊れない？

[kou]

自分で「How to reproduce」にまとめている通り、CMAKE_INSTALL_LIBDIRとかに絶対パスを指定していなくても壊れています。
pkg-config --cflags --libs msgpack-cで出てくるパスが相対パスになっているからです。

[kou]

まずは、以下のケースに対応して

• 何も指定しない
• CMAKE_INSTALL_INCLUDEDIR/CMAKE_INSTALL_LIBDIRに相対パスを指定（-DCMAKE_INSTALL_LIBDIR=lib64とか）

それが終わってから以下のケースに対応するのがいいんじゃないかな。

• CMAKE_INSTALL_INCLUDEDIR/CMAKE_INSTALL_LIBDIRに絶対パスを指定（-DCMAKE_INSTALL_LIBDIR=/tmp/local/lib/x86_64-linux-gnuとか）

[kou]

前者は↓で動くようになりませんか？

diff --git a/msgpack-c.pc.in b/msgpack-c.pc.in
index 20806625..14806b7b 100644
--- a/msgpack-c.pc.in
+++ b/msgpack-c.pc.in
@@ -1,7 +1,7 @@
 prefix=@prefix@
 exec_prefix=@exec_prefix@
-libdir=@libdir@
-includedir=@includedir@
+libdir=${prefix}/@libdir@
+includedir=${prefix}/@includedir@
 
 Name: MessagePack
 Description: Binary-based efficient object serialization library

[otegami]

レビューありがとうございます。
確かにいきなり絶対パスを指定対応するのも唐突なので分けて対応していきたいと思います。

• 何も指定しない && 相対パスの対応
• 絶対パスの対応

[otegami]

何も指定しない && 相対パスの対応

下記で完了したので、次は、絶対パスの対応をしたい。

• Support relative path for CMAKE_INSTALL_*DIR msgpack/msgpack-c#1119