Package Details: irssi-libnotify-git r59.7ada59a-4

Git Clone URL: https://aur.archlinux.org/irssi-libnotify-git.git (read-only, click to copy)
Package Base: irssi-libnotify-git
Description: libnotify plugin for irssi
Upstream URL: https://github.com/stickster/irssi-libnotify
Keywords: irssi libnotify
Licenses: GPL
Submitter: gtmanfred
Maintainer: rrt
Last Packager: rrt
Votes: 22
Popularity: 0.000000
First Submitted: 2012-04-19 21:38
Last Updated: 2021-03-18 19:01

Latest Comments

rrt commented on 2021-01-31 19:26

@VannTen thank you for your contribution, I reviewed it and merged it upstream!

VannTen commented on 2020-11-04 15:51

I'd like to propose a patch for this package (you can apply by copying in clipboard and using xsel | git am (or another cli clipboard manager)

From 938acee212b5d530523471da76843299f78c4c78 Mon Sep 17 00:00:00 2001
From: Max Gautier <mg@max.gautier.name>
Date: Wed, 4 Nov 2020 16:42:44 +0100
Subject: [PATCH] Put systemd unit in global user units directory

---
 PKGBUILD | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/PKGBUILD b/PKGBUILD
index d21c42f..256d680 100644
--- a/PKGBUILD
+++ b/PKGBUILD
@@ -6,7 +6,7 @@ _pkgname_base=irssi-libnotify
 pkgname=$_pkgname_base-git

 pkgver=r59.7ada59a
-pkgrel=2
+pkgrel=3
 pkgdesc="libnotify plugin for irssi"
 arch=("any")
 url="https://github.com/stickster/irssi-libnotify"
@@ -23,23 +23,23 @@ md5sums=('SKIP'
          'bf976922f1e60bfd6a6f2dbb682d8d79')

 pkgver() {
-  cd $srcdir/$pkgname
+  cd $pkgname

   printf "r%s.%s" "$(git rev-list --count HEAD)" "$(git rev-parse --short HEAD)"
 }

 build() {
-  cd $srcdir/$pkgname
+  cd $pkgname

   sed -i 's:python:&2:' notify-listener.py
   sed -i 's?~/bin/irssi-notifier.sh?/usr/bin/irssi-notifier?' notify.pl
 }

 package() {
-  cd $srcdir/$pkgname
+  cd $pkgname

   install -Dm755 irssi-notifier.sh  $pkgdir/usr/bin/irssi-notifier
   install -Dm755 notify-listener.py $pkgdir/usr/bin/notify-listener.py
   install -Dm644 notify.pl          $pkgdir/usr/share/irssi/scripts/notify.pl
-  install -Dm755 notify-listener.service $pkgdir/home/$USER/.config/systemd/user/notify-listener.service
+  install -Dm644 notify-listener.service $pkgdir/usr/lib/systemd/user/notify-listener.service
 }
-- 
2.29.2

It does two things :

  • replace cd $srcdir/$package with only cd $package. makepkg already leave us in the $srcdir package.

  • Place the systemd user unit in the correct place. With the current PKGBUILD, it is placed in the building user $HOME, which could be different from the installing user. (eg Building and pushing on private package repo).

Thanks for your time :)

wookietreiber commented on 2016-09-14 07:44

@rrt I just made you a maintainer of the package. Feel free to apply these changes yourself!

rrt commented on 2016-08-23 17:52

Thank you for this package! However, few things are missing:

1. python2-dbus is missing as dependency

$ notify-listener
/usr/bin/notify-listener:21: PyGIWarning: Notify was imported without specifying a version first. Use gi.require_version('Notify', '0.7') before import to ensure that the right version gets loaded.
from gi.repository import Notify, GObject
Traceback (most recent call last):
File "/usr/bin/notify-listener", line 25, in <module>
import dbus
ImportError: No module named dbus

After install, it worked with the same warning.

2. Update the instructions

The most important being number 5 without which no notifications arrived:
...
5. Start the listener int he background:
$ notify-listener &
...

Perhaps, copy-paste them from Github!

3. Suggestions

* I would rename the binary notify-listener to libnotify-irssi-listener in /usr/bin to avoid future name collisions.

* It seems quite useful to make a unit for systemctl that should be placed in .config/systemd/user/libnotify-irssi.service that would contain the below which would allow to start it with 'systemctl --user start irssi-libnotify' instead of a background task:

[Unit]
Description=Irssi-libnotify

[Service]
ExecStart=/usr/bin/notify-listener

[Install]
WantedBy=default.target

Cheers,
rrt

wookietreiber commented on 2015-07-27 14:55

Done. Thanks for letting me know.

Brottweiler commented on 2015-07-27 10:44

You should probably update this to use https://github.com/stickster/irssi-libnotify

gtmanfred commented on 2012-11-04 04:38

@dakanya, how are you starting X?

please make sure you are completely up to date and have merged all your pacnews.

https://wiki.archlinux.org/index.php/General_Troubleshooting#Session_permissions

dakanya commented on 2012-11-03 21:51

I'm trying to get this to work but this error has me stumped. I suspect it's something to do with systemd and dbus.

# Failed to open connection to "session" message bus: Unable to autolaunch a dbus-daemon without a $DISPLAY for X11

gtmanfred commented on 2012-04-27 21:56

daemons don't work right now, I am working on them