New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 670037 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Dec 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Feature

Blocking:
issue 662755



Sign in to add a comment

Hook PPDProvider into manual CUPS setup flow

Project Member Reported by justincarlson@chromium.org, Nov 30 2016

Issue description

We need to explicitly hand the PPD to the cache when setting up a printer manually.
 

Comment 1 by skau@chromium.org, Nov 30 2016

Cc: justincarlson@chromium.org
Owner: skau@chromium.org
Status: Started (was: Assigned)
I'm going to steal since I'm looking at the flow right now.
Project Member

Comment 2 by bugdroid1@chromium.org, Dec 2 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform2/+/d53c85f1c073ba0835e4cf90b316a50e0d5bd02a

commit d53c85f1c073ba0835e4cf90b316a50e0d5bd02a
Author: Sean Kau <skau@chromium.org>
Date: Thu Dec 01 00:17:57 2016

debugd: Fix segment count for the PPDCache folder.

The file name is included in the segment count so it should
be increased to six.

BUG= chromium:670037 
TEST=Add a printer with the dbus client using a PPDCache path.
Change-Id: Ia9ad576667cdb36eca3aa3dd907d21db28b28b7c
Reviewed-on: https://chromium-review.googlesource.com/415430
Commit-Ready: Sean Kau <skau@chromium.org>
Tested-by: Sean Kau <skau@chromium.org>
Reviewed-by: Justin Carlson <justincarlson@google.com>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/d53c85f1c073ba0835e4cf90b316a50e0d5bd02a/debugd/src/cups_tool.cc

Comment 4 by skau@chromium.org, Dec 7 2016

Status: Fixed (was: Started)
User specified PPDs are now saved in the PPDCache folder.

Sign in to add a comment