diff options
Diffstat (limited to 'x11-wm/swayfx')
| -rw-r--r-- | x11-wm/swayfx/Makefile | 1 | ||||
| -rw-r--r-- | x11-wm/swayfx/files/extra-patch-wordexp | 60 |
2 files changed, 29 insertions, 32 deletions
diff --git a/x11-wm/swayfx/Makefile b/x11-wm/swayfx/Makefile index f53808ea6c88..8e4890d5cdef 100644 --- a/x11-wm/swayfx/Makefile +++ b/x11-wm/swayfx/Makefile @@ -1,5 +1,6 @@ PORTNAME= sway DISTVERSION= 0.5.3 +PORTREVISION= 1 CATEGORIES= x11-wm wayland PKGNAMESUFFIX= ${GH_PROJECT:S/${PORTNAME}//} diff --git a/x11-wm/swayfx/files/extra-patch-wordexp b/x11-wm/swayfx/files/extra-patch-wordexp index 5a9f187e11df..eb1f79f130f5 100644 --- a/x11-wm/swayfx/files/extra-patch-wordexp +++ b/x11-wm/swayfx/files/extra-patch-wordexp @@ -1,4 +1,5 @@ https://github.com/swaywm/sway/issues/8673#issuecomment-2818260191 +https://github.com/swaywm/sway/issues/8828#issuecomment-3290045852 --- common/stringop.c.orig 2025-04-20 11:31:10 UTC +++ common/stringop.c @@ -11,43 +12,38 @@ https://github.com/swaywm/sway/issues/8673#issuecomment-2818260191 #include "list.h" #include "log.h" #include "stringop.h" -@@ -318,11 +320,34 @@ bool expand_path(char **path) { - char *ptr = strstr(*path, " ") + 1; +@@ -319,7 +321,33 @@ bool expand_path(char **path) { memmove(ptr + 1, ptr, strlen(ptr) + 1); *ptr = '\\'; -- } + } - if (wordexp(*path, &p, 0) != 0 || p.we_wordv[0] == NULL) { -- wordfree(&p); -- return false; -- } -+ } + -+ // FreeBSD wordexp bug workaround: -+ // 1. Reset SIGCHLD handler -+ // 2. Run wordexp -+ // 3. Restore SIGCHLD handler and reap any processes that exited -+ struct sigaction prev_action; -+ struct sigaction new_action = { -+ .sa_flags = SA_RESETHAND, -+ }; -+ if (sigaction(SIGCHLD, &new_action, &prev_action) != 0) { -+ sway_log(SWAY_ERROR, "Restoring sigchild before wordexp failed"); -+ return false; -+ } ++ // FreeBSD wordexp bug workaround: ++ // 1. Reset SIGCHLD handler ++ // 2. Run wordexp ++ // 3. Restore SIGCHLD handler and reap any processes that exited ++ struct sigaction prev_action; ++ struct sigaction new_action = { ++ .sa_flags = SA_RESETHAND, ++ }; ++ if (sigaction(SIGCHLD, &new_action, &prev_action) != 0) { ++ sway_log(SWAY_ERROR, "Restoring sigchild before wordexp failed"); ++ return false; ++ } + -+ int res = wordexp(*path, &p, 0); ++ int res = wordexp(*path, &p, 0); + -+ if (sigaction(SIGCHLD, &prev_action, NULL) != 0) { -+ sway_log(SWAY_ERROR, "Restoring sigchild after wordexp failed"); -+ } ++ if (sigaction(SIGCHLD, &prev_action, NULL) != 0) { ++ sway_log(SWAY_ERROR, "Restoring sigchild after wordexp failed"); ++ } + -+ int status; -+ while (waitpid(-1, &status, WNOHANG) != -1 || errno == EINTR); ++ int status; ++ int wres; ++ do { ++ wres = waitpid(-1, &status, WNOHANG); ++ } while (wres > 0 || (wres == -1 && errno == EINTR)); + -+ if (res != 0 || p.we_wordv[0] == NULL) { -+ wordfree(&p); -+ return false; -+ } - free(*path); - *path = join_args(p.we_wordv, p.we_wordc); - wordfree(&p); ++ if (res != 0 || p.we_wordv[0] == NULL) { + wordfree(&p); + return false; + } |