Bug 11939 - Alternative fix for bug 11816, that includes sys/filio.h
Summary: Alternative fix for bug 11816, that includes sys/filio.h
Status: NEW
Alias: None
Product: Samba 4.1 and newer
Classification: Unclassified
Component: Build (show other bugs)
Version: 4.4.3
Hardware: All Solaris
: P5 trivial (vote)
Target Milestone: ---
Assignee: Samba QA Contact
QA Contact: Samba QA Contact
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-05-28 10:45 UTC by Jorge Schrauwen
Modified: 2016-05-29 10:41 UTC (History)
2 users (show)

See Also:

Attachments
source3 source patch (331 bytes, text/plain)
2016-05-28 10:45 UTC, Jorge Schrauwen 		no flags Details
source3 build patch (1.04 KB, patch)
2016-05-28 10:45 UTC, Jorge Schrauwen 		no flags Details
source4 source patch (324 bytes, patch)
2016-05-28 10:46 UTC, Jorge Schrauwen 		no flags Details
source4 build patch (897 bytes, patch)
2016-05-28 10:46 UTC, Jorge Schrauwen 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Jorge Schrauwen 2016-05-28 10:45:17 UTC 
Created attachment 12138 [details]
source3 source patch

According to Youzhong Yang, the inotify support is good enough that samba is happy with it.

Attached here is a different set of patches that only include sys/filio.h if HAVE_SYS_FILIO_H is defined.

The checks are also update to check and set HAVE_SYS_FILIO_H when running on illumos.
Comment 1 Jorge Schrauwen 2016-05-28 10:45:42 UTC 
Created attachment 12139 [details]
source3 build patch
Comment 2 Jorge Schrauwen 2016-05-28 10:46:00 UTC 
Created attachment 12140 [details]
source4 source patch
Comment 3 Jorge Schrauwen 2016-05-28 10:46:18 UTC 
Created attachment 12141 [details]
source4 build patch