diff mbox

[ovs-dev,02/21] ofp-actions: Define IPPORT_FTP by ourselves

Message ID 1445228952-22445-2-git-send-email-yamamoto@midokura.com
State Awaiting Upstream
Headers show

Commit Message

Takashi Yamamoto Oct. 19, 2015, 4:28 a.m. UTC
IPPORT_FTP is not always provided by system headers.  (eg. NetBSD, OS X)

This hides the enum on Linux but I don't think it causes a problem.

Signed-off-by: YAMAMOTO Takashi <yamamoto@midokura.com>
---
 lib/ofp-actions.h | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Ben Pfaff Nov. 9, 2015, 11:21 p.m. UTC | #1
On Mon, Oct 19, 2015 at 01:28:53PM +0900, YAMAMOTO Takashi wrote:
> IPPORT_FTP is not always provided by system headers.  (eg. NetBSD, OS X)
> 
> This hides the enum on Linux but I don't think it causes a problem.
> 
> Signed-off-by: YAMAMOTO Takashi <yamamoto@midokura.com>

I don't think ofp-actions.h is an ideal place for this, but it seems
good enough for now.

Acked-by: Ben Pfaff <blp@ovn.org>
diff mbox

Patch

diff --git a/lib/ofp-actions.h b/lib/ofp-actions.h
index 773b617..f5a5b78 100644
--- a/lib/ofp-actions.h
+++ b/lib/ofp-actions.h
@@ -499,6 +499,10 @@  struct {                                \
     uint8_t recirc_table;               \
 }
 
+#if !defined(IPPORT_FTP)
+#define	IPPORT_FTP  21
+#endif
+
 /* OFPACT_CT.
  *
  * Used for NXAST_CT. */