This patch uses a "belt & suspenders" approach: either the inclusion of

<sys/types.h> before the define, or changing the variable name is sufficient,

but both are used out of precaution. The patch was developed in-house;

the changing of the variable name part has been fed upstream and will be

included in the next version.

--- xmlto-0.0.28/xmlif/xmlif.c.~1~ 2015-11-10 04:05:27.000000000 -0800

+++ xmlto-0.0.28/xmlif/xmlif.c 2016-03-14 13:33:03.905766962 -0700

@@ -20,6 +20,7 @@

#include <string.h>

#include <errno.h>

#include <stdlib.h>

+#include <sys/types.h>

/* end standard C headers. */

@@ -1306,7 +1307,7 @@

#define INITIAL 0

#define attrib 1

-#define val 2

+#define xmlif_val 2

#ifndef YY_NO_UNISTD_H

/* Special case for "unistd.h", since it is non-ANSI. We include it way

@@ -1594,7 +1595,7 @@

case 8:

YY_RULE_SETUP

#line 217 "xmlif/xmlif.l"

-{BEGIN(val);}

+{BEGIN(xmlif_val);}

YY_BREAK

case 9:

YY_RULE_SETUP

@@ -1636,7 +1637,7 @@

#line 1637 "xmlif/xmlif.c"

case YY_STATE_EOF(INITIAL):

case YY_STATE_EOF(attrib):

-case YY_STATE_EOF(val):

+case YY_STATE_EOF(xmlif_val):

yyterminate();

case YY_END_OF_BUFFER: