fix(v4): add missing sys/resource.h header for getrusage - #834
Conversation
|
You found this independently and you were right — thank you. It is already on #if defined(__APPLE__) || defined(__linux__) || defined(__FreeBSD__)
#include <sys/resource.h> /* getrusage/RUSAGE_SELF for v4_serve_rss_gb;
* on Windows compat.h supplies the shim. */
#endifThe guard is there because Windows has no Closing as already fixed, not as wrong. Two people finding the same missing header on the same day is a good sign for the project, and the next one you find will very likely not already be in flight. Separately, and unrelated to the code: thank you for what you wrote on #814. That reporter came back, retested, and confirmed the model reasons correctly — which would not have happened if he had been argued out of the thread first. |
Fixes a compilation break on Linux/WSL platforms where struct rusage and getrusage storage layouts are unrecognized.