search

murgo / IrssiNotifier

notifies android phones from irc private messages
Apache License 2.0
169 stars 63 forks source link

Fixed bug where screen socket path wasn't found #184

Closed benedicteb closed 5 years ago

benedicteb commented 8 years ago

It seems as if the script wasn't able to find the screen socket path due to the regex being a bit too harsh. I changed the regex a bit - made it multiline and added optional symbols on both sides to make it a bit more robust.

I tested it on my Fedora 23 and it works well.

This bug might have been triggered by a new version of screen since I know I fixed something similar in the screen_away script earlier.

Hope this is useful!

murgo commented 8 years ago

Thanks. I'll merge this soonish, too bad I'm not too happy about the current state of the irssinotifier.pl because last merge caused some problems with some people... I'll probably revert those earlier changes and merge this on top of that.

Zmegolaz commented 7 years ago

Anythoughts to merging this? I just ran into it, took me a long time to figure out what it was and fix it.