On 02/15/17 20:49, Mike Holmes wrote: > > > On 14 February 2017 at 15:32, Maxim Uvarov <maxim.uva...@linaro.org > <mailto:maxim.uva...@linaro.org>> wrote: > > Signed-off-by: Maxim Uvarov <maxim.uva...@linaro.org > <mailto:maxim.uva...@linaro.org>> > --- > .travis.yml | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/.travis.yml b/.travis.yml > index ef9c3283..d7f4c66e 100644 > --- a/.travis.yml > +++ b/.travis.yml > @@ -100,6 +100,9 @@ before_install: > - make > /dev/null > - sudo insmod ./netmap.ko > - popd > +# Download tool to check odp contibutor agreement > + - git clone > https://git.linaro.org/people/maxim.uvarov/odp-agreement.git > <https://git.linaro.org/people/maxim.uvarov/odp-agreement.git> > > > Should this just be in odp/scripts and thus self contained ? If not > should it be in an official Linaro repo rather than a private one ? > > I vote in odp/scripts becasue the .travis itself is already tailored to > this reference implimentation of the ODP API, and this is an extension > of that >
Well.. there is database of users which accepted agreement. And we can not put that DB to main git. That is why it can not be in scripts. The right place is github web hook integration. The same way as SAP did for their projects. But it will take some time to understand web code setup and do required changes. After set up it has to look like this: https://github.com/muvarov/odp/pull/1 Maxim. > > > + > > script: > - echo $TRAVIS_COMMIT_RANGE > @@ -112,9 +115,11 @@ script: > if [ $? -ne 0 ]; then > git format-patch HEAD^; > perl ./scripts/checkpatch.pl <http://checkpatch.pl> > *.patch; > + ./odp-agreement/check_patchlist.sh *.patch; > fi; > else > perl ./scripts/checkpatch.pl <http://checkpatch.pl> > *.patch; > + ./odp-agreement/check_patchlist.sh *.patch; > fi > > - ./bootstrap > -- > 2.11.0.295.gd7dffce > > > > > -- > Mike Holmes > Program Manager - Linaro Networking Group > Linaro.org <http://www.linaro.org/>* **│ *Open source software for ARM SoCs > "Work should be fun and collaborative, the rest follows" > > __ > >
