[-- Attachment #1: Type: text/plain, Size: 1866 bytes --] There is a new pull request by NeelChotai against master on the void-packages repository https://github.com/NeelChotai/void-packages Solaar-update https://github.com/void-linux/void-packages/pull/33425 Solaar: update to 1.0.7, install udev rules Fixes #33422. Solaar currently runs with the error message: ``` ERROR: Solaar depends on a udev file that is not present. For more information see the Solaar installation directions at https://pwr-solaar.github.io/Solaar/installation ``` <!-- Mark items with [x] where applicable --> #### General - [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements) #### Have the results of the proposed changes been tested? - [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me - [ ] I generally don't use the affected packages but briefly tested this PR <!-- If GitHub CI cannot be used to validate the build result (for example, if the build is likely to take several hours), make sure to [skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration). When skipping CI, uncomment and fill out the following section. Note: for builds that are likely to complete in less than 2 hours, it is not acceptable to skip CI. --> <!-- #### Does it build and run successfully? (Please choose at least one native build and, if supported, at least one cross build. More are better.) - [ ] I built this PR locally for my native architecture, (ARCH-LIBC) - [ ] I built this PR locally for these architectures (if supported. mark crossbuilds): - [ ] aarch64-musl - [ ] armv7l - [ ] armv6l-musl --> A patch file from https://github.com/void-linux/void-packages/pull/33425.patch is attached [-- Warning: decoded text below may be mangled, UTF-8 assumed --] [-- Attachment #2: github-pr-Solaar-update-33425.patch --] [-- Type: text/x-diff, Size: 1172 bytes --] From ed4176eea88c4e5f00577e41dd006c4ee0046fa3 Mon Sep 17 00:00:00 2001 From: Neel Chotai <neel.chotai@hyperexponential.com> Date: Fri, 8 Oct 2021 17:21:04 +0100 Subject: [PATCH] Solaar: update to 1.0.7, install udev rules --- srcpkgs/Solaar/template | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/srcpkgs/Solaar/template b/srcpkgs/Solaar/template index 393cede0f0ae..85a7ebf1a82e 100644 --- a/srcpkgs/Solaar/template +++ b/srcpkgs/Solaar/template @@ -1,6 +1,6 @@ # Template file for 'Solaar' pkgname=Solaar -version=1.0.6 +version=1.0.7 revision=1 build_style=python3-module hostmakedepends="python3-setuptools" @@ -10,6 +10,10 @@ maintainer="Young Jin Park <youngjinpark20@gmail.com>" license="GPL-3.0-or-later" homepage="https://pwr-solaar.github.io/Solaar/" distfiles="https://github.com/pwr-Solaar/Solaar/archive/${version}.tar.gz" -checksum=fb879136911978f8f0183262a86f2b47a50b1816516599ae9a6681ad1d9a4e49 +checksum=39c025b4186b6cb4620bc52d1d20e2d841082982c8be0fed155398faee7a9cd1 # Package provides no tests make_check=no + +post_install() { + vinstall rules.d/42-logitech-unify-permissions.rules 644 usr/lib/udev/rules.d +}
[-- Attachment #1: Type: text/plain, Size: 551 bytes --] New comment by ahesford on void-packages repository https://github.com/void-linux/void-packages/pull/33425#issuecomment-939194470 Comment: Rules are available in the package at ``` /usr/share/solaar/udev-rules.d/42-logitech-unify-permissions.rules ``` I thought about enabling these rules some time in the past but, as the comments note, rules that grant write permissions to ordinary users might be undesirable in multi-user systems. I concluded it was better to leave the rule as it is so individual administrators can opt in as they see fit.
[-- Attachment #1: Type: text/plain, Size: 671 bytes --] New comment by NeelChotai on void-packages repository https://github.com/void-linux/void-packages/pull/33425#issuecomment-939317325 Comment: > Rules are available in the package at > > ``` > /usr/share/solaar/udev-rules.d/42-logitech-unify-permissions.rules > ``` > > I thought about enabling these rules some time in the past but, as the comments note, rules that grant write permissions to ordinary users might be undesirable in multi-user systems. I concluded it was better to leave the rule as it is so individual administrators can opt in as they see fit. This sounds reasonable, I will update with an `INSTALL.msg` indicating where the udev rules are.
[-- Attachment #1: Type: text/plain, Size: 1871 bytes --] There is an updated pull request by NeelChotai against master on the void-packages repository https://github.com/NeelChotai/void-packages Solaar-update https://github.com/void-linux/void-packages/pull/33425 Solaar: update to 1.0.7, install udev rules Fixes #33422. Solaar currently runs with the error message: ``` ERROR: Solaar depends on a udev file that is not present. For more information see the Solaar installation directions at https://pwr-solaar.github.io/Solaar/installation ``` <!-- Mark items with [x] where applicable --> #### General - [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements) #### Have the results of the proposed changes been tested? - [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me - [ ] I generally don't use the affected packages but briefly tested this PR <!-- If GitHub CI cannot be used to validate the build result (for example, if the build is likely to take several hours), make sure to [skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration). When skipping CI, uncomment and fill out the following section. Note: for builds that are likely to complete in less than 2 hours, it is not acceptable to skip CI. --> <!-- #### Does it build and run successfully? (Please choose at least one native build and, if supported, at least one cross build. More are better.) - [ ] I built this PR locally for my native architecture, (ARCH-LIBC) - [ ] I built this PR locally for these architectures (if supported. mark crossbuilds): - [ ] aarch64-musl - [ ] armv7l - [ ] armv6l-musl --> A patch file from https://github.com/void-linux/void-packages/pull/33425.patch is attached [-- Warning: decoded text below may be mangled, UTF-8 assumed --] [-- Attachment #2: github-pr-Solaar-update-33425.patch --] [-- Type: text/x-diff, Size: 1631 bytes --] From f715081a154d5a8c47be53559484e63dd9c22572 Mon Sep 17 00:00:00 2001 From: Neel Chotai <neel.chotai@hyperexponential.com> Date: Fri, 8 Oct 2021 17:21:04 +0100 Subject: [PATCH] Solaar: update to 1.0.7 --- srcpkgs/Solaar/INSTALL.msg | 6 ++++++ srcpkgs/Solaar/template | 4 ++-- 2 files changed, 8 insertions(+), 2 deletions(-) create mode 100644 srcpkgs/Solaar/INSTALL.msg diff --git a/srcpkgs/Solaar/INSTALL.msg b/srcpkgs/Solaar/INSTALL.msg new file mode 100644 index 000000000000..a09ce30882e5 --- /dev/null +++ b/srcpkgs/Solaar/INSTALL.msg @@ -0,0 +1,6 @@ +Solaar depends on a udev file available at +/usr/share/solaar/udev-rules.d/42-logitech-unify-permissions.rules. You may +want to install this file to /usr/lib/udev/rules.d/ and reload udev rules. + +This behaviour may not be desirable on multi-user systems as it grants write +permissions to any seated user. diff --git a/srcpkgs/Solaar/template b/srcpkgs/Solaar/template index 393cede0f0ae..9377d7a11fa6 100644 --- a/srcpkgs/Solaar/template +++ b/srcpkgs/Solaar/template @@ -1,6 +1,6 @@ # Template file for 'Solaar' pkgname=Solaar -version=1.0.6 +version=1.0.7 revision=1 build_style=python3-module hostmakedepends="python3-setuptools" @@ -10,6 +10,6 @@ maintainer="Young Jin Park <youngjinpark20@gmail.com>" license="GPL-3.0-or-later" homepage="https://pwr-solaar.github.io/Solaar/" distfiles="https://github.com/pwr-Solaar/Solaar/archive/${version}.tar.gz" -checksum=fb879136911978f8f0183262a86f2b47a50b1816516599ae9a6681ad1d9a4e49 +checksum=39c025b4186b6cb4620bc52d1d20e2d841082982c8be0fed155398faee7a9cd1 # Package provides no tests make_check=no
[-- Attachment #1: Type: text/plain, Size: 286 bytes --] New comment by NeelChotai on void-packages repository https://github.com/void-linux/void-packages/pull/33425#issuecomment-940842123 Comment: @ahesford updated, should this warning be made more explicit? Is it worth dropping a link to https://pwr-solaar.github.io/Solaar/installation?
[-- Attachment #1: Type: text/plain, Size: 786 bytes --] New review comment by ahesford on void-packages repository https://github.com/void-linux/void-packages/pull/33425#discussion_r726999940 Comment: ```suggestion To use Solaar without root privileges, copy the file /usr/share/solaar/udev-rules.d/42-logitech-unify-permissions.rules into the directory /etc/udev/rules.d/ and reload udev rules. ``` I don't really think we need an install message---it adds noise that most people will just ignore during installation and upgrades, the rule file is installed where upstream puts it, and we generally expect people to read the documentation of packages they install---but if it will clear up some confusion, the suggested change presents the optional rule while also providing a hint about how to use Solaar without using the rule.
[-- Attachment #1: Type: text/plain, Size: 1851 bytes --] There is an updated pull request by NeelChotai against master on the void-packages repository https://github.com/NeelChotai/void-packages Solaar-update https://github.com/void-linux/void-packages/pull/33425 Solaar: update to 1.0.7 Fixes #33422. Solaar currently runs with the error message: ``` ERROR: Solaar depends on a udev file that is not present. For more information see the Solaar installation directions at https://pwr-solaar.github.io/Solaar/installation ``` <!-- Mark items with [x] where applicable --> #### General - [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements) #### Have the results of the proposed changes been tested? - [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me - [ ] I generally don't use the affected packages but briefly tested this PR <!-- If GitHub CI cannot be used to validate the build result (for example, if the build is likely to take several hours), make sure to [skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration). When skipping CI, uncomment and fill out the following section. Note: for builds that are likely to complete in less than 2 hours, it is not acceptable to skip CI. --> <!-- #### Does it build and run successfully? (Please choose at least one native build and, if supported, at least one cross build. More are better.) - [ ] I built this PR locally for my native architecture, (ARCH-LIBC) - [ ] I built this PR locally for these architectures (if supported. mark crossbuilds): - [ ] aarch64-musl - [ ] armv7l - [ ] armv6l-musl --> A patch file from https://github.com/void-linux/void-packages/pull/33425.patch is attached [-- Warning: decoded text below may be mangled, UTF-8 assumed --] [-- Attachment #2: github-pr-Solaar-update-33425.patch --] [-- Type: text/x-diff, Size: 1505 bytes --] From 910d3d6479d1f57db6392e0f22498eb7525460d1 Mon Sep 17 00:00:00 2001 From: Neel Chotai <neel.chotai@hyperexponential.com> Date: Fri, 8 Oct 2021 17:21:04 +0100 Subject: [PATCH] Solaar: update to 1.0.7 --- srcpkgs/Solaar/INSTALL.msg | 3 +++ srcpkgs/Solaar/template | 4 ++-- 2 files changed, 5 insertions(+), 2 deletions(-) create mode 100644 srcpkgs/Solaar/INSTALL.msg diff --git a/srcpkgs/Solaar/INSTALL.msg b/srcpkgs/Solaar/INSTALL.msg new file mode 100644 index 000000000000..7596828f668a --- /dev/null +++ b/srcpkgs/Solaar/INSTALL.msg @@ -0,0 +1,3 @@ +To use Solaar without root privileges, copy the file + /usr/share/solaar/udev-rules.d/42-logitech-unify-permissions.rules +into the directory /etc/udev/rules.d/ and reload udev rules. diff --git a/srcpkgs/Solaar/template b/srcpkgs/Solaar/template index 393cede0f0ae..9377d7a11fa6 100644 --- a/srcpkgs/Solaar/template +++ b/srcpkgs/Solaar/template @@ -1,6 +1,6 @@ # Template file for 'Solaar' pkgname=Solaar -version=1.0.6 +version=1.0.7 revision=1 build_style=python3-module hostmakedepends="python3-setuptools" @@ -10,6 +10,6 @@ maintainer="Young Jin Park <youngjinpark20@gmail.com>" license="GPL-3.0-or-later" homepage="https://pwr-solaar.github.io/Solaar/" distfiles="https://github.com/pwr-Solaar/Solaar/archive/${version}.tar.gz" -checksum=fb879136911978f8f0183262a86f2b47a50b1816516599ae9a6681ad1d9a4e49 +checksum=39c025b4186b6cb4620bc52d1d20e2d841082982c8be0fed155398faee7a9cd1 # Package provides no tests make_check=no
[-- Attachment #1: Type: text/plain, Size: 188 bytes --] New review comment by NeelChotai on void-packages repository https://github.com/void-linux/void-packages/pull/33425#discussion_r727008947 Comment: I'm happy with this approach, updated.
[-- Attachment #1: Type: text/plain, Size: 209 bytes --] New comment by ahesford on void-packages repository https://github.com/void-linux/void-packages/pull/33425#issuecomment-940909269 Comment: Please rebase, Solaar had been revbumped for the Python 3.10 update
[-- Attachment #1: Type: text/plain, Size: 1851 bytes --] There is an updated pull request by NeelChotai against master on the void-packages repository https://github.com/NeelChotai/void-packages Solaar-update https://github.com/void-linux/void-packages/pull/33425 Solaar: update to 1.0.7 Fixes #33422. Solaar currently runs with the error message: ``` ERROR: Solaar depends on a udev file that is not present. For more information see the Solaar installation directions at https://pwr-solaar.github.io/Solaar/installation ``` <!-- Mark items with [x] where applicable --> #### General - [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements) #### Have the results of the proposed changes been tested? - [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me - [ ] I generally don't use the affected packages but briefly tested this PR <!-- If GitHub CI cannot be used to validate the build result (for example, if the build is likely to take several hours), make sure to [skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration). When skipping CI, uncomment and fill out the following section. Note: for builds that are likely to complete in less than 2 hours, it is not acceptable to skip CI. --> <!-- #### Does it build and run successfully? (Please choose at least one native build and, if supported, at least one cross build. More are better.) - [ ] I built this PR locally for my native architecture, (ARCH-LIBC) - [ ] I built this PR locally for these architectures (if supported. mark crossbuilds): - [ ] aarch64-musl - [ ] armv7l - [ ] armv6l-musl --> A patch file from https://github.com/void-linux/void-packages/pull/33425.patch is attached [-- Warning: decoded text below may be mangled, UTF-8 assumed --] [-- Attachment #2: github-pr-Solaar-update-33425.patch --] [-- Type: text/x-diff, Size: 1586 bytes --] From ca634b8091da60b661e6755eb096227b7d239610 Mon Sep 17 00:00:00 2001 From: Neel Chotai <neel.chotai@hyperexponential.com> Date: Fri, 8 Oct 2021 17:21:04 +0100 Subject: [PATCH] Solaar: update to 1.0.7 --- srcpkgs/Solaar/INSTALL.msg | 3 +++ srcpkgs/Solaar/template | 6 +++--- 2 files changed, 6 insertions(+), 3 deletions(-) create mode 100644 srcpkgs/Solaar/INSTALL.msg diff --git a/srcpkgs/Solaar/INSTALL.msg b/srcpkgs/Solaar/INSTALL.msg new file mode 100644 index 000000000000..7596828f668a --- /dev/null +++ b/srcpkgs/Solaar/INSTALL.msg @@ -0,0 +1,3 @@ +To use Solaar without root privileges, copy the file + /usr/share/solaar/udev-rules.d/42-logitech-unify-permissions.rules +into the directory /etc/udev/rules.d/ and reload udev rules. diff --git a/srcpkgs/Solaar/template b/srcpkgs/Solaar/template index 689322c13dd9..9377d7a11fa6 100644 --- a/srcpkgs/Solaar/template +++ b/srcpkgs/Solaar/template @@ -1,7 +1,7 @@ # Template file for 'Solaar' pkgname=Solaar -version=1.0.6 -revision=2 +version=1.0.7 +revision=1 build_style=python3-module hostmakedepends="python3-setuptools" depends="python3-pyudev python3-psutil python3-yaml python3-xlib" @@ -10,6 +10,6 @@ maintainer="Young Jin Park <youngjinpark20@gmail.com>" license="GPL-3.0-or-later" homepage="https://pwr-solaar.github.io/Solaar/" distfiles="https://github.com/pwr-Solaar/Solaar/archive/${version}.tar.gz" -checksum=fb879136911978f8f0183262a86f2b47a50b1816516599ae9a6681ad1d9a4e49 +checksum=39c025b4186b6cb4620bc52d1d20e2d841082982c8be0fed155398faee7a9cd1 # Package provides no tests make_check=no
[-- Attachment #1: Type: text/plain, Size: 247 bytes --] New comment by NeelChotai on void-packages repository https://github.com/void-linux/void-packages/pull/33425#issuecomment-940993925 Comment: > Please rebase, Solaar had been revbumped for the Python 3.10 update Thanks for the reminder, done.
[-- Attachment #1: Type: text/plain, Size: 1683 bytes --] There's a merged pull request on the void-packages repository Solaar: update to 1.0.7 https://github.com/void-linux/void-packages/pull/33425 Description: Fixes #33422. Solaar currently runs with the error message: ``` ERROR: Solaar depends on a udev file that is not present. For more information see the Solaar installation directions at https://pwr-solaar.github.io/Solaar/installation ``` <!-- Mark items with [x] where applicable --> #### General - [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements) #### Have the results of the proposed changes been tested? - [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me - [ ] I generally don't use the affected packages but briefly tested this PR <!-- If GitHub CI cannot be used to validate the build result (for example, if the build is likely to take several hours), make sure to [skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration). When skipping CI, uncomment and fill out the following section. Note: for builds that are likely to complete in less than 2 hours, it is not acceptable to skip CI. --> <!-- #### Does it build and run successfully? (Please choose at least one native build and, if supported, at least one cross build. More are better.) - [ ] I built this PR locally for my native architecture, (ARCH-LIBC) - [ ] I built this PR locally for these architectures (if supported. mark crossbuilds): - [ ] aarch64-musl - [ ] armv7l - [ ] armv6l-musl -->