~emersion/public-inbox

This thread contains a patchset. You're looking at the original emails, but you may wish to use the patch review UI. Review patch
1

[PATCH libscfg] Install headers

Details
Message ID
<20211231020829.584812-1-me@tadeo.ca>
DKIM signature
missing
Download raw message
Patch: +2 -0
---
The libscfg header doesn't seem to get installed, this should fix
that.

I'm not sure whether install_subdir is the cleanest way to do this,
but it's what wlroots seems to use.

 meson.build | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/meson.build b/meson.build
index a82333e..7181bc3 100644
--- a/meson.build
+++ b/meson.build
@@ -19,6 +19,8 @@ add_project_arguments(cc.get_supported_arguments([
	'-Wno-unused-parameter',
]), language: 'c')

install_subdir('include', install_dir: get_option('includedir'))

scfg_inc = include_directories('include')

scfg_lib = library(
--
2.33.1
Details
Message ID
<elQt4_KyvxXbaWGd5RmMJDCxUGSmj92HcIf9Zj52qM2g1BiavC6CUSnL5M9ZTARMO8p_hpSDyo_Y7oIMqgMpfkzovkQPkRGaWbCOPDKPrmY=@emersion.fr>
In-Reply-To
<20211231020829.584812-1-me@tadeo.ca> (view parent)
DKIM signature
missing
Download raw message
Thanks for the patch! However, I think this will install include/ to
/usr/include/, resulting in a /usr/include/include/ subdir.

I think we can just use install_headers() instead since we have a single one
to install.
Reply to thread Export thread (mbox)