On Thu, 23 Apr 2015 07:22:12 -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]> > Signed-off-by: Gurucharan Shetty <[email protected]> > --- > AUTHORS | 1 + > INSTALL.md | 2 -- > tests/automake.mk | 13 ++++++++++--- > 3 files changed, 11 insertions(+), 5 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). > - > 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..c302a4b 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 $@ > + $(AM_V_GEN)$(AUTOTEST) -I '$(srcdir)' -o [email protected] [email protected] > + patch -p0 [email protected] $(TESTSUITE_PATCH) > + $(AM_V_at)mv [email protected] $@ > +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 This reverts the previous fix to build in another directory. s/ -o testsuite.tmp/ -o [email protected]/ Since it's now only used on Windows I don't care, but still I think 'patch' will break on Windows too in the same scenario. fbl > > $(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
