Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

install is ignored in configure_file if install_dir is set #4160

Closed
3v1n0 opened this issue Sep 12, 2018 · 3 comments

Comments

Projects
None yet
3 participants
@3v1n0
Copy link
Contributor

commented Sep 12, 2018

If I have something like:

foreach pc: [PACKAGE_NAME, PACKAGE_NAME + '-uninstalled']
  configure_file(input: pc + '.pc.in',
    output: pc + '.pc',
    configuration: pc_conf,
    install: not pc.contains('-uninstalled'),
    install_dir: join_paths(libdir, 'pkgconfig'))
endforeach

The -uninstalled file is always installed, although install is set to false. The only way not to install it setting install_dir to [].

In executable instead this works as expected, while setting an empty path doesn't work if install is true.

@nirbheek

This comment has been minimized.

Copy link
Member

commented Sep 12, 2018

Duplicate of #3983

@nirbheek nirbheek marked this as a duplicate of #3983 Sep 12, 2018

@nirbheek nirbheek closed this Sep 12, 2018

@xclaesse

This comment has been minimized.

Copy link
Member

commented Nov 8, 2018

Unrelated to your issue, but you could be interested by this PR #4436 to generate -uninstalled.pc files.

@3v1n0

This comment has been minimized.

Copy link
Contributor Author

commented Nov 8, 2018

@xclaesse yeah, I was looking at it few minutes before you mentioned... Because yes I might need it.

Thanks :)

gnomesysadmins pushed a commit to GNOME/mutter that referenced this issue Feb 4, 2019

meson: Do not install cogl config.env if installed tests are disabled
Until meson 0.50, setting the install parameter in 'configure_file' is ignored
if 'install_dir' is set. Then until mutter doesn't depend on such meson version
cogl_installed_tests_libexecdir should be empty unless have_installed_tests is
false, or this file will be installed anyway.

See mesonbuild/meson#4160
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.