[PR PATCH] colord: add run script

[PR PATCH] colord: add run script

[gch1p]

[-- Attachment #1: Type: text/plain, Size: 302 bytes --]

There is a new pull request by gch1p against master on the void-packages repository

https://github.com/gch1p/void-packages colord
https://github.com/void-linux/void-packages/pull/26875

colord: add run script


A patch file from https://github.com/void-linux/void-packages/pull/26875.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-colord-26875.patch --]
[-- Type: text/x-diff, Size: 990 bytes --]

From 5e83b4d661b7618808706faeb836c43804952864 Mon Sep 17 00:00:00 2001
From: Evgeny Zinoviev <me@ch1p.io>
Date: Wed, 2 Dec 2020 00:44:52 +0300
Subject: [PATCH] colord: add run script

---
 srcpkgs/colord/files/colord/run | 9 +++++++++
 srcpkgs/colord/template         | 4 ++++
 2 files changed, 13 insertions(+)
 create mode 100755 srcpkgs/colord/files/colord/run

diff --git a/srcpkgs/colord/files/colord/run b/srcpkgs/colord/files/colord/run
new file mode 100755
index 00000000000..44dc0a8426e
--- /dev/null
+++ b/srcpkgs/colord/files/colord/run
@@ -0,0 +1,9 @@
+#!/bin/sh
+set -e
+
+[ -d /var/lib/colord ] || {
+   	mkdir -p /var/lib/colord
+	chown -R colord:colord /var/lib/colord
+}
+
+exec chpst -u colord /usr/libexec/colord 2>&1
diff --git a/srcpkgs/colord/template b/srcpkgs/colord/template
index 6e77160111a..25634b7808c 100644
--- a/srcpkgs/colord/template
+++ b/srcpkgs/colord/template
@@ -65,3 +65,7 @@ colord-devel_package() {
 		fi
 	}
 }
+
+post_install() {
+	vsv colord
+}

Re: colord: add run script

[ericonr]

[-- Attachment #1: Type: text/plain, Size: 290 bytes --]

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/26875#issuecomment-743715404

Comment:
For this to take effect, you need to increase the package's revision.

Could you add an explanation to the commit message as to why this is needed?

Re: [PR PATCH] [Updated] colord: add run script

[gch1p]

[-- Attachment #1: Type: text/plain, Size: 307 bytes --]

There is an updated pull request by gch1p against master on the void-packages repository

https://github.com/gch1p/void-packages colord
https://github.com/void-linux/void-packages/pull/26875

colord: add run script


A patch file from https://github.com/void-linux/void-packages/pull/26875.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-colord-26875.patch --]
[-- Type: text/x-diff, Size: 1326 bytes --]

From 599dce6e2346c2c8ec28b75a2acc65e5ef2c3c6f Mon Sep 17 00:00:00 2001
From: Evgeny Zinoviev <me@ch1p.io>
Date: Wed, 2 Dec 2020 00:44:52 +0300
Subject: [PATCH] colord: add run script

colord is a daemon so it needs a run script. I personally
use it for managing color profiles for my displays: Xfce has
color management support via colord + xiccd.
---
 srcpkgs/colord/files/colord/run | 6 ++++++
 srcpkgs/colord/template         | 6 +++++-
 2 files changed, 11 insertions(+), 1 deletion(-)
 create mode 100755 srcpkgs/colord/files/colord/run

diff --git a/srcpkgs/colord/files/colord/run b/srcpkgs/colord/files/colord/run
new file mode 100755
index 00000000000..2bb3c953cce
--- /dev/null
+++ b/srcpkgs/colord/files/colord/run
@@ -0,0 +1,6 @@
+#!/bin/sh
+[ -d /var/lib/colord ] || {
+   	mkdir -p /var/lib/colord
+	chown -R colord:colord /var/lib/colord
+}
+exec chpst -u colord /usr/libexec/colord 2>&1
diff --git a/srcpkgs/colord/template b/srcpkgs/colord/template
index 6e77160111a..24b19c33b9e 100644
--- a/srcpkgs/colord/template
+++ b/srcpkgs/colord/template
@@ -1,7 +1,7 @@
 # Template file for 'colord'
 pkgname=colord
 version=1.4.5
-revision=1
+revision=2
 build_style=meson
 build_helper="gir"
 # man pages fail to generate
@@ -65,3 +65,7 @@ colord-devel_package() {
 		fi
 	}
 }
+
+post_install() {
+	vsv colord
+}

Re: colord: add run script

[gch1p]

[-- Attachment #1: Type: text/plain, Size: 153 bytes --]

New comment by gch1p on void-packages repository

https://github.com/void-linux/void-packages/pull/26875#issuecomment-747556393

Comment:
@ericonr done.

Re: colord: add run script

[Duncaen]

[-- Attachment #1: Type: text/plain, Size: 229 bytes --]

New comment by Duncaen on void-packages repository

https://github.com/void-linux/void-packages/pull/26875#issuecomment-747558768

Comment:
This should create the directory using `make_dirs` as its not on a temporary filesystem.

Re: [PR PATCH] [Updated] colord: add run script

[gch1p]

[-- Attachment #1: Type: text/plain, Size: 307 bytes --]

There is an updated pull request by gch1p against master on the void-packages repository

https://github.com/gch1p/void-packages colord
https://github.com/void-linux/void-packages/pull/26875

colord: add run script


A patch file from https://github.com/void-linux/void-packages/pull/26875.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-colord-26875.patch --]
[-- Type: text/x-diff, Size: 1218 bytes --]

From cfd8d8dd1da30128beab6adc183c5e41d450c297 Mon Sep 17 00:00:00 2001
From: Evgeny Zinoviev <me@ch1p.io>
Date: Wed, 2 Dec 2020 00:44:52 +0300
Subject: [PATCH] colord: add run script

colord is a daemon so it needs a run script. I personally
use it for managing color profiles for my displays: Xfce has
color management support via colord + xiccd.
---
 srcpkgs/colord/files/colord/run | 2 ++
 srcpkgs/colord/template         | 6 +++++-
 2 files changed, 7 insertions(+), 1 deletion(-)
 create mode 100755 srcpkgs/colord/files/colord/run

diff --git a/srcpkgs/colord/files/colord/run b/srcpkgs/colord/files/colord/run
new file mode 100755
index 00000000000..5650cdaa888
--- /dev/null
+++ b/srcpkgs/colord/files/colord/run
@@ -0,0 +1,2 @@
+#!/bin/sh
+exec chpst -u colord /usr/libexec/colord 2>&1
diff --git a/srcpkgs/colord/template b/srcpkgs/colord/template
index 6e77160111a..24b19c33b9e 100644
--- a/srcpkgs/colord/template
+++ b/srcpkgs/colord/template
@@ -1,7 +1,7 @@
 # Template file for 'colord'
 pkgname=colord
 version=1.4.5
-revision=1
+revision=2
 build_style=meson
 build_helper="gir"
 # man pages fail to generate
@@ -65,3 +65,7 @@ colord-devel_package() {
 		fi
 	}
 }
+
+post_install() {
+	vsv colord
+}

Re: colord: add run script

[gch1p]

[-- Attachment #1: Type: text/plain, Size: 257 bytes --]

New comment by gch1p on void-packages repository

https://github.com/void-linux/void-packages/pull/26875#issuecomment-747601088

Comment:
It seems it's not needed, the directory is already created using `make_dirs` in the template file, my bad. Removed it.

Re: [PR PATCH] [Closed]: colord: add run script

[the-maldridge]

[-- Attachment #1: Type: text/plain, Size: 156 bytes --]

There's a closed pull request on the void-packages repository

colord: add run script
https://github.com/void-linux/void-packages/pull/26875

Description: