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

adding lpr --help option #5326

Closed
wants to merge 1 commit into from
Closed

adding lpr --help option #5326

wants to merge 1 commit into from

Conversation

zdohnal
Copy link
Contributor

@zdohnal zdohnal commented Jun 8, 2018

Hi Mike,

currently I'm doing another cleanup in CUPS on Fedora and I found some patches from which can help CUPS itself. This one is about adding --help parameter to lpr command.

Do you find it usable?

@michaelrsweet michaelrsweet self-assigned this Jun 8, 2018
@michaelrsweet michaelrsweet added the enhancement New feature or request label Jun 8, 2018
@michaelrsweet michaelrsweet added this to the CUPS 2.3.x milestone Jun 8, 2018
@michaelrsweet
Copy link
Collaborator

Maybe for 2.3.x (although the text will likely change to normalize the format with any other program help we have...)

@zdohnal
Copy link
Contributor Author

zdohnal commented Jun 11, 2018

Ok, I'll look into other helps for lp* commands and try to normalize it with them as example.

@michaelrsweet
Copy link
Collaborator

One of the things we do in other commands is to write out the options individually so that we have a few, common and short strings to localize rather than a bunch of large ones that are unique to every program.

@zdohnal
Copy link
Contributor Author

zdohnal commented Aug 15, 2018

I didn't forget about this issue - I'll get into it in near future, I hope :(

@michaelrsweet
Copy link
Collaborator

OK, I've pushed a full set of changes so that all user commands now support --help.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants