Bug 50153

Summary:	colord-sane: only install the systemd file when installing colord-sane
Product:	colord	Reporter:	Christoph Brill <egore>
Component:	daemon	Assignee:	Richard Hughes <richard>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	medium
Version:	unspecified
Hardware:	Other
OS:	All
Whiteboard:
i915 platform:		i915 features:
Attachments:	colord-0.1.18-dont-install-service.patch colord-0.1.18-disable-sane.patch

Description Christoph Brill 2012-05-20 12:36:27 UTC

Created attachment 61894 [details] [review]
colord-0.1.18-dont-install-service.patch

Right now colord-sane.service is installed if systemd is used. It also needs to be checked if the binary was actually built. The attached patch fixes that.

Comment 1 Christoph Brill 2012-05-20 12:43:29 UTC

Created attachment 61895 [details] [review]
colord-0.1.18-disable-sane.patch

I thought about the whole Makefile a bit more. If the user does not want sane to be installed, we don't need to do anything in the Makefile. The attached patch should do exactly what I described before.

Comment 2 Richard Hughes 2012-05-21 06:00:54 UTC

Makes sense. Do you want me to apply the second patch only?

Comment 3 Richard Hughes 2012-05-22 04:16:23 UTC

Comment on attachment 61895 [details] [review]
colord-0.1.18-disable-sane.patch

Review of attachment 61895 [details] [review]:
-----------------------------------------------------------------

Committed as:

commit a44ac58af394245c1363629ba9d2de6ee6936d34
Author: Christoph Brill <egore@gmx.de>
Date:   Tue May 22 12:14:37 2012 +0100

    Do not install any parts of colord-sane if --disable-sane is specified
    
    Signed-off-by: Richard Hughes <richard@hughsie.com>

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.