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

The option of "-n COPIES" doesn't work #312

Closed
szlt5 opened this issue Nov 28, 2023 · 2 comments
Closed

The option of "-n COPIES" doesn't work #312

szlt5 opened this issue Nov 28, 2023 · 2 comments
Assignees
@michaelrsweet
Labels
bug Something isn't working priority-high
Milestone
Stable

Comments

@szlt5
Copy link

szlt5 commented Nov 28, 2023

Describe the bug
The option of "-n COPIES" doesn't work.

To Reproduce
Steps to reproduce the behavior:

  1. Run below command legacy-printer-app --help, below usage shows
...
Options:
-n COPIES    Specify number of copies (submit)
  1. Submit a file for printing with copies set as 2
legacy-printer-app submit -n 2 testpage.ps
  1. Only 1 copy is printed out.

Expected behavior
Two copies should be printed out.

Additional context

  1. If use below command to submit, two copies are printed out.
legacy-printer-app submit -o copies=2 testpage.ps
  1. In function papplMainloop(), the option '-n' is converted as "num-copies". Should the option '-n' be converted as "copies"?
@michaelrsweet michaelrsweet self-assigned this Nov 28, 2023
@michaelrsweet michaelrsweet added bug Something isn't working priority-high labels Nov 28, 2023
@michaelrsweet michaelrsweet added this to the Stable milestone Nov 28, 2023
@michaelrsweet
Copy link
Owner

Yes, it needs to just be "copies". I don't know why I wrote it as "num-copies"...

michaelrsweet added a commit that referenced this issue Nov 30, 2023
@michaelrsweet
Fix -n option to set number of copies (Issue #312)
88a67cf
michaelrsweet added a commit that referenced this issue Nov 30, 2023
@michaelrsweet
Fix -n option to set number of copies (Issue #312)
90b6496
@michaelrsweet
Copy link
Owner

[master 90b6496] Fix -n option to set number of copies (Issue #312)

[v1.4.x 88a67cf] Fix -n option to set number of copies (Issue #312)

@ribbons ribbons mentioned this issue Dec 20, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@michaelrsweet michaelrsweet

Labels
bug Something isn't working priority-high
Projects
None yet
Milestone
Stable
Development

No branches or pull requests
2 participants
@michaelrsweet @szlt5