Yes, thanks.

Acked-by: Ben Pfaff <b...@nicira.com>


On Tue, Apr 22, 2014 at 12:54:31PM -0700, Gurucharan Shetty wrote:
> Something like this, right?
> 
> diff --git a/tests/atlocal.in b/tests/atlocal.in
> index 06e7384..60f3dbf 100644
> --- a/tests/atlocal.in
> +++ b/tests/atlocal.in
> @@ -78,3 +78,13 @@ FreeBSD|NetBSD)
>      LOOPBACK_INTERFACE=lo0
>      ;;
>  esac
> +
> +# Check for MINGW platform.
> +case `uname` in
> +MINGW*)
> +    IS_WIN32="yes"
> +    ;;
> +*)
> +    IS_WIN32="no"
> +    ;;
> +esac
> 
> On Tue, Apr 22, 2014 at 9:01 AM, Ben Pfaff <b...@nicira.com> wrote:
> > On Fri, Apr 18, 2014 at 11:04:08AM -0700, Gurucharan Shetty wrote:
> >> Signed-off-by: Gurucharan Shetty <gshe...@nicira.com>
> >> ---
> >>  tests/atlocal.in |    3 +++
> >>  1 file changed, 3 insertions(+)
> >>
> >> diff --git a/tests/atlocal.in b/tests/atlocal.in
> >> index 06e7384..8267554 100644
> >> --- a/tests/atlocal.in
> >> +++ b/tests/atlocal.in
> >> @@ -77,4 +77,7 @@ Linux)
> >>  FreeBSD|NetBSD)
> >>      LOOPBACK_INTERFACE=lo0
> >>      ;;
> >> +MINGW*)
> >> +    IS_WIN32="yes"
> >> +    ;;
> >>  esac
> >
> > This seems a little awkward as an addition to code that mainly gets the
> > name of the loopback interface.  I'd put it in a separate case
> > statement, and set IS_WIN32=no in the other cases.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to