Alexandre Julliard wrote:
Robert Shearman rob@codeweavers.com writes:
The point is that all of the files need it. Only tools/widl/parser.l needs to include unistd.h and it already does so manually.
Then you should put the option in all the files. That sort of thing doesn't belong in the global make rules.
I disagree. The global make rules, among other things, should enforce policies whether this be telling the C compiler to error on certain non-portable constructs like declarations after statements or telling flex not to output code that includes headers that might not be present, like unistd.h.
However, I've sent a patch that turns the option on individually for each file so that you can choose which route to take.