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

Add man pages for example apps. #156

Open
wants to merge 1 commit into
base: master
from

Conversation

@fancycode
Copy link
Member

commented Sep 11, 2019

This is a first draft open for discussion.

@farindk

This comment has been minimized.

Copy link
Contributor

commented Sep 11, 2019

This is good, but what I always wanted to do is to clean-up these command line tools. The way they take arguments and options is not consistent. It even confuses myself. Maybe heif-convert and heif-enc could be combined into a single program. Thus, I am not sure we should publish the man pages before this is cleaned up.

@fancycode

This comment has been minimized.

Copy link
Member Author

commented Sep 11, 2019

Oh ok, I wasn't aware of that. Depending on when the refactoring of the command line tools will happen, it probably still makes sense to add man pages for what we have right now and change them later together with the tools.

@farindk

This comment has been minimized.

Copy link
Contributor

commented Sep 11, 2019

We can clean up the tools in the next release. I'm just a bit hesitant as I do not want to break any existing stuff.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.