Project

General

Profile

Bug #2735

lighttpd 1.4.40 compilation fails on illumos (OpenIndiana)

Added by alp over 3 years ago. Updated over 3 years ago.

Status:
Fixed
Priority:
Normal
Assignee:
-
Category:
core
Target version:
Start date:
2016-07-18
Due date:
% Done:

0%

Estimated time:
Missing in 1.5.x:

Description

Compilation fails, because FIONREAD is not defined. Attached patch fixes the issue.

09-fileio.h.patch (1.2 KB) 09-fileio.h.patch alp, 2016-07-18 07:47

History

#1

Updated by gstrauss over 3 years ago

  • Category set to core
  • Status changed from New to Patch Pending
  • Target version changed from 1.4.x to 1.4.41

Sorry about that, and thanks for the patch. I hadn't found any modern references to filio.h on the internets and removed it in development some months ago. It will be added back in the next version of lighttpd.

#2

Updated by gstrauss over 3 years ago

I'll add it in the header which contains #include <sys/ioctl.h>

diff --git a/src/sys-socket.h b/src/sys-socket.h
index 04c9d94..e3ca029 100644
--- a/src/sys-socket.h
+++ b/src/sys-socket.h
@@ -21,6 +21,11 @@
 #include <arpa/inet.h>

 #include <netdb.h>
+
+#ifdef HAVE_SYS_FILIO_H
+#include <sys/filio.h>  /* FIONREAD (for illumos (OpenIndiana)) */
+#endif
+
 #endif

 #endif
#3

Updated by gstrauss over 3 years ago

  • Status changed from Patch Pending to Fixed

Fix committed to master

Revision a3ec906e

 
[core] #include <sys/filio.h> for FIONREAD (fixes #2735)

illumos (OpenIndiana) gets FIONREAD from <sys/filio.h>

Also available in: Atom