Add support for QNX build environment and make Linux-only functions more POSIX-like.#149
Add support for QNX build environment and make Linux-only functions more POSIX-like.#149mgorchak-blackberry wants to merge 2 commits intoKhronosGroup:mainfrom
Conversation
…like to make them work on variety POSIX-compatible platforms.
|
|
|
Hi @mgorchak-blackberry, thank you for your contribution (and for your patience)! We discussed this PR in our weekly teleconference and our biggest concern is maintaining this build environment. Would it be possible to add this configuration to our CI? If not, we can merge this PR and we'll make our best effort not to break it, but it's possible we'll inadvertantly introduce a build break because it's not something we'll test regularly. Thanks! |
| // attempt to load all files in the directory | ||
| for (dirEntry = readdir(dir); dirEntry; dirEntry = readdir(dir) ) | ||
| { | ||
| #if defined(DT_REG) |
There was a problem hiding this comment.
This change is a bit hard to follow because it breaks up a switch statement with an #ifdef. Would it be possible to refactor this code slightly to increase readability / maintainability? I think this could look something like: Initialize a flag to "false", set it to "true" in the #ifdef code when appropriate, then check the flag outside of the #ifdefs.
3 participants
Hello!
This PR consist of two patches, first adds "build-qnx" folder to the project to provide support for QNX platform, second brings more POSIX like behavior to the linux platform functions what makes them usable under any POSIX compliant OS. Not all operation systems have extended "dirent" structure to obtain filesystem entries details and type.
Thank you!