Reason for glslang/MachineIndependent/unistd.h?

[mgadams]

Reason for glslang/MachineIndependent/unistd.h?

[xorgy]

Seems it's been there since 3DLabs last touched glslang in 2005.

In that import, the README.txt talks about why the header is there.

2.  You may have a missing header file, unistd.h.  Create an empty one in
some standard devstudio or SDK system include directory.  (Bison/flex
generate a #include <unistd.h>, this makes them happy.)

I don't think this happens anymore, so it should be safe to remove unistd.h. Though I have not tested flex, and I have not tested versions of bison older than than 3.0.4.

[mgadams]

I think you're correct. The one platform I'd be slightly concerned about would be Windows, and I confirmed that the bison.exe in the tools directory doesn't generate code including unistd.h.

Given that, I've created pull request #88 to remove it.

[johnkslang]

I think this is a problem from 2003 that does not exist in 2015. Fixed with pull request #88.

(Flex is no longer part of glslang.)

I suspect the existing MSVC project files should also be removed. Only CMake is being maintained.