foomatic-rip did not catch or ignore signal 13 |
|||||
Issue descriptionChrome Version: 60 OS: ChromeOS What steps will reproduce the problem? (1) Load PPD for Ricoh-MP_C3503 http://www.openprinting.org/ppd-o-matic.php?driver=pxlcolor-Ricoh&printer=Ricoh-MP_C3503&show=0 (2) Setup Printer (3) Press Print What is the expected result? Print job fails in error because the PPD depends on perl and we don't install Perl What happens instead? Erroneous report of a successful print job. Please use labels and text to provide additional information. For graphics-related bugs, please copy/paste the contents of the about:gpu page at the end of this report.
,
Aug 30 2017
,
Sep 27 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/overlays/portage-stable/+/f6b430772a44507b37614dda9936fba19890ceec commit f6b430772a44507b37614dda9936fba19890ceec Author: Luum Habtemariam <luum@google.com> Date: Wed Sep 27 19:58:02 2017 foomatic-rip failure status fix Changes execution of renderer thread in the foomatic-rip package to fail whenever any of its individual sub-commands fails Global SIGPIPE signal handler now only pertains to parent process. Children will still fail and parent process will now notice and gracefully exit accordingly. BUG= chromium:726758 TEST=Printed job and foomatic now reports correct error code Change-Id: I9e60ec3edde800a8451cdc1e48c4204ae94ca526 Reviewed-on: https://chromium-review.googlesource.com/673026 Commit-Ready: Luum Habtemariam <luum@chromium.org> Tested-by: Luum Habtemariam <luum@chromium.org> Reviewed-by: Chirantan Ekbote <chirantan@chromium.org> Reviewed-by: Sean Kau <skau@chromium.org> [rename] https://crrev.com/f6b430772a44507b37614dda9936fba19890ceec/net-print/cups-filters/cups-filters-1.8.2-r4.ebuild [add] https://crrev.com/f6b430772a44507b37614dda9936fba19890ceec/net-print/cups-filters/files/cups-filters-1.8.2-foomatic-rip-sig-pipe.patch
,
Sep 27 2017
per patch above
,
Oct 7 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/44be13b1e1367b4f4d815e716b1de92ef54b7aa2 commit 44be13b1e1367b4f4d815e716b1de92ef54b7aa2 Author: Luum Habtemariam <luum@chromium.org> Date: Sat Oct 07 00:54:59 2017 Cups filter/pipeline failure fix Consider cups pipeline failing as an error and kill corresponding print job BUG= chromium:726758 Change-Id: Icacca129be13d97631dda5ccd9c9a7d9e6ee9da4 Reviewed-on: https://chromium-review.googlesource.com/673723 Reviewed-by: Sean Kau <skau@chromium.org> Commit-Queue: Luum Habtemariam <luum@chromium.org> Cr-Commit-Position: refs/heads/master@{#507250} [modify] https://crrev.com/44be13b1e1367b4f4d815e716b1de92ef54b7aa2/chrome/browser/chromeos/printing/cups_print_job.cc [modify] https://crrev.com/44be13b1e1367b4f4d815e716b1de92ef54b7aa2/chrome/browser/chromeos/printing/cups_print_job.h [modify] https://crrev.com/44be13b1e1367b4f4d815e716b1de92ef54b7aa2/chrome/browser/chromeos/printing/cups_print_job_manager_impl.cc [modify] https://crrev.com/44be13b1e1367b4f4d815e716b1de92ef54b7aa2/tools/metrics/histograms/enums.xml
,
Oct 7 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/overlays/portage-stable/+/8d736c049685ddfe70f34e38a8ce406605750153 commit 8d736c049685ddfe70f34e38a8ce406605750153 Author: Luum Habtemariam <luum@chromium.org> Date: Sat Oct 07 04:31:13 2017 net-print/cups-filters: Added patch description Added description to cups-filters-1.8.2-foomatic-rip-sig-pipe.patch BUG= chromium:726758 TEST=manually confirmed patch behavior unchanged Change-Id: I41753ba72245e3f34b2c5dd60afb2a7af56325ad Reviewed-on: https://chromium-review.googlesource.com/689741 Commit-Ready: Luum Habtemariam <luum@chromium.org> Tested-by: Luum Habtemariam <luum@chromium.org> Reviewed-by: Brian Norris <briannorris@chromium.org> [modify] https://crrev.com/8d736c049685ddfe70f34e38a8ce406605750153/net-print/cups-filters/files/cups-filters-1.8.2-foomatic-rip-sig-pipe.patch [rename] https://crrev.com/8d736c049685ddfe70f34e38a8ce406605750153/net-print/cups-filters/cups-filters-1.8.2-r5.ebuild
,
Jan 22 2018
,
Jan 23 2018
|
|||||
►
Sign in to add a comment |
|||||
Comment 1 by skau@chromium.org
, Aug 30 2017Owner: luum@chromium.org