travis: add code style checks

Message ID 20170206153802.3912-1-maxim.uvarov@linaro.org
State New
Headers show

Commit Message

Maxim Uvarov Feb. 6, 2017, 3:38 p.m.
run checkpatch.pl to check code style for applied patches.
Do not check spelling now. Will check it only on merge requests.

Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>

---
 .travis.yml | 5 +++++
 1 file changed, 5 insertions(+)

-- 
2.11.0.295.gd7dffce

Comments

Mike Holmes Feb. 8, 2017, 6:27 p.m. | #1
https://travis-ci.org/mike-holmes-linaro/odp/jobs/199704885

I had trouble running this Maxim, I have not dug into why


0.00s$ echo $TRAVIS_COMMIT_RANGE

1c409144d6e6...9329b4025f3d

The command "echo $TRAVIS_COMMIT_RANGE" exited with 0.

0.01s$ ODP_PACHES=`echo $TRAVIS_COMMIT_RANGE | sed 's/\.//'`

The command "ODP_PACHES=`echo $TRAVIS_COMMIT_RANGE | sed 's/\.//'`"
exited with 0.

0.00s$ git format-patch $ODP_PACHES

fatal: ambiguous argument '1c409144d6e6..9329b4025f3d': unknown
revision or path not in the working tree.

Use '--' to separate paths from revisions, like this:

'git <command> [<revision>...] -- [<file>...]'

The command "git format-patch $ODP_PACHES" exited with 128.

0.08s0K$ perl ./scripts/checkpatch.pl *.patch

No codespell typos will be found - file
'/usr/share/codespell/dictionary.txt': No such file or directory

./scripts/checkpatch.pl: *.patch: open failed - No such file or directory

The command "perl ./scripts/checkpatch.pl *.patch" exited with 2.


On 6 February 2017 at 10:38, Maxim Uvarov <maxim.uvarov@linaro.org> wrote:

> run checkpatch.pl to check code style for applied patches.

> Do not check spelling now. Will check it only on merge requests.

>

> Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>

> ---

>  .travis.yml | 5 +++++

>  1 file changed, 5 insertions(+)

>

> diff --git a/.travis.yml b/.travis.yml

> index 67af2fc2..0c643399 100644

> --- a/.travis.yml

> +++ b/.travis.yml

> @@ -91,6 +91,11 @@ before_install:

>          - popd

>

>  script:

> +        - echo $TRAVIS_COMMIT_RANGE

> +        - ODP_PACHES=`echo $TRAVIS_COMMIT_RANGE | sed 's/\.//'`

> +        - git format-patch $ODP_PACHES

> +        - perl ./scripts/checkpatch.pl *.patch

> +

>          - ./bootstrap

>          - ./configure

>          - make doxygen-doc

> --

> 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"

Patch

diff --git a/.travis.yml b/.travis.yml
index 67af2fc2..0c643399 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -91,6 +91,11 @@  before_install:
         - popd
 
 script:
+        - echo $TRAVIS_COMMIT_RANGE
+        - ODP_PACHES=`echo $TRAVIS_COMMIT_RANGE | sed 's/\.//'`
+        - git format-patch $ODP_PACHES
+        - perl ./scripts/checkpatch.pl *.patch
+
         - ./bootstrap
         - ./configure
         - make doxygen-doc