OpenBSD support

[bigodel]

I love your script but unfortunately it doesn't work on OpenBSD. It starts with the fact that since bash isn't native to OpenBSD, the path to bash is actually /usr/local/bin/bash, because it installs every third party package at /usr/local/bin. I'll try, using your clipnotify (I was actully going to implement my own, but yours works very well and is pretty simple, I'll just maybe incorporate it into the same repo and try to install it system wide if possible) to implement it in strictly POSIX shell, in order for it to work on as many *nix systems as possible.

I will close this issue, I just wanted to tell you that I really love the simplicity of your approach and that unfortunately it doesn't work on my OS (and that this gave me inspiration to try and make one that works on my machine :P).

Best regards, João Pedro.

[cdown]

I believe both of these should allow it to work (the pledge commit is not required but is good convention on OpenBSD):

• clipmenu: 7c3bd5060ab9475e5791fd0fa756110362ddd89c
• clipnotify: https://github.com/cdown/clipnotify/commit/eb156445572bad2eec7ac6f2dd7cd21d2eaaca6e

Try out these commits and let me know if they don't work on OpenBSD :-) I have no OpenBSD machine readily available to test, but they should work.

[bigodel]

The bash issue is fixed, but there are still some problems. On clipnotify, I need to add -I/usr/X11R6/include and -L/usr/X11R6/lib to the compiling command, because that's the default location of X11 in OpenBSD. That's something I haven't managed to fix yet, so if you find a solution to make it run on a system where X11 is installed in a different location, I'd be pleased to know.

Regarding clipmenud, timeout is a GNU coreutils program, and I couldn't find it in the ports or packages to install on OpenBSD. I did install coreutils though, and after adding the correct flags to the compiling command of clipnotify and changing timeout to gtimeout on clipmenud everything works fine!

[cdown]

• f748a2e removes hard dependency on GNU timeout.
• https://github.com/cdown/clipnotify/commit/36ee24aafb70f5d0aa9216139b929bfbd18a9f92 adds lib/include dirs for OpenBSD. They're harmless when these directories don't exist.

Anything else to do? If not, I'll make new releases of both. :-)

[bigodel]

clipmenu and clipmenud both work fine.

clipnotify on the other hand compiles but dumps its core on execution due to the pledge("stdio rpath", NULL) call.

[cdown]

I probably was overzealous with the pledging. Can you try with a few other possible values? Maybe start with the following and whittle it down until it SIGABRTs again:

pledge("stdio rpath wpath proc")

That said, I'm pretty surprised existing pledge call isn't enough, maybe there is some interesting behaviour inside one of the xlib calls...

[bigodel]

Well, no matter what value I put in the pledge call it aborts. I tried including err.h and using the err() function instead of perror() but it made no difference. Debugging on gdb I found this:

Breakpoint 1, main () at clipnotify.c:21
21          if (MAYBE_PLEDGE("stdio rpath wpath proc", NULL) < 0) {
(gdb) s
pledge () at -:3
3       -: No such file or directory.

Hope it helps, I couldn't really understand what it meant.

[bigodel]

Oh, and I also added #include <unistd.h> to the true case in the #ifdef __OpenBSD__ to silence the implicit declaration of function pledge warning and -g -ggdb -O0 to the Makefile in order to debug.

[bigodel]

I think it might be better to open an issue on the clipnotify repository, since all issues with clipmenu are now fixed.

[cdown]

Yeah, sure. For now I'll just remove the pledge call and release both. Thanks for bringing this up :)

[cdown]

clipmenu 5.5.1 released, clipnotify 1.0.2 released.