Hi,

Thanks for developing libevent, it is very useful!

When trying to compile libevent 1.4.4 on HP-UX 11.31, I hit the
problem that "compat/sys/_time.h" name clashes with
"/usr/include/sys/_time.h".

The reason it is included, is that "event.c" includes
"/usr/include/time.h" that include "sys/_time.h".

My suggestion is, as "_time.h" is not a standard name anyway, that
"compat/_time.h" is renamed to "compat/libevent_time.h",
"compat/_lev_time.h" or similar. Or play catch up with adding another
"_", i.e. "compat/__time.h" :-)

Another related subject, I don't know if the idea with "#define
_SYS_TIME_H" is also to try avoid that other system headers indirectly
includes "/usr/include/sys/time.h" if a build is using
"compat/_time.h"? If that is the case, there are other variations of
that symbol that should be set in "_time.h", like

  _H_SYS_TIME         (AIX and i5/OS PASE)
  _SYS_TIME_INCLUDED  (HP-UX)
  _SVC_TIME_H         (OpenServer 6)  

kent
_______________________________________________
Libevent-users mailing list
Libevent-users@monkey.org
http://monkeymail.org/mailman/listinfo/libevent-users

Reply via email to