-
-
Notifications
You must be signed in to change notification settings - Fork 48
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
List raw sockets during printers
subcommand if available
#340
base: master
Are you sure you want to change the base?
Conversation
printers
subcommand if availableprinters
subcommand if available
ff683a2
to
1d497ad
Compare
Provide a way how to show path to raw socket via CLI without 3rd part tools.
658e938
to
006c0ba
Compare
printers
subcommand if availableprinters
subcommand if available
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.
See comments. I'd also like to see this under a "--verbose" option so the default just lists the printer names.
} | ||
|
||
if (printer_name && printer_uri) | ||
printf("%s - printer - %s\n", printer_name, printer_uri); |
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.
Not keen on the format here - better to display the raw URL separately, e.g.:
PRINTER-NAME
ipp://HOSTNAME:PORT/ipp/print/PRINTER-NAME
ipps://HOSTNAME:PORT/ipp/print/PRINTER-NAME
socket://HOSTNAME:PORT
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.
Also, any text would need to be localized, so the above format avoids that particular issue...
|
||
if (printer->raw_active) | ||
{ | ||
if (!ra || cupsArrayFind(ra, "smi55357-printer-socket-uri-supported")) |
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.
How about "smi55357-socket-uri-configured"? You need to configure for raw printing support and we don't need to have "printer" in the name.
if (attr == NULL) | ||
break; | ||
|
||
value_tag = IPP_TAG_CUPS_INVALID; |
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.
I don't think you need to initialize a default attr_name
or value_tag
value. And printer-uri-supported is multi-valued so we might as well show all of the values.
Provide a way how to show path to raw socket via CLI without 3rd part tools.