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

0.8.0 manpage installs to the wrong place #1942

Closed
nekopsykose opened this issue Oct 7, 2023 · 0 comments · Fixed by #1943
Closed

0.8.0 manpage installs to the wrong place #1942

nekopsykose opened this issue Oct 7, 2023 · 0 comments · Fixed by #1943
Labels

Comments

@nekopsykose
Copy link
Contributor

nekopsykose commented Oct 7, 2023

Describe the bug

wayfire.1 is installed to both /usr/share/man and /usr/man

To Reproduce
install with manpages

Expected behavior
no manpage in /usr/man

Wayfire version
0.8.0

this is fixed by

--- a/man/meson.build
+++ b/man/meson.build
@@ -1,7 +1,5 @@
 configure_file(input: 'wayfire.1.in',
                output: 'wayfire.1',
-               install: true,
-               install_dir: 'man',
                configuration: conf_data)
 
 install_man(join_paths(meson.project_build_root(), 'man', 'wayfire.1'))

doing install:true on the configured file and also install_man installs the same thing twice. for the configure_file(), install_dir is relative to prefix. that's why it's /usr/man.

so just removing the install: properties works as expected as install_man already works.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant