Arch Linux User Repository

Hi @Skeleton-Hacker!

There are some issues in your PKBUILD that needs to be addressed in order for this package to build correctly by others.

First, source is set to a local directory (presumably on your own machine). You most likely want to set it to your Github repository instead:

source=("git+file:///home/yajatr/Documents/Projects/CLI_Reminder")

In the repository README it says that the system integration checks for due reminders every minute, but the systemd timer that gets installed runs every second:

[Unit]
Description=Periodically check for due reminders

[Timer]
OnBootSec=1s
OnUnitActiveSec=1s
AccuracySec=1s

[Install]
WantedBy=timers.target

The systemd service that gets installed defines the DBUS_SESSION_BUS_ADDRESS which is hardcoded to unix:path=/run/user/1000/bus, which assumes that the user running this service has UID=1000 and will not work for users with another UID. Same thing with DISPLAY - hardcoded to :0, which might not always be the case:

[Unit]
Description=Check for due reminders

[Service]
Type=oneshot
ExecStart=/usr/bin/remindme notify --desktop
Environment=DISPLAY=:0
Environment=DBUS_SESSION_BUS_ADDRESS=unix:path=/run/user/1000/bus

I am not sure if you really need to define these environment variables in the systemd user service unit. According to this post, they are automatically inherited by systemd units when running as a user service (systemctl --user [...]). You can verify this by running the following command:

$ systemctl --user show-environment

However, if you really must you should use the following value for DBUS_SESSION_BUS_ADDRESS: unix:path=%t/bus which would expand to unix:path=/run/user/$UID/bus (where $UID is replaced with the UID of current user).

Lastly, the package is named cli-reminder but the services and configuration directory seems to use the name remindme. It might be good to use the same name in all places to improve discoverability.