Bug 330034

Summary: Debian kfreebsd fails to compile the acl support
Product: [Unmaintained] kio Reporter: Maximiliano Curia <maxy>
Component: generalAssignee: David Faure <faure>
Status: RESOLVED NOT A BUG    
Severity: normal CC: nate
Priority: NOR    
Version First Reported In: 4.11.5   
Target Milestone: ---   
Platform: Debian unstable   
OS: Other   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

Description Maximiliano Curia 2014-01-16 13:38:21 UTC
Hi,

Debian kfreebsd is a system that runs the gnu stack on top of the freebsd kernel, so things like Q_OS_FREEBSD, which are commonly used to detect a Freebsd kernel, are not defined. But,
__FreeBSD_kernel__ is.

In particular, kio/kfile/kpropertiesdialog.cpp static bool fileSystemSupportsACL( const QByteArray& path ) at line 1896, checks for #ifdef Q_OS_FREEBSD while it should check for #ifdef __FreeBSD_kernel__, as it's using the kernel api.
Also, note that it needs to include the additional headers:
// This is the *BSD branch
#ifdef HAVE_SYS_MOUNT_H
#ifdef HAVE_SYS_PARAM_H
#include <sys/param.h>
#endif
#include <sys/mount.h>
#endif

Thanks,



Reproducible: Always
Comment 1 Nate Graham 2018-05-03 20:41:26 UTC
The Debian variant in question has been discontinued, so this is no longer relevant or needed.