-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Unable to build on OmniOS due to missing PATH_MAX definition #1344
Comments
I think we should just include it unconditionally, that should be safe. Hopefully... I'll give it a go. |
On OmniOS, compilation fails because of a missing PATH_MAX definition: $ gmake CC cconv.o In file included from stat.h:6:0, from thread_options.h:7, from cconv.c:4: diskutil.h:52:12: error: 'PATH_MAX' undeclared here (not in a function); did you mean 'INT8_MAX'? char path[PATH_MAX]; ^~~~~~~~ INT8_MAX gmake: *** [Makefile:505: cconv.o] Error 1 Add limits.h to fix that. Link: #1344 Signed-off-by: Jens Axboe <axboe@kernel.dk>
Yeah, it should be harmless, but I figured I would minimize the affect at least in my own testing. It seems like we are trying not to have any system level includes in this file, which was another reason for trying to limit blast radius. |
I did push a fix, so feel free to close this one provided it works for you now. |
Yup, LGTM. Thanks a lot for getting this incorporated quickly, I appreciate it! |
Please acknowledge the following before creating a ticket
Description of the bug:
Observing the following when no changes are made building from master.
With the following change I am able to build without a problem.
This is is likely not the most correct approach, but it works. I would not mind fixing this so that it correctly built on Omni.
Environment:
fio version: Latest ref on the master branch
Reproduction steps
The text was updated successfully, but these errors were encountered: