Arch Linux User Repository

I would really appreciate it if I didn't have to re-enable the systemd service after every update... I don't get why it needs to be disabled constantly? This is the only software that does this with every single update.

It would be great if this package could be changed to not include system-modifying commands (like enabling/disabling systemd services and reloading udev rules) as part of the install script. I view this as dangerous and deeply problematic.

It shouldn't be necessary to reload udev rules, as there is a default hook that does this, which I would expect to be present at /usr/share/libalpm/hooks/35-systemd-udev-reload.hook.

A package should not start, stop, enable, or disable services. It is generally safe to leave processes running even if the files they were started from change out from under them on-disk, because if they have open file descriptors from the old versions, they will continue to be able to access the files. Instead, consider printing a warning that recommends the user restart the service, if needed.

That should now be addressed

It does add a wrong command like udevadm control --reload-rulea which has a typo. I don't get why are you messing with user configs disabling, reloading and reenabling services. Just update the binaries, that's all.

The update doesn't fix the new dependencies required which makes it unable to build in a clean chroot environment, basically breaking builds.

Updated to 0.8.0

Any update on this? We are at version 8.0 and the new interface is very nice compared to the last.

Looking to get an update to this as its had two releases here in the last month or so. It looks like they super picked up development. If you don't have time you can abandon it and I will pick it up.

The new 0.7.6 version introduced a new dependency on libpipewire too. There are many changes needed at this point that I added a new PKGBUILD from scratch at https://gitlab.com/chaotic-aur/pkgbuilds/-/tree/main/openlinkhub, feel free to pick them up if you want

Here's a patch that does solve the two things I mentioned previously: the version number and the unneeded .install script (this patch can be replicated to the -git package too for versioning):

Edit: the new build options were taken from https://wiki.archlinux.org/title/Go_package_guidelines

diff --git a/.SRCINFO b/.SRCINFO
index 23c99da..fd47406 100644
--- a/.SRCINFO
+++ b/.SRCINFO
@@ -1,9 +1,8 @@
 pkgbase = openlinkhub
    pkgdesc = Open source Linux interface for iCUE LINK Hub and other Corsair AIOs, Hubs. [Latest Release - source]
    pkgver = 0.7.5
-   pkgrel = 1
+   pkgrel = 2
    url = https://github.com/jurkovic-nikola/OpenLinkHub
-   install = openlinkhub.install
    arch = x86_64
    license = GPL3
    makedepends = go>=1.23.8
@@ -16,12 +15,10 @@ pkgbase = openlinkhub
    conflicts = openlinkhub
    backup = etc/udev/rules.d/99-openlinkhub.rules
    source = openlinkhub::git+https://github.com/jurkovic-nikola/OpenLinkHub.git#tag=0.7.5
-   source = openlinkhub.install
    source = openlinkhub.sysusers
    source = openlinkhub.service
    source = openlinkhub.tmpfiles
    sha256sums = 0899b779883126df7d20eaf064ee0a6a52218032dcfb9d243ec743aeed1ac4e4
-   sha256sums = 0820bcc60c77bd23178f4766f92f01dae2a75ae704ad6ac40ecf598a55002d36
    sha256sums = 8c9f747bc6484290cb97b40e5904dc02cce2672e59e0f6ad720a1cd6a7b9d900
    sha256sums = 858fd197e13a6bc2756e090f622adcac0d02d20007c366d0dff93258898e256e
    sha256sums = 70c1d136ed639a84c6aca077df51ff857c32df8db5d74cc7df48f463708bdd0b
diff --git a/PKGBUILD b/PKGBUILD
index 9d08da3..5f6682d 100644
--- a/PKGBUILD
+++ b/PKGBUILD
@@ -7,32 +7,25 @@ _binlocation=/usr/bin/"${pkgname%-*}"
 _applocation=/var/lib/"${pkgname%-*}"
 _tag=0.7.5
 pkgver=0.7.5
-pkgrel=1
+pkgrel=2
 pkgdesc="Open source Linux interface for iCUE LINK Hub and other Corsair AIOs, Hubs. [Latest Release - source]"
 arch=('x86_64')
 url="https://github.com/jurkovic-nikola/OpenLinkHub"
 license=('GPL3')
-groups=()
 depends=('systemd' 'i2c-tools')
 makedepends=('go>=1.23.8' 'git' 'base-devel' 'systemd') 
 provides=("${pkgname%-*}")
 conflicts=("${pkgname%-*}")
-replaces=()
 backup=(
    "etc/udev/rules.d/99-openlinkhub.rules"
    )
-options=()
-install="${pkgname%-*}".install
 source=(
    "${pkgname%-*}::git+https://github.com/jurkovic-nikola/${_upstreamname}.git#tag=${_tag}"
-   "${pkgname%-*}".install
    "${pkgname%-*}".sysusers
    "${pkgname%-*}".service
    "${pkgname%-*}".tmpfiles
 )
-noextract=()           
 sha256sums=('0899b779883126df7d20eaf064ee0a6a52218032dcfb9d243ec743aeed1ac4e4'
-            '0820bcc60c77bd23178f4766f92f01dae2a75ae704ad6ac40ecf598a55002d36'
             '8c9f747bc6484290cb97b40e5904dc02cce2672e59e0f6ad720a1cd6a7b9d900'
             '858fd197e13a6bc2756e090f622adcac0d02d20007c366d0dff93258898e256e'
             '70c1d136ed639a84c6aca077df51ff857c32df8db5d74cc7df48f463708bdd0b')
@@ -44,7 +37,13 @@ pkgver() {

 build() {
    cd "${pkgname%-*}"
-   go build .
+   go build \
+       -trimpath \
+       -buildmode=pie \
+       -mod=readonly \
+       -modcacherw \
+       -ldflags "-X OpenLinkHub/src/version.Version=${pkgver}" \
+       .
 }

 package() {
diff --git a/openlinkhub.install b/openlinkhub.install
deleted file mode 100644
index 17b9e01..0000000
--- a/openlinkhub.install
+++ /dev/null
@@ -1,25 +0,0 @@
-post_install() {
-   udevadm control --reload-rules
-   echo -e "\e[1;7m[OpenLinkHub]\e[0m: \e[92mInstall Complete\e[0m "
-   echo -e "\e[1;7m[OpenLinkHub]\e[0m: WebUI-> \e[1;7mhttp://127.0.0.1:27003\e[0m"
-   echo -e "\e[1;7m[OpenLinkHub]\e[0m: Start Service: \e[92msudo systemctl enable --now openlinkhub\e[0m"
-}
-
-pre_upgrade() {
-   systemctl disable --now openlinkhub
-}
-
-post_upgrade() {
-   echo -e "\e[1;7m[OpenLinkHub]\e[0m: \e[92mUpgrade Complete\e[0m"
-   echo -e "\e[1;7m[OpenLinkHub]\e[0m: WebUI-> \e[1;7mhttp://127.0.0.1:27003\e[0m"
-   echo -e "\e[1;7m[OpenLinkHub]\e[0m: Restart Service: \e[92msudo systemctl restart openlinkhub\e[0m"
-}
-
-pre_remove() {
-   systemctl disable --now openlinkhub
-}
-
-post_remove() {
-   udevadm control --reload-rules
-   systemctl daemon-reload
-}