The inotify code uses the Linux-specific /proc/self/fd/<FD> symlinks to translate file descriptors to filesystem paths. On FreeBSD, do this translation using its own sysctl instead.
Signed-off-by: Damjan Jovanovic damjan.jov@gmail.com --- server/change.c | 132 ++++++++++++++++++++++++++++++++++++++++-------- 1 file changed, 111 insertions(+), 21 deletions(-)
On Fri, Dec 03, 2021 at 06:05:50AM +0200, Damjan Jovanovic wrote:
The inotify code uses the Linux-specific /proc/self/fd/<FD> symlinks to translate file descriptors to filesystem paths. On FreeBSD, do this translation using its own sysctl instead.
Signed-off-by: Damjan Jovanovic damjan.jov@gmail.com
server/change.c | 132 ++++++++++++++++++++++++++++++++++++++++-------- 1 file changed, 111 insertions(+), 21 deletions(-)
This is causing a build failure on macOS:
gcc -m32 -c -o server/change.o server/change.c -Iserver -Iinclude -D__WINESRC__ -Wall -pipe -fno-stack-protector \ -fno-strict-aliasing -Wdeclaration-after-statement -Wempty-body -Wignored-qualifiers -Winit-self \ -Wno-pragma-pack -Wstrict-prototypes -Wtype-limits -Wvla -Wwrite-strings -Wpointer-arith -gdwarf-4 \ -fno-omit-frame-pointer -O2 -g -Wno-deprecated-declarations -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=0 server/change.c:236:2: error: No fd_path() function for this platform #error No fd_path() function for this platform
The fd_path() stuff probably needs to be inside the #ifdef HAVE_SYS_INOTIFY_H block.
Huw.
On Fri, Dec 3, 2021 at 9:26 AM Huw Davies huw@codeweavers.com wrote:
On Fri, Dec 03, 2021 at 06:05:50AM +0200, Damjan Jovanovic wrote:
The inotify code uses the Linux-specific /proc/self/fd/<FD> symlinks to translate file descriptors to filesystem paths. On FreeBSD, do this translation
using
its own sysctl instead.
Signed-off-by: Damjan Jovanovic damjan.jov@gmail.com
server/change.c | 132 ++++++++++++++++++++++++++++++++++++++++-------- 1 file changed, 111 insertions(+), 21 deletions(-)
This is causing a build failure on macOS:
gcc -m32 -c -o server/change.o server/change.c -Iserver -Iinclude -D__WINESRC__ -Wall -pipe -fno-stack-protector \ -fno-strict-aliasing -Wdeclaration-after-statement -Wempty-body -Wignored-qualifiers -Winit-self \ -Wno-pragma-pack -Wstrict-prototypes -Wtype-limits -Wvla -Wwrite-strings -Wpointer-arith -gdwarf-4 \ -fno-omit-frame-pointer -O2 -g -Wno-deprecated-declarations -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=0 server/change.c:236:2: error: No fd_path() function for this platform #error No fd_path() function for this platform
The fd_path() stuff probably needs to be inside the #ifdef HAVE_SYS_INOTIFY_H block.
Huw.
No, that sysctl really is FreeBSD-only. MacOS, NetBSD and OpenBSD all implement fd->path translation in different, incompatible ways. MacOS's KERN_FILE sysctl doesn't appear to filter by pid ( https://developer.apple.com/library/archive/documentation/System/Conceptual/...), I think you are supposed to use the proc_pidinfo() function, see http://blog.palominolabs.com/2012/06/19/getting-the-files-being-used-by-a-pr...
I'll submit a new version with some fixes, but someone with a Mac will have to develop the Apple implementation.
Damjan
-
Damjan Jovanovic -
Huw Davies