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
khal: build and install man page #66962
Conversation
install -D misc/__khal $out/share/zsh/site-functions/__khal | ||
|
||
# man page | ||
make -C doc man |
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.
Makefile doesn't support install for this?
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.
doesn't seem like it https://github.com/pimutils/khal/blob/master/doc/Makefile
While at it maybe you could install https://github.com/pimutils/khal/blob/master/misc/khal.desktop else I will do it in a follow up PR |
On Mon, 19 Aug 2019 22:07:52 -0700, Matthieu Coudron ***@***.***> wrote:
While at it maybe you could install https://github.com/pimutils/khal/blob/master/misc/khal.desktop else I will do it in a follow up PR
I started, the following addition to postInstall should do the trick:
```
# desktop
install -Dm755 misc/khal.desktop -t $out/share/applications
```
This doesn't work as-is however, since the current khal release doesn't
have the `.desktop` file.
Looks like someone we know was behind the PR adding it! ;)
Accordingly I may let you tackle this. I'll give it a go..
Maybe just fetchpatch the commit (in addition to snippet above),
drop the patch when bumping khal?
…
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
#66962 (comment) part: text/html
|
let's wait for the next release then |
Oops, I went ahead and added it 😇. Works here! \o/ |
Thanks. Works for me. |
Motivation for this change
Man page is useful to have!
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @gebner