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

lpadmin: separate enable and encrypt option? #5308

Closed
senatormailman opened this issue May 2, 2018 · 3 comments
Closed

lpadmin: separate enable and encrypt option? #5308

senatormailman opened this issue May 2, 2018 · 3 comments
Assignees
Labels
wontfix This will not be worked on

Comments

@senatormailman
Copy link

I can't see any previous issues regarding this, but this seems to have been the case for a bit. I understand that doing 'lpadmin -p $Printername -E' will enable the printer queue that had just been created while 'lpadmin -E -p $Printername' will just create the printer queue but the connection will be encrypted.

Is there a reason for this? Would separating the encrypt and enable options (to something like -E for enable and -e for encrypt) be breaking any kind of functionality?

Thanks for all the work!

@michaelrsweet
Copy link
Collaborator

@senatormailman This is a historical issue dating back to CUPS 1.1.x when we added support for encryption - lpadmin already supported a "-E" option so we opted to overload -E (-E before -p = use encryption, -E after -p = enable-and-accept) rather than use a different "use encryption" option for lpadmin vs. every other command (where -E means use encryption only).

"-e" actually is already used for System V lpadmin (-e copies the named printer to the new one) so we couldn't use that.

@michaelrsweet michaelrsweet added wontfix This will not be worked on Common Issue labels May 2, 2018
@michaelrsweet michaelrsweet self-assigned this May 2, 2018
@senatormailman
Copy link
Author

Ah, that makes sense. Thanks again!

@JackGrinningCat
Copy link

JackGrinningCat commented Aug 7, 2018

I believe those html files are autocreated but here is an PR to improve the documentation.
#5369

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

3 participants