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.

Sign up for GitHub

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

Jump to bottom

WIP: Add manpage #446

Closed
wants to merge 4 commits into from
Closed

WIP: Add manpage #446

wants to merge 4 commits into from

Conversation

JoeLametta
Copy link
Collaborator

This manpage is based on morituri's one (Debian version).
See: https://manpages.debian.org/jessie/morituri/rip.1

This is still WIP.

Fixes #73.

@JoeLametta
WIP: Add manpage
a4fd28d 
This manpage is based on morituri's one (Debian version).
See: https://manpages.debian.org/jessie/morituri/rip.1

Fixes #73.

Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
@JoeLametta
Improve manpage
3f89c76 
Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
@JoeLametta
Remove "profile" option
c531fd0 
Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
@JoeLametta
Misc changes
f5e301e 
- Improved formatting
- Added new whipper cd rip options

Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
@JoeLametta
Copy link
Collaborator Author

@MerlijnWajer @Freso what's your opinion about this: is it garbage or good as a starting point?

@MerlijnWajer
Copy link
Collaborator

I'd personally write one in restructuredtext and use rst2man to generate one.

@JoeLametta
Copy link
Collaborator Author

I'd personally write one in restructuredtext and use rst2man to generate one.

OK, do you have a draft ready?

jwflory
jwflory approved these changes Apr 16, 2020
View reviewed changes
Copy link

@jwflory jwflory left a comment

Choose a reason for hiding this comment

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

From the P.O.V. of a downstream Linux packager, it would be easy for a Linux packager to use this manpage to bundle documentation on most distros. I was hoping to see manpages in the Fedora package. I could help get these into Fedora if upstream ships a manpage 🙂

@baldurmen baldurmen mentioned this pull request May 28, 2020
Closed
@baldurmen
Copy link
Contributor

I'm happy to take over this PR and use rst2man, as I've used it in the past for other projects using cli clients with sub-commands :)

@baldurmen baldurmen mentioned this pull request May 29, 2020
Merged
@JoeLametta
Copy link
Collaborator Author

@baldurmen Thanks for the effort: it is surely appreciated!

@JoeLametta
Copy link
Collaborator Author

Closed because of #490.

@JoeLametta JoeLametta closed this May 30, 2020
@JoeLametta JoeLametta deleted the feature/issue-73-add-manpage branch May 30, 2020 09:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jwflory jwflory jwflory approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

add manpage
4 participants
@JoeLametta @MerlijnWajer @baldurmen @jwflory