This is an archive of the discontinued LLVM Phabricator instance.

[Support] Build fix for Haiku when checking for a local filesystem
ClosedPublic

Authored by nielx on Jul 6 2018, 2:35 AM.

Reviewers

chandlerc

Summary

Haiku does not expose information about local versus remote mounts, so just return false, like Cygwin.

LGTM

This revision is now accepted and ready to land.Jul 6 2018, 6:44 PM

Thank you for the review. I do not have commit access. Can you help me land this commit? Thanks!

I've committed this as r337389. Thanks for submitting the patch.

Path

Size

lib/

Support/

Unix/

3 lines

Show First 20 Lines • Show All 365 Lines • ▼ Show 20 Lines	default:
return true;		return true;
}		}
#elif defined(__CYGWIN__)		#elif defined(__CYGWIN__)
// Cygwin doesn't expose this information; would need to use Win32 API.		// Cygwin doesn't expose this information; would need to use Win32 API.
return false;		return false;
#elif defined(__Fuchsia__)		#elif defined(__Fuchsia__)
// Fuchsia doesn't yet support remote filesystem mounts.		// Fuchsia doesn't yet support remote filesystem mounts.
return true;		return true;
		#elif defined(__HAIKU__)
		// Haiku doesn't expose this information.
		return false;
#elif defined(__sun)		#elif defined(__sun)
// statvfs::f_basetype contains a null-terminated FSType name of the mounted target		// statvfs::f_basetype contains a null-terminated FSType name of the mounted target
StringRef fstype(Vfs.f_basetype);		StringRef fstype(Vfs.f_basetype);
// NFS is the only non-local fstype??		// NFS is the only non-local fstype??
return !fstype.equals("nfs");		return !fstype.equals("nfs");
#else		#else
return !!(STATVFS_F_FLAG(Vfs) & MNT_LOCAL);		return !!(STATVFS_F_FLAG(Vfs) & MNT_LOCAL);
#endif		#endif
▲ Show 20 Lines • Show All 636 Lines • Show Last 20 Lines