-
Notifications
You must be signed in to change notification settings - Fork 124
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
#include <sys/sysctl.h> deprecated from glibc on recent distros stil needed on mac. annd conditional import. #20
Comments
Could you propose a pull request to fix this? |
I`ll have a look at what i can do. Is it possible to just remove the lines all together ? That might be a better fix then adding extra bloat. |
I checked on CentOS7 and it compiles fine without the line. Mac needs it though, even on the most recent Ventura build. So I would suggest replacing the
with:
If you'd rather I do the PR, I can take care of it. |
There's a PR in review now that should solve this for you. I built it successfully on Mac Ventura 13.1 and CentOS7. |
Closes #20 The sysctl include is not needed on CentOS7 and as of glibc 2.32 it has been removed. So support distros/compilers later than that by removing it completely on Linux. Signed-off-by: Roger Nelson <roger.nelson@autodesk.com>
THX ! |
#include <sys/sysctl.h> Deprecated from glibc on EL9.
Please add conditional statement that checks how recent your system is to OpenRV/src/lib/base/TwkUtil/SystemInfo.cpp,
The text was updated successfully, but these errors were encountered: