I don't know if it matters, but we usually have
if ...
pledge("some list"...
else ...
pledge("other list"...
rather than using a variable.
On 2018/01/13 16:14, Timo Myyrä wrote:
> Hi,
>
> Re-sending diff to update my email and fixing the pledge in dunst when using
> icons.
>
> timo
>
> Index: Makefile
> ===================================================================
> RCS file: /cvs/ports/x11/dunst/Makefile,v
> retrieving revision 1.3
> diff -u -p -r1.3 Makefile
> --- Makefile 31 Aug 2017 20:57:00 -0000 1.3
> +++ Makefile 17 Dec 2017 15:13:56 -0000
> @@ -5,12 +5,13 @@ COMMENT= customizable and lightweight no
> GH_ACCOUNT = dunst-project
> GH_PROJECT = dunst
> GH_TAGNAME = v1.2.0
> +REVISION = 0
>
> CATEGORIES = x11
>
> HOMEPAGE= https://dunst-project.org/
>
> -MAINTAINER = Timo Myyra <[email protected]>
> +MAINTAINER = Timo Myyra <[email protected]>
>
> # BSD
> PERMIT_PACKAGE_CDROM= Yes
> Index: patches/patch-src_dunst_c
> ===================================================================
> RCS file: /cvs/ports/x11/dunst/patches/patch-src_dunst_c,v
> retrieving revision 1.1
> diff -u -p -r1.1 patch-src_dunst_c
> --- patches/patch-src_dunst_c 31 Aug 2017 20:57:00 -0000 1.1
> +++ patches/patch-src_dunst_c 17 Dec 2017 15:13:56 -0000
> @@ -13,13 +13,21 @@ Index: src/dunst.c
> #include "dbus.h"
> #include "menu.h"
> #include "notification.h"
> -@@ -325,6 +328,9 @@ int dunst_main(int argc, char *argv[])
> - int owner_id = initdbus();
> -
> - x_setup();
> -+
> -+ if (pledge("stdio rpath proc exec", NULL) == -1)
> +@@ -368,6 +371,17 @@ int dunst_main(int argc, char *argv[])
> + * graceful termination */
> + guint term_src = g_unix_signal_add(SIGTERM, quit_signal, NULL);
> + guint int_src = g_unix_signal_add(SIGINT, quit_signal, NULL);
> ++
> ++ /* allow prot_exec if icons are used */
> ++ char *pledge_str;
> ++ if (settings.icon_position != icons_off) {
> ++ pledge_str = "stdio rpath proc exec prot_exec";
> ++ } else {
> ++ pledge_str = "stdio rpath proc exec";
> ++ }
> ++
> ++ if (pledge(pledge_str, NULL) == -1)
> + err(1, "pledge");
>
> - if (settings.startup_notification) {
> - notification *n = notification_create();
> + run(NULL);
> + g_main_loop_run(mainloop);
>
