-
Notifications
You must be signed in to change notification settings - Fork 44
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
Issue #492 - MacOS man #503
Conversation
@ilyash can you please also try the install both in your mac and linux in order to make sure everything works correctly? thanks :) |
As agreed during the weekly, please add tests so that correct installation is tested during/after the build on GitHub |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add tests
@ilyash-b added the tests directly in the makefile at b40e798. Do you have any better idea, or is it ok like this? Also noticed that this is breaking the docker build, since apparently inside docker:
Any idea for this? |
@ilyash-b changes applied, please re-review |
MacOS man by default is stored in
/usr/local/shared/man/man1
instead of/usr/local/man/man1