Arch Linux User Repository

Fixed

Doesn't build. You forgot to update the checksums

Oh, guess I forgot about that. Sorry. I pushed your changes and updated to the latest version from the wiki. I appreciate your input.

Here is the patch in case you forgot about this:

From e2852550f9f0730a267f600910c8c367bbbe418a Mon Sep 17 00:00:00 2001
From: Max Gautier <mg@max.gautier.name>
Date: Mon, 26 Aug 2024 10:57:56 +0200
Subject: [PATCH] Put udev rules in /usr

This allow to drop the manual reload of the rules since the systemd
packages ships a pacman hooks which does it for us.
---
 PKGBUILD                   | 3 +--
 reload-udev-daemon.install | 7 -------
 2 files changed, 1 insertion(+), 9 deletions(-)
 delete mode 100644 reload-udev-daemon.install

diff --git a/PKGBUILD b/PKGBUILD
index 720b27a..51eeac4 100644
--- a/PKGBUILD
+++ b/PKGBUILD
@@ -6,7 +6,6 @@ pkgdesc="udev rules to allow RPCS3 to communicate with DualShock 3, 4 and DualSe
 arch=('any')
 url="https://wiki.rpcs3.net/index.php?title=Help:Controller_Configuration"
 license=('CCPL:by-sa-4.0')
-install=reload-udev-daemon.install
 source=("99-ds3-controllers.rules"
         "99-ds4-controllers.rules"
         "99-dualsense-controllers.rules")
@@ -16,5 +15,5 @@ sha256sums=('f79b90f75ff0937472c1dc8fbb306e113fdc5ef97b804df9e5fb4f6e2a217a83'

 package() {
    cd "${srcdir}"
-   find . -name "*.rules" -exec install -Dm644 "{}" -t "${pkgdir}/etc/udev/rules.d/" \;
+   find . -name "*.rules" -exec install -Dm644 "{}" -t "${pkgdir}/usr/lib/udev/rules.d/" \;
 }
diff --git a/reload-udev-daemon.install b/reload-udev-daemon.install
deleted file mode 100644
index ba47941..0000000
--- a/reload-udev-daemon.install
+++ /dev/null
@@ -1,7 +0,0 @@
-post_install() {
-   echo "To apply the changes you either need to:"
-   echo "  1. restart your computer"
-   echo "OR"
-   echo "  2. run the following command and then replug in your controller:"
-   echo "     sudo udevadm control --reload-rules"
-}
-- 
2.46.0

@VannTen Thank you for your contribution. I'll implement these changes in the next couple of days when I have some time to spare.

Not also that by putting them into /usr/lib/udev/rules.d you can drop the reload-udev-daemon.install because systemd already ships a pacman hook that does that :

==> /usr/share/libalpm/hooks/30-systemd-udev-reload.hook <==
[Trigger]
Type = Path
Operation = Install
Operation = Upgrade
Operation = Remove
Target = usr/lib/udev/rules.d/*

[Action]
Description = Reloading device manager configuration...
When = PostTransaction
Exec = /usr/share/libalpm/scripts/systemd-hook udev-reload

The rules should indeed be placed in /usr/lib/udev/rules.d (take a look at man 7 udev).

Basically packages should usually not include things in /etc, it's supposed to be for local administration not managed by a package manager :)

dualsense needs to have "666" not "660"

I updated the wiki and forgot to say something here sorry.

According to the official wiki the udev rules are supposed to be placed into /etc. But if you have more insight in that subject I'm happy to fix that issue.

This package installs udev rules to /etc. Shouldn't they be placed in /usr/lib/udev?