gryf/wmaker
1
0
Fork 0
mirror of https://github.com/gryf/wmaker.git synced 2026-03-19 09:13:33 +01:00
Code Issues Projects Releases Wiki Activity

wmaker: filter inotify events only to expected config files

Browse Source 
This patch is checking if the notification wmaker is receiving
from the Defaults directory is coming from a proper expected config file.
Until now, using vim for example to open any of the files,
for example WMRootMenu would reload the configs, even before saving
the file.
You would see in the logs:
warning: Inotify: Reading config files in defaults database.

because vim by default is creating a .swp file in that same directory.
This commit is contained in:
David Maciejak
2026-03-03 20:45:16 -05:00
committed by Carlos R. Mafra
parent c6f029c946
commit b27b44db72
1 changed files with 22 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

26
src/event.c
View File

@@ -357,10 +357,28 @@ static void handle_inotify_events(void)
wPreferences.flags.noupdates = 1;
}
if ((pevent->mask & IN_MODIFY) && oneShotFlag == 0) {
wwarning(_("Inotify: Reading config files in defaults database."));
wDefaultsCheckDomains(NULL);
oneShotFlag = 1;
/* Only react when a known config file inside the Defaults directory is affected */
if (pevent->len > 0) {
char *fname = pevent->name;
const char *allowed[] = { "WMRootMenu", "WMWindowAttributes",
"WindowMaker", "WMState", "WMGLOBAL", NULL };
int i, matched = 0;
for (i = 0; allowed[i]; i++) {
if (strcmp(fname, allowed[i]) == 0) {
matched = 1;
break;
}
}
if (matched) {
/* react to events that indicate a file was created/moved/written */
if ((pevent->mask & (IN_MODIFY | IN_CLOSE_WRITE | IN_MOVED_TO | IN_CREATE)) && oneShotFlag == 0) {
wwarning(_("Inotify: Reading config files in defaults database."));
wDefaultsCheckDomains(NULL);
oneShotFlag = 1;
}
}
}
/* move to next event in the buffer */