#930 closed defect (fixed)
don't delete printers we have not added
| Reported by: | Antoine Martin | Owned by: | Antoine Martin |
|---|---|---|---|
| Priority: | major | Milestone: | 0.16 |
| Component: | server | Version: | 0.15.x |
| Keywords: | printing | Cc: |
Description
We should detect when the lpadmin command fails and keep track of the printers we have successfully added so that we only remove those during cleanup.
This is an issue because if the same printer is plugged into 2 computers, each may have the same definition and since we don't add a prefix to the forwarded printer name, we will try to add the same name - which fails, but then we end up deleting the printer we did not create.
Change History (4)
comment:1 Changed 6 years ago by
| Status: | new → assigned |
|---|
comment:2 Changed 6 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Not backporting it because it is too big, focusing on getting 0.16 finished instead.
Closing.
comment:4 Changed 5 months ago by
this ticket has been moved to: https://github.com/Xpra-org/xpra/issues/930
Note: See
TracTickets for help on using
tickets.
copyleft 2010 - 2021
The first hurdle is that the reaper code somehow makes python unaware of the actual returncode from the lpadmin command, r10119 fixes that (should backport too).
r10120 takes care of not registering printers until the lpadmin command returns success (returncode=0).
A bit too big for backports, maybe later. (so keeping the ticket open)