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

Ensure the manpage location is correct. #86

Merged
merged 1 commit into from
Aug 22, 2024
Merged

Conversation

svin24
Copy link
Contributor

@svin24 svin24 commented Aug 13, 2024

For some reason the manpage is installed in the root directory as /man1.

I am still learning Cmake so if this is not good then don't merge but after testing it i can at least confirm it works.

For some reason the manpage is installed in the root directory.
@vaxerski vaxerski requested a review from fufexan August 13, 2024 17:22
Copy link
Member

@fufexan fufexan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm alright with this, though how are you building hyprpicker? There should be a default value for the MANDIR var.

@svin24
Copy link
Contributor Author

svin24 commented Aug 14, 2024

i just used

cmake -B build
cmake --build build

I don't really mind the application being installed in /usr/local/bin
it helps differentiate it from any potential system packages.

I just saw a manpage installed on my root and thought that can't be right.

I am not sure why other hyprland applications don't have this issue to be honest.

@fufexan fufexan merged commit 38fe668 into hyprwm:main Aug 22, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants