* Re: colord: add run script
2020-12-01 21:45 [PR PATCH] colord: add run script gch1p
@ 2020-12-12 7:04 ` ericonr
2020-12-17 16:38 ` [PR PATCH] [Updated] " gch1p
` (5 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: ericonr @ 2020-12-12 7:04 UTC (permalink / raw)
To: ml
[-- 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?
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PR PATCH] [Updated] colord: add run script
2020-12-01 21:45 [PR PATCH] colord: add run script gch1p
2020-12-12 7:04 ` ericonr
@ 2020-12-17 16:38 ` gch1p
2020-12-17 16:38 ` gch1p
` (4 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: gch1p @ 2020-12-17 16:38 UTC (permalink / raw)
To: ml
[-- 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
+}
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: colord: add run script
2020-12-01 21:45 [PR PATCH] colord: add run script gch1p
2020-12-12 7:04 ` ericonr
2020-12-17 16:38 ` [PR PATCH] [Updated] " gch1p
@ 2020-12-17 16:38 ` gch1p
2020-12-17 16:43 ` Duncaen
` (3 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: gch1p @ 2020-12-17 16:38 UTC (permalink / raw)
To: ml
[-- 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.
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: colord: add run script
2020-12-01 21:45 [PR PATCH] colord: add run script gch1p
` (2 preceding siblings ...)
2020-12-17 16:38 ` gch1p
@ 2020-12-17 16:43 ` Duncaen
2020-12-17 17:56 ` [PR PATCH] [Updated] " gch1p
` (2 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: Duncaen @ 2020-12-17 16:43 UTC (permalink / raw)
To: ml
[-- 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.
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PR PATCH] [Updated] colord: add run script
2020-12-01 21:45 [PR PATCH] colord: add run script gch1p
` (3 preceding siblings ...)
2020-12-17 16:43 ` Duncaen
@ 2020-12-17 17:56 ` gch1p
2020-12-17 17:58 ` gch1p
2020-12-30 7:01 ` [PR PATCH] [Closed]: " the-maldridge
6 siblings, 0 replies; 8+ messages in thread
From: gch1p @ 2020-12-17 17:56 UTC (permalink / raw)
To: ml
[-- 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
+}
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: colord: add run script
2020-12-01 21:45 [PR PATCH] colord: add run script gch1p
` (4 preceding siblings ...)
2020-12-17 17:56 ` [PR PATCH] [Updated] " gch1p
@ 2020-12-17 17:58 ` gch1p
2020-12-30 7:01 ` [PR PATCH] [Closed]: " the-maldridge
6 siblings, 0 replies; 8+ messages in thread
From: gch1p @ 2020-12-17 17:58 UTC (permalink / raw)
To: ml
[-- 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.
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [PR PATCH] [Closed]: colord: add run script
2020-12-01 21:45 [PR PATCH] colord: add run script gch1p
` (5 preceding siblings ...)
2020-12-17 17:58 ` gch1p
@ 2020-12-30 7:01 ` the-maldridge
6 siblings, 0 replies; 8+ messages in thread
From: the-maldridge @ 2020-12-30 7:01 UTC (permalink / raw)
To: ml
[-- 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:
^ permalink raw reply [flat|nested] 8+ messages in thread