|
|
|
project('pkgconfig-gen', 'c', meson_version: '>=0.60.0')
|
|
|
|
|
|
|
|
# Some CI runners does not have zlib, just skip them as we need some common
|
|
|
|
# external dependency.
|
|
|
|
cc = meson.get_compiler('c')
|
|
|
|
if not cc.find_library('z', required: false).found()
|
|
|
|
error('MESON_SKIP_TEST: zlib missing')
|
|
|
|
endif
|
|
|
|
|
|
|
|
# First check we have pkg-config >= 0.29
|
treewide: internally avoid deprecated machine file uses of "pkgconfig"
We support this in a machine file:
```
[binaries]
pkgconfig = 'pkg-config'
pkg-config = 'pkg-config'
```
and you can use either one, because internally we look up both. If you
only set *one* of them, this plays awkwardly with setting $PKG_CONFIG,
since we don't know which one you set in the machine file and the
*other* one will be initialized from the environment instead.
In commit 22df45a31981874310a78dde0df59a6a7c5ebb29 we changed program
lookup of config-tool style dependencies to use the regular tool names
and only fall back on the strange internal names. This affected the
pkg-config class too.
The result is that instead of preferring `pkgconfig =` followed by
$PKG_CONFIG followed by `pkg-config =`, we inverted the lookup order.
This is a good idea anyway, because now it behaves consistently with
`find_program('pkg-config')`.
Unfortunately, we documented the wrong name in a bunch of places, and
also used the wrong name in various testsuite bits, which meant that if
you set $PKG_CONFIG and then ran the testsuite, it would fail.
Correct these references, because they are buggy.
One test case expected to find_program() a native copy for convenience
of testing against natively installed glib. Force it to resolve a native
copy.
1 year ago
|
|
|
pkgconfig = find_program('pkg-config', native: true, required: false)
|
|
|
|
if not pkgconfig.found()
|
|
|
|
error('MESON_SKIP_TEST: pkg-config not found')
|
|
|
|
endif
|
|
|
|
|
|
|
|
v = run_command(pkgconfig, '--version', check: true).stdout().strip()
|
|
|
|
if v.version_compare('<0.29')
|
|
|
|
error('MESON_SKIP_TEST: pkg-config version \'' + v + '\' too old')
|
|
|
|
endif
|
|
|
|
|
|
|
|
python = find_program('python3')
|
|
|
|
fs = import('fs')
|
|
|
|
pkgg = import('pkgconfig')
|
|
|
|
|
|
|
|
lib = shared_library('simple', 'simple.c')
|
|
|
|
libver = '1.0'
|
|
|
|
h = install_headers('simple.h')
|
|
|
|
|
|
|
|
pkgg.generate(
|
|
|
|
libraries : [lib, '-lz'],
|
|
|
|
subdirs : '.',
|
|
|
|
version : libver,
|
|
|
|
name : 'libsimple',
|
|
|
|
filebase : 'simple',
|
|
|
|
description : 'A simple demo library.',
|
|
|
|
requires : 'glib-2.0', # Not really, but only here to test that this works.
|
|
|
|
requires_private : ['gio-2.0', 'gobject-2.0'],
|
|
|
|
libraries_private : [lib, '-lz'],
|
|
|
|
)
|
|
|
|
|
|
|
|
env = environment()
|
|
|
|
env.prepend('PKG_CONFIG_PATH', meson.current_build_dir() / 'meson-private')
|
|
|
|
|
|
|
|
test(
|
|
|
|
'pkgconfig-validation',
|
|
|
|
pkgconfig,
|
|
|
|
args: ['--validate', 'simple'],
|
|
|
|
env : env,
|
|
|
|
)
|
|
|
|
|
Set RPATH for all non-system libs with absolute paths
If a pkg-config dependency has multiple libraries in it, which is the
most common case when it has a Requires: directive, or when it has
multiple -l args in Libs: (rare), then we don't add -Wl,-rpath
directives to it when linking.
The existing test wasn't catching it because it was linking to
a pkgconfig file with a single library in it. Update the test to
demonstrate this.
This function was originally added for shared libraries in the source
directory, which explains the name:
https://github.com/mesonbuild/meson/pull/2397
However, since now it is also used for linking to *all* non-system
shared libraries that we link to with absolute paths:
https://github.com/mesonbuild/meson/pull/3092
But that PR is incomplete / wrong, because only adding RPATHs for
dependencies that specify a single library, which is simply
inconsistent. Things will work for some dependencies and not work for
others, with no logical reason for it.
We should add RPATHs for *all* libraries. There are no special length
limits for RPATHs that I can find.
For ELF, DT_RPATH or DT_RUNPATH are used, which are just stored in
a string table (DT_STRTAB). The maximum length is only a problem when
editing pre-existing tags.
For Mach-O, each RPATH is stored in a separate LC_RPATH entry so there
are no length issues there either.
Fixes https://github.com/mesonbuild/meson/issues/9543
Fixes https://github.com/mesonbuild/meson/issues/4372
3 years ago
|
|
|
answerlib = shared_library('answer', 'answer.c')
|
|
|
|
|
|
|
|
pkgg.generate(answerlib,
|
|
|
|
name : 'libanswer',
|
|
|
|
description : 'An answer library.',
|
|
|
|
extra_cflags : ['-DLIBFOO'],
|
Set RPATH for all non-system libs with absolute paths
If a pkg-config dependency has multiple libraries in it, which is the
most common case when it has a Requires: directive, or when it has
multiple -l args in Libs: (rare), then we don't add -Wl,-rpath
directives to it when linking.
The existing test wasn't catching it because it was linking to
a pkgconfig file with a single library in it. Update the test to
demonstrate this.
This function was originally added for shared libraries in the source
directory, which explains the name:
https://github.com/mesonbuild/meson/pull/2397
However, since now it is also used for linking to *all* non-system
shared libraries that we link to with absolute paths:
https://github.com/mesonbuild/meson/pull/3092
But that PR is incomplete / wrong, because only adding RPATHs for
dependencies that specify a single library, which is simply
inconsistent. Things will work for some dependencies and not work for
others, with no logical reason for it.
We should add RPATHs for *all* libraries. There are no special length
limits for RPATHs that I can find.
For ELF, DT_RPATH or DT_RUNPATH are used, which are just stored in
a string table (DT_STRTAB). The maximum length is only a problem when
editing pre-existing tags.
For Mach-O, each RPATH is stored in a separate LC_RPATH entry so there
are no length issues there either.
Fixes https://github.com/mesonbuild/meson/issues/9543
Fixes https://github.com/mesonbuild/meson/issues/4372
3 years ago
|
|
|
)
|
|
|
|
|
|
|
|
# Test that name_prefix='' and name='libfoo' results in '-lfoo'
|
Set RPATH for all non-system libs with absolute paths
If a pkg-config dependency has multiple libraries in it, which is the
most common case when it has a Requires: directive, or when it has
multiple -l args in Libs: (rare), then we don't add -Wl,-rpath
directives to it when linking.
The existing test wasn't catching it because it was linking to
a pkgconfig file with a single library in it. Update the test to
demonstrate this.
This function was originally added for shared libraries in the source
directory, which explains the name:
https://github.com/mesonbuild/meson/pull/2397
However, since now it is also used for linking to *all* non-system
shared libraries that we link to with absolute paths:
https://github.com/mesonbuild/meson/pull/3092
But that PR is incomplete / wrong, because only adding RPATHs for
dependencies that specify a single library, which is simply
inconsistent. Things will work for some dependencies and not work for
others, with no logical reason for it.
We should add RPATHs for *all* libraries. There are no special length
limits for RPATHs that I can find.
For ELF, DT_RPATH or DT_RUNPATH are used, which are just stored in
a string table (DT_STRTAB). The maximum length is only a problem when
editing pre-existing tags.
For Mach-O, each RPATH is stored in a separate LC_RPATH entry so there
are no length issues there either.
Fixes https://github.com/mesonbuild/meson/issues/9543
Fixes https://github.com/mesonbuild/meson/issues/4372
3 years ago
|
|
|
lib2 = shared_library('libfoo', 'foo.c',
|
|
|
|
link_with: answerlib,
|
|
|
|
name_prefix : '',
|
|
|
|
version : libver)
|
|
|
|
|
Set RPATH for all non-system libs with absolute paths
If a pkg-config dependency has multiple libraries in it, which is the
most common case when it has a Requires: directive, or when it has
multiple -l args in Libs: (rare), then we don't add -Wl,-rpath
directives to it when linking.
The existing test wasn't catching it because it was linking to
a pkgconfig file with a single library in it. Update the test to
demonstrate this.
This function was originally added for shared libraries in the source
directory, which explains the name:
https://github.com/mesonbuild/meson/pull/2397
However, since now it is also used for linking to *all* non-system
shared libraries that we link to with absolute paths:
https://github.com/mesonbuild/meson/pull/3092
But that PR is incomplete / wrong, because only adding RPATHs for
dependencies that specify a single library, which is simply
inconsistent. Things will work for some dependencies and not work for
others, with no logical reason for it.
We should add RPATHs for *all* libraries. There are no special length
limits for RPATHs that I can find.
For ELF, DT_RPATH or DT_RUNPATH are used, which are just stored in
a string table (DT_STRTAB). The maximum length is only a problem when
editing pre-existing tags.
For Mach-O, each RPATH is stored in a separate LC_RPATH entry so there
are no length issues there either.
Fixes https://github.com/mesonbuild/meson/issues/9543
Fixes https://github.com/mesonbuild/meson/issues/4372
3 years ago
|
|
|
pkgg.generate(lib2,
|
|
|
|
libraries : [lib2, answerlib],
|
|
|
|
name : 'libfoo',
|
|
|
|
version : libver,
|
|
|
|
description : 'A foo library.',
|
|
|
|
variables : ['foo=bar', 'datadir=${prefix}/data'],
|
|
|
|
extra_cflags : ['-DLIBFOO'],
|
|
|
|
)
|
|
|
|
|
|
|
|
pkgg.generate(
|
|
|
|
name : 'libhello',
|
|
|
|
description : 'A minimalistic pkgconfig file.',
|
|
|
|
version : libver,
|
|
|
|
)
|
|
|
|
|
|
|
|
pkgg.generate(
|
|
|
|
name : 'libhello_nolib',
|
|
|
|
description : 'A minimalistic pkgconfig file.',
|
|
|
|
version : libver,
|
|
|
|
dataonly: true,
|
|
|
|
variables : {
|
|
|
|
'foo': 'bar',
|
|
|
|
# prefix is not set by default for dataonly pc files, but it is allowed to
|
|
|
|
# define it manually.
|
|
|
|
'prefix': get_option('prefix'),
|
|
|
|
'escaped_var': 'hello world',
|
|
|
|
},
|
|
|
|
unescaped_variables: {
|
|
|
|
'unescaped_var': 'hello world',
|
|
|
|
}
|
|
|
|
)
|
|
|
|
|
|
|
|
# Regression test for 2 cases:
|
|
|
|
# - link_whole from InternalDependency used to be ignored, but we should still
|
|
|
|
# recurse to add libraries they link to. In this case it must add `-lsimple1`
|
|
|
|
# in generated pc file.
|
|
|
|
# - dependencies from InternalDependency used to be ignored. In this it must add
|
|
|
|
# `-lz` in generated pc file.
|
|
|
|
simple1 = shared_library('simple1', 'simple.c')
|
|
|
|
stat1 = static_library('stat1', 'simple.c', link_with: simple1)
|
|
|
|
dep = declare_dependency(link_whole: stat1, dependencies: cc.find_library('z'))
|
|
|
|
simple2 = library('simple2', 'simple.c')
|
|
|
|
pkgg.generate(simple2, libraries: dep)
|
|
|
|
|
|
|
|
# Regression test: as_system() does a deepcopy() of the InternalDependency object
|
|
|
|
# which caused `-lsimple3` to be duplicated because generator used to compare
|
|
|
|
# Target instances instead of their id.
|
|
|
|
simple3 = shared_library('simple3', 'simple.c')
|
|
|
|
dep1 = declare_dependency(link_with: simple3)
|
|
|
|
dep2 = dep1.as_system()
|
|
|
|
pkgg.generate(libraries: [dep1, dep2],
|
|
|
|
name: 'simple3',
|
|
|
|
description: 'desc')
|
|
|
|
|
|
|
|
# Regression test: stat2 is both link_with and link_whole, it should not appear
|
|
|
|
# in generated pc file.
|
|
|
|
stat2 = static_library('stat2', 'simple.c', install: true)
|
|
|
|
simple4 = library('simple4', 'simple.c', link_with: stat2)
|
|
|
|
simple5 = library('simple5', 'simple5.c', link_with: simple4, link_whole: stat2)
|
|
|
|
pkgg.generate(simple5)
|
|
|
|
|
|
|
|
# Test passing a linkable CustomTarget and CustomTargetIndex to generator.
|
|
|
|
# Do this only with gcc/clang to not have to deal with other compiler command
|
|
|
|
# line specificities.
|
|
|
|
if cc.get_id() in ['gcc', 'clang']
|
|
|
|
ct = custom_target('ct',
|
|
|
|
input: 'simple.c',
|
|
|
|
output: 'libct.so',
|
|
|
|
command: [cc.cmd_array(), '@INPUT@', '-shared', '-o', '@OUTPUT@'],
|
|
|
|
)
|
|
|
|
pkgg.generate(libraries: ct,
|
|
|
|
name: 'ct',
|
|
|
|
description: 'custom target'
|
|
|
|
)
|
|
|
|
pkgg.generate(libraries: ct[0],
|
|
|
|
name: 'ct0',
|
|
|
|
description: 'custom target index'
|
|
|
|
)
|
|
|
|
endif
|
|
|
|
|
|
|
|
# Regression test: A library linking to an uninstalled custom_target static
|
|
|
|
# library used to crash when generating its pkgconfig file.
|
|
|
|
# Copy libstat2.a to libstat3.a to have a static library as custom target.
|
|
|
|
infile = stat2.full_path()
|
|
|
|
outfile = meson.current_build_dir() / 'libstat3.a'
|
|
|
|
script = 'import shutil ; shutil.copyfile("@0@", "@1@")'.format(infile, outfile)
|
|
|
|
ct = custom_target('stat3',
|
|
|
|
input: stat2,
|
|
|
|
output: fs.name(outfile),
|
|
|
|
command: [python, '-c', script],
|
|
|
|
)
|
|
|
|
simple6 = library('simple6', 'dependencies/dummy.c', link_with: ct)
|
|
|
|
pkgg.generate(simple6)
|
|
|
|
|
|
|
|
# implicit variables
|
|
|
|
pkgg.generate(
|
|
|
|
name : 'libvartest',
|
|
|
|
description : 'Check that implicit vars are created',
|
|
|
|
version : libver,
|
|
|
|
variables: ['datadir=${prefix}/data', 'foo=${datadir}/foo', 'bar=${bindir}/bar']
|
|
|
|
)
|
|
|
|
pkgg.generate(
|
|
|
|
name : 'libvartest2',
|
|
|
|
description : 'Check that libdir is not an implicit var',
|
|
|
|
version : libver,
|
|
|
|
variables: ['bar=${libdir}/bar']
|
|
|
|
)
|
|
|
|
|
|
|
|
# Regression test: variables kwarg should listify single string value
|
|
|
|
pkgg.generate(
|
|
|
|
name : 'libvartest3',
|
|
|
|
description : 'Check that variables can be single string',
|
|
|
|
variables: 'foo=bar',
|
|
|
|
)
|
|
|
|
|
|
|
|
# without a mainlib, name/description are mandatory
|
|
|
|
testcase expect_error('pkgconfig.generate: if a library is not passed as a positional argument, the \'name\' keyword argument is required.')
|
|
|
|
pkgg.generate(description: 'empty data')
|
|
|
|
endtestcase
|
|
|
|
|
|
|
|
testcase expect_error('pkgconfig.generate: if a library is not passed as a positional argument, the \'description\' keyword argument is required.')
|
|
|
|
pkgg.generate(name: 'foobar')
|
|
|
|
endtestcase
|
|
|
|
|
|
|
|
# Make sure the -uninstalled.pc file contains both include directories.
|
|
|
|
# See dependencies/test2.c that gets built against both simple7.pc and
|
|
|
|
# simple7-uninstalled.pc.
|
|
|
|
simple7 = library('simple7', include_directories: 'inc1')
|
|
|
|
dep = declare_dependency(include_directories: 'inc2')
|
|
|
|
install_headers('inc1/inc1.h', 'inc2/inc2.h')
|
|
|
|
pkgg.generate(simple7, libraries: dep)
|