Thanks Maxim and Dmitry

Yes, these patches actually fix problems in installcheck, I'll add a
.travis installcheck target as Maxim suggested, run local tests and then
update the PR.

2) Would it make sense to run make installcheck with all possible
options enabled?

[]: Found that --with-dpdk-path=`pwd`/dpdk/${TARGET}
--with-netmap-path=`pwd`/netmap introduces further failures in
installcheck, if needed I'll add the patches in this PR.

Best Regards, Yi


On 21 June 2017 at 05:44, Dmitry Eremin-Solenikov <
[email protected]> wrote:

> On 20.06.2017 22:53, Maxim Uvarov wrote:
> > Yi, how about something like:
> >
> > diff --git a/.travis.yml b/.travis.yml
> > index 50b3b9c2..3d641209 100644
> > --- a/.travis.yml
> > +++ b/.travis.yml
> > @@ -139,6 +139,13 @@ jobs:
> >                            - ./configure
> >                            - sudo
> > LD_LIBRARY_PATH="/usr/local/lib:$LD_LIBRARY_PATH" make distcheck
> >                  - stage: test
> > +                  env: TEST=install-check
> > +                  compiler: gcc
> > +                  script:
> > +                          - ./bootstrap
> > +                          - ./configure
> > +                          - sudo
> > LD_LIBRARY_PATH="/usr/local/lib:$LD_LIBRARY_PATH" make install-check
>
> 1) There is no install-check target, only installcheck (at least with
> Automake 1.15)
>
> 2) Would it make sense to run make installcheck with all possible
> options enabled?
>
> > +                - stage: test
> >                    env: TEST=doxygen
> >                    compiler: gcc
> >                    script:
> >
> >
> > On 06/19/17 11:00, Github ODP bot wrote:
> >> From: Yi He <[email protected]>
> >>
> >> enable samples and tests when make distcheck to
> >> confirm samples and test programs installation and
> >> distribution builds.
> >>
> >> Signed-off-by: Yi He <[email protected]>
> >> ---
> >> /** Email created from pull request 35 (heyi-linaro:install-check)
> >>  ** https://github.com/Linaro/odp/pull/35
> >>  ** Patch: https://github.com/Linaro/odp/pull/35.patch
> >>  ** Base sha: 4bda102286cf55b0bc93fdf8f8f276d5528fdf14
> >>  ** Merge commit sha: 44aa7de8407d751055353d99ed665dc7733f8dbb
> >>  **/
> >>  .travis.yml | 2 +-
> >>  1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/.travis.yml b/.travis.yml
> >> index 50b3b9c2..8407906e 100644
> >> --- a/.travis.yml
> >> +++ b/.travis.yml
> >> @@ -136,7 +136,7 @@ jobs:
> >>                    compiler: gcc
> >>                    script:
> >>                            - ./bootstrap
> >> -                          - ./configure
> >> +                          - ./configure --prefix=$HOME/odp-install
> --enable-test-cpp --enable-test-vald --enable-test-helper
> --enable-test-perf --enable-test-perf-proc --enable-test-example
> --with-dpdk-path=`pwd`/dpdk/${TARGET} --with-netmap-path=`pwd`/netmap
> >>                            - sudo 
> >> LD_LIBRARY_PATH="/usr/local/lib:$LD_LIBRARY_PATH"
> make distcheck
> >>                  - stage: test
> >>                    env: TEST=doxygen
> >>
> >
>
>
> --
> With best wishes
> Dmitry
>

Reply via email to