On Thu, 23 Apr 2015 08:42:47 -0700
Gurucharan Shetty <[email protected]> wrote:

> On CentOS machines which use autoconf version 2.63, the patch
> application would fail.
> 
> Reported-by: Ian Stokes <[email protected]>
> Tested-by: Ian Stokes <[email protected]>
> Signed-off-by: Gurucharan Shetty <[email protected]>
> ---
>  AUTHORS           |    1 +
>  INSTALL.md        |    2 --
>  tests/automake.mk |    7 +++++++
>  3 files changed, 8 insertions(+), 2 deletions(-)
> 
> diff --git a/AUTHORS b/AUTHORS
> index a2b4388..8973dec 100644
> --- a/AUTHORS
> +++ b/AUTHORS
> @@ -255,6 +255,7 @@ Henrik Amren            [email protected]
>  Hiroshi Tanaka          [email protected]
>  Hiroshi Miyata          [email protected]
>  Hyojoon Kim             [email protected]
> +Ian Stokes              [email protected]
>  Igor Ganichev           [email protected]
>  Igor Sever              [email protected]
>  Jacob Cherkas           [email protected]
> diff --git a/INSTALL.md b/INSTALL.md
> index 8156898..9f8c57e 100644
> --- a/INSTALL.md
> +++ b/INSTALL.md
> @@ -45,8 +45,6 @@ you will need the following software:
>  
>    - Python 2.x, for x >= 4.
>  
> -  - patch (The utility that is used to patch files).

The patch is still needed on Windows, so I suggest to keep it with
a reference for "Windows-only" or move the requirement to
INSTALL.Windows.md.

Otherwise the patch looks good to me.
Thanks
fbl

>  On Linux, you may choose to compile the kernel module that comes with
>  the Open vSwitch distribution or to use the kernel module built into
>  the Linux kernel (version 3.3 or later).  See the [FAQ.md] question
> diff --git a/tests/automake.mk b/tests/automake.mk
> index 7c699b2..e63fb19 100644
> --- a/tests/automake.mk
> +++ b/tests/automake.mk
> @@ -200,10 +200,17 @@ clean-local:
>       test ! -f '$(TESTSUITE)' || $(SHELL) '$(TESTSUITE)' -C tests
> --clean 
>  AUTOTEST = $(AUTOM4TE) --language=autotest
> +
> +if WIN32
>  $(TESTSUITE): package.m4 $(TESTSUITE_AT) $(COMMON_MACROS_AT)
> $(TESTSUITE_PATCH) $(AM_V_GEN)$(AUTOTEST) -I '$(srcdir)' -o
> testsuite.tmp [email protected] patch -p0 testsuite.tmp $(TESTSUITE_PATCH)
>       $(AM_V_at)mv testsuite.tmp $@
> +else
> +$(TESTSUITE): package.m4 $(TESTSUITE_AT) $(COMMON_MACROS_AT)
> +     $(AM_V_GEN)$(AUTOTEST) -I '$(srcdir)' -o [email protected] [email protected]
> +     $(AM_V_at)mv [email protected] $@
> +endif
>  
>  $(KMOD_TESTSUITE): package.m4 $(KMOD_TESTSUITE_AT)
> $(COMMON_MACROS_AT) $(AM_V_GEN)$(AUTOTEST) -I '$(srcdir)' -o [email protected]
> [email protected]


_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to