Hi Lance,

Lance Richardson <lrich...@redhat.com> writes:

> Add minimimal support for building under OS X.  With this series,
> "make check" reports:
>
> ERROR: 1831 tests were run, 30 failed (1 expected failure).  199 tests were 
> skipped.
> testsuite: 232 368 394 395 396 397 398 740 910 920 1018 1019 1020 1021 1022 
> 1023 1084 1343 1951 2013 2014 2015 2016 2025 2026 2027 2028 2029 2030 failed
>
> Changes from v1:
>    - Combined v1 patches 4 and 5 into v2 patch 4. This was partly to
>      address the issue of patch 4 having commit comments for a file in patch
>      5 and vice-versa.
>    - Added an implementation of RT_ROUNDUP() for OS X, no longer punting
>      in route_table_fallback_lookup(). Untested.
>    - Added patch to avoid collision between system ntohll() and htonll()
>      (should have been in v1, I somehow dropped it via "git rebase -i")
>
> Lance Richardson (4):
>   ofproto-dpif: rename wait() to avoid collision with system wait(2)
>   timeval: Add clock_gettime() for OS X
>   utilities: OS X compatibility
>   osx: handle differences between OS X and other BSDs
>   byte-order: use system ntohll() and htonll() for OS X
>
>  lib/byte-order.h       |  4 ++--
>  lib/netdev-bsd.c       |  2 ++
>  lib/route-table-bsd.c  |  7 +++++++
>  lib/rtbsd.c            |  4 ++++
>  lib/timeval.c          | 33 ++++++++++++++++++++++++++++++++-
>  ofproto/ofproto-dpif.c |  4 ++--
>  utilities/ovs-pki.in   |  2 +-
>  7 files changed, 50 insertions(+), 6 deletions(-)

I think you should also add an osx entry to the .travis.yml as follows
to integrate with the travis build environment (which I guess supports
OS X these days). Shouldn't hold this up, but maybe as a follow up
series.

diff --git a/.travis.yml b/.travis.yml
index 52c9362..ba5f851 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -3,6 +3,10 @@ compiler:
   - gcc
   - clang
 
+os:
+  - linux
+  - osx
+
 addons:
   apt:
     packages:
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to