Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mini_event.h:142:17: error: field 'ev_timeout' has incomplete type #189

Closed
noloader opened this issue Mar 11, 2020 · 1 comment
Closed

Comments

@noloader
Copy link
Contributor

noloader commented Mar 11, 2020

Hi Everyone, @wcawijngaards,

I'm testing Master on NetBSD 8.1. configure goes well, but make results in:

libtool: compile:  gcc -I. -I/usr/pkg/include -I/usr/local/include -DSRCDIR=. -g
 -O2 -flto -pthread -c ./util/mini_event.c  -fPIC -DPIC -o .libs/mini_event.o
In file included from ./util/mini_event.c:44:0:
./util/mini_event.h:142:17: error: field 'ev_timeout' has incomplete type
  struct timeval ev_timeout;
                 ^
*** [mini_event.lo] Error code 1

DragonFly 5.6, OpenBSD 12.1, FreeBSD 12.1 are OK.

@noloader
Copy link
Contributor Author

noloader commented Mar 11, 2020

This clears the compile error on NetBSD 8.1:

$ git diff util/mini_event.h
diff --git a/util/mini_event.h b/util/mini_event.h
index 204894d9..1734ca57 100644
--- a/util/mini_event.h
+++ b/util/mini_event.h
@@ -54,6 +54,8 @@
 
 #if defined(USE_MINI_EVENT) && !defined(USE_WINSOCK)
 
+#include <sys/time.h>
+
 #ifndef HAVE_EVENT_BASE_FREE
 #define HAVE_EVENT_BASE_FREE
 #endif 

Self tests are OK on NetBSD 8.1.

gthess added a commit that referenced this issue Mar 11, 2020
@gthess gthess closed this as completed in 67b4ab2 Mar 11, 2020
jedisct1 added a commit to jedisct1/unbound that referenced this issue Mar 20, 2020
* nlnet/master: (149 commits)
  - Fix .travis.yml error, missing 'env' option.
  - Merge PR#194: Add libevent testing to Travis, by Jeffrey Walton.
  Add changelog entries for PR#134.
  - Log warning when using outgoing-port-permit and outgoing-port-avoid   while explicit port randomisation is disabled.
  - Fix NLnetLabs#158: open tls-session-ticket-keys as binary, for Windows. By Daisuke   HIGASHI.
  - Merge PR#191: Update iOS testing on Travis, by Jeffrey Walton.
  Add libevent testing to Travis
  Sync with upstream
  - Fix NLnetLabs#192: In the unbound-checkconf tool, the module config of   dns64 subnetcache respip validator iterator is whitelisted, it was   reported it seems to work.
  - Fix compile of test tools without protobuf.
  - Add check to make sure RPZ records are subdomain of configured zone origin.
  - Changelog entry for (Fix NLnetLabs#189, Merge PR NLnetLabs#190).
  Changelog for NLnetLabs#188 and configure script created.  Removed unneeded whitespace.
  Fix NLnetLabs#188: unbound-control.c:882:6: error: 'execlp' is unavailable: not available on tvOS
  Fix NetBSD compile (GH NLnetLabs#189)
  - Changelog note for PR NLnetLabs#186: Fix unrecognized 'echo -n' option on OS X,   by noloader.
  Fix unrecognized 'echo -n' option on OS X Also see NLnetLabs#183. This PR also updates a few typos in README-Travis.md, and expands the discussion of PKG_CONFIG_PATH for those who are not familiar with it.
  Fix changelog note, it is NLnetLabs#182, not NLnetLabs#184.
  Changelog note for NLnetLabs#184. - Fix PR NLnetLabs#184 from noloader: Add iOS testing to Travis.
  Add iOS testing to Travis
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant