wandenberg / nginx-push-stream-module

A pure stream http push technology for your Nginx setup. Comet made easy and really scalable.
Other
2.21k stars 295 forks source link

compile failed on windows #308

Closed wangdbang closed 6 months ago

wangdbang commented 7 months ago

I compiled nginx with push stream module on windows 10 through VC2022 x86, but failed, printed "unexpected end of file while looking for precompiled header. Did you forget to add '#include ngx_conf.h' to your source?"

wangdbang commented 7 months ago

I tried nginx version 1.25.3, 1.23.4, 1.24.0, 1.20.2, all failed.

wandenberg commented 7 months ago

@wangdbang I have never tried to compile the module or even Nginx on Windows.
It looks like to be an issue of a reference to a file. Can you identify from which .c file is this ngx_conf.h being referenced?
Anyway, is it a requirement to run on Windows in a production environment?
The module is heavily dependent on an event machine, kqueue on MacOS and epoll on linux, on windows the alternative would be using the poll implementation of Nginx as far as I know, which would give you a very bad performance on the module.
So, unless it is really a must to run on Windows, I would recommend switching to a linux based OS.

wangdbang commented 7 months ago

@wandenberg , the referenced file was ngx_http_push_stream_module.c,as you said, switch to linux maybe a good choice, thanks.