diff mbox

[v2] travis.yml: add coverity support

Message ID 1479494062-17110-1-git-send-email-mike.holmes@linaro.org
State New
Headers show

Commit Message

Mike Holmes Nov. 18, 2016, 6:34 p.m. UTC
Enables static analysis when pushing to a branch coverity_scan on github
See https://scan.coverity.com/travis_ci

Signed-off-by: Mike Holmes <mike.holmes@linaro.org>

---
v2:
  Include all the executables in the scan (Maxim)

 .travis.yml | 20 +++++++++++++++++++-
 1 file changed, 19 insertions(+), 1 deletion(-)

-- 
2.7.4

Comments

Maxim Uvarov Nov. 18, 2016, 7:04 p.m. UTC | #1
On 11/18/16 21:34, Mike Holmes wrote:
> Enables static analysis when pushing to a branch coverity_scan on github

> See https://scan.coverity.com/travis_ci

>

> Signed-off-by: Mike Holmes <mike.holmes@linaro.org>

> ---

> v2:

>    Include all the executables in the scan (Maxim)

>

>   .travis.yml | 20 +++++++++++++++++++-

>   1 file changed, 19 insertions(+), 1 deletion(-)

>

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

> index 0d1add3..afcc353 100644

> --- a/.travis.yml

> +++ b/.travis.yml

> @@ -1,8 +1,17 @@

>   # Copyright (c) 2016, Linaro Limited

>   # All rights reserved.

>   # SPDX-License-Identifier:     BSD-3-Clause

> +#

> +# Please update xxxx for your coverity token and notification email if required

> +# pushing to github/master will run make check

> +# pushing to github/coverity_scan will also launch a static analysis

> +# See https://scan.coverity.com/travis_ci

>   

> -

> +env:

> +  global:

> +    # COVERITY_SCAN_TOKEN

> +    # ** specific to your project **

> +    - secure: "xxxx"

>   

>   language: c

>   compiler: clang

> @@ -33,3 +42,12 @@ script:

>           - make check

>           - make doxygen-doc

>           - make distcheck

> +

> +addons:

> +    coverity_scan:

> +        project:

> +            name: "$TRAVIS_REPO_SLUG"

> +        notification_email: xxxx

> +        build_command_prepend: "./bootstrap && ./configure --enable-test-cpp --enable-test-vald --enable-test-helper --enable-test-perf --enable-user-guides --enable-test-perf-proc --enable-test-example"


user-guides might be not needed here....

Maxim.

> +        build_command:   "make"

> +        branch_pattern: coverity_scan
Mike Holmes Nov. 18, 2016, 7:13 p.m. UTC | #2
I toyed with that and fell on cut and paste simple becasue I was not
able to make a variable that would span the scope.


On 18 November 2016 at 14:04, Maxim Uvarov <maxim.uvarov@linaro.org> wrote:
> On 11/18/16 21:34, Mike Holmes wrote:

>>

>> Enables static analysis when pushing to a branch coverity_scan on github

>> See https://scan.coverity.com/travis_ci

>>

>> Signed-off-by: Mike Holmes <mike.holmes@linaro.org>

>> ---

>> v2:

>>    Include all the executables in the scan (Maxim)

>>

>>   .travis.yml | 20 +++++++++++++++++++-

>>   1 file changed, 19 insertions(+), 1 deletion(-)

>>

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

>> index 0d1add3..afcc353 100644

>> --- a/.travis.yml

>> +++ b/.travis.yml

>> @@ -1,8 +1,17 @@

>>   # Copyright (c) 2016, Linaro Limited

>>   # All rights reserved.

>>   # SPDX-License-Identifier:     BSD-3-Clause

>> +#

>> +# Please update xxxx for your coverity token and notification email if

>> required

>> +# pushing to github/master will run make check

>> +# pushing to github/coverity_scan will also launch a static analysis

>> +# See https://scan.coverity.com/travis_ci

>>   -

>> +env:

>> +  global:

>> +    # COVERITY_SCAN_TOKEN

>> +    # ** specific to your project **

>> +    - secure: "xxxx"

>>     language: c

>>   compiler: clang

>> @@ -33,3 +42,12 @@ script:

>>           - make check

>>           - make doxygen-doc

>>           - make distcheck

>> +

>> +addons:

>> +    coverity_scan:

>> +        project:

>> +            name: "$TRAVIS_REPO_SLUG"

>> +        notification_email: xxxx

>> +        build_command_prepend: "./bootstrap && ./configure

>> --enable-test-cpp --enable-test-vald --enable-test-helper --enable-test-perf

>> --enable-user-guides --enable-test-perf-proc --enable-test-example"

>

>

> user-guides might be not needed here....

>

> Maxim.

>

>

>> +        build_command:   "make"

>> +        branch_pattern: coverity_scan

>

>




-- 
Mike Holmes
Program Manager - Linaro Networking Group
Linaro.org │ Open source software for ARM SoCs
"Work should be fun and collaborative, the rest follows"
Maxim Uvarov Nov. 18, 2016, 7:15 p.m. UTC | #3
ok, let's leave that. Maybe later we will have some code in user guide
also....

On 18 November 2016 at 22:13, Mike Holmes <mike.holmes@linaro.org> wrote:

> I toyed with that and fell on cut and paste simple becasue I was not

> able to make a variable that would span the scope.

>

>

> On 18 November 2016 at 14:04, Maxim Uvarov <maxim.uvarov@linaro.org>

> wrote:

> > On 11/18/16 21:34, Mike Holmes wrote:

> >>

> >> Enables static analysis when pushing to a branch coverity_scan on github

> >> See https://scan.coverity.com/travis_ci

> >>

> >> Signed-off-by: Mike Holmes <mike.holmes@linaro.org>

> >> ---

> >> v2:

> >>    Include all the executables in the scan (Maxim)

> >>

> >>   .travis.yml | 20 +++++++++++++++++++-

> >>   1 file changed, 19 insertions(+), 1 deletion(-)

> >>

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

> >> index 0d1add3..afcc353 100644

> >> --- a/.travis.yml

> >> +++ b/.travis.yml

> >> @@ -1,8 +1,17 @@

> >>   # Copyright (c) 2016, Linaro Limited

> >>   # All rights reserved.

> >>   # SPDX-License-Identifier:     BSD-3-Clause

> >> +#

> >> +# Please update xxxx for your coverity token and notification email if

> >> required

> >> +# pushing to github/master will run make check

> >> +# pushing to github/coverity_scan will also launch a static analysis

> >> +# See https://scan.coverity.com/travis_ci

> >>   -

> >> +env:

> >> +  global:

> >> +    # COVERITY_SCAN_TOKEN

> >> +    # ** specific to your project **

> >> +    - secure: "xxxx"

> >>     language: c

> >>   compiler: clang

> >> @@ -33,3 +42,12 @@ script:

> >>           - make check

> >>           - make doxygen-doc

> >>           - make distcheck

> >> +

> >> +addons:

> >> +    coverity_scan:

> >> +        project:

> >> +            name: "$TRAVIS_REPO_SLUG"

> >> +        notification_email: xxxx

> >> +        build_command_prepend: "./bootstrap && ./configure

> >> --enable-test-cpp --enable-test-vald --enable-test-helper

> --enable-test-perf

> >> --enable-user-guides --enable-test-perf-proc --enable-test-example"

> >

> >

> > user-guides might be not needed here....

> >

> > Maxim.

> >

> >

> >> +        build_command:   "make"

> >> +        branch_pattern: coverity_scan

> >

> >

>

>

>

> --

> Mike Holmes

> Program Manager - Linaro Networking Group

> Linaro.org │ Open source software for ARM SoCs

> "Work should be fun and collaborative, the rest follows"

>
Mike Holmes Nov. 28, 2016, 3:04 p.m. UTC | #4
ping, are you happy to merge this or do I respin it ?

I think you are the only other user so far unless Christophe gets chance to
try it

On 18 November 2016 at 14:15, Maxim Uvarov <maxim.uvarov@linaro.org> wrote:

> ok, let's leave that. Maybe later we will have some code in user guide

> also....

>

> On 18 November 2016 at 22:13, Mike Holmes <mike.holmes@linaro.org> wrote:

>

>> I toyed with that and fell on cut and paste simple becasue I was not

>> able to make a variable that would span the scope.

>>

>>

>> On 18 November 2016 at 14:04, Maxim Uvarov <maxim.uvarov@linaro.org>

>> wrote:

>> > On 11/18/16 21:34, Mike Holmes wrote:

>> >>

>> >> Enables static analysis when pushing to a branch coverity_scan on

>> github

>> >> See https://scan.coverity.com/travis_ci

>> >>

>> >> Signed-off-by: Mike Holmes <mike.holmes@linaro.org>

>> >> ---

>> >> v2:

>> >>    Include all the executables in the scan (Maxim)

>> >>

>> >>   .travis.yml | 20 +++++++++++++++++++-

>> >>   1 file changed, 19 insertions(+), 1 deletion(-)

>> >>

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

>> >> index 0d1add3..afcc353 100644

>> >> --- a/.travis.yml

>> >> +++ b/.travis.yml

>> >> @@ -1,8 +1,17 @@

>> >>   # Copyright (c) 2016, Linaro Limited

>> >>   # All rights reserved.

>> >>   # SPDX-License-Identifier:     BSD-3-Clause

>> >> +#

>> >> +# Please update xxxx for your coverity token and notification email if

>> >> required

>> >> +# pushing to github/master will run make check

>> >> +# pushing to github/coverity_scan will also launch a static analysis

>> >> +# See https://scan.coverity.com/travis_ci

>> >>   -

>> >> +env:

>> >> +  global:

>> >> +    # COVERITY_SCAN_TOKEN

>> >> +    # ** specific to your project **

>> >> +    - secure: "xxxx"

>> >>     language: c

>> >>   compiler: clang

>> >> @@ -33,3 +42,12 @@ script:

>> >>           - make check

>> >>           - make doxygen-doc

>> >>           - make distcheck

>> >> +

>> >> +addons:

>> >> +    coverity_scan:

>> >> +        project:

>> >> +            name: "$TRAVIS_REPO_SLUG"

>> >> +        notification_email: xxxx

>> >> +        build_command_prepend: "./bootstrap && ./configure

>> >> --enable-test-cpp --enable-test-vald --enable-test-helper

>> --enable-test-perf

>> >> --enable-user-guides --enable-test-perf-proc --enable-test-example"

>> >

>> >

>> > user-guides might be not needed here....

>> >

>> > Maxim.

>> >

>> >

>> >> +        build_command:   "make"

>> >> +        branch_pattern: coverity_scan

>> >

>> >

>>

>>

>>

>> --

>> Mike Holmes

>> Program Manager - Linaro Networking Group

>> Linaro.org │ Open source software for ARM SoCs

>> "Work should be fun and collaborative, the rest follows"

>>

>

>



-- 
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"
Maxim Uvarov Nov. 29, 2016, 2:32 p.m. UTC | #5
Merged,

I tested on my github and it works. But takes 2-3 business days for 
Coverity team to confirm match project to open source.

Maxim.

On 11/28/16 18:04, Mike Holmes wrote:
> ping, are you happy to merge this or do I respin it ?

>

> I think you are the only other user so far unless Christophe gets 

> chance to try it

>

> On 18 November 2016 at 14:15, Maxim Uvarov <maxim.uvarov@linaro.org 

> <mailto:maxim.uvarov@linaro.org>> wrote:

>

>     ok, let's leave that. Maybe later we will have some code in user

>     guide also....

>

>     On 18 November 2016 at 22:13, Mike Holmes <mike.holmes@linaro.org

>     <mailto:mike.holmes@linaro.org>> wrote:

>

>         I toyed with that and fell on cut and paste simple becasue I

>         was not

>         able to make a variable that would span the scope.

>

>

>         On 18 November 2016 at 14:04, Maxim Uvarov

>         <maxim.uvarov@linaro.org <mailto:maxim.uvarov@linaro.org>> wrote:

>         > On 11/18/16 21:34, Mike Holmes wrote:

>         >>

>         >> Enables static analysis when pushing to a branch

>         coverity_scan on github

>         >> See https://scan.coverity.com/travis_ci

>         <https://scan.coverity.com/travis_ci>

>         >>

>         >> Signed-off-by: Mike Holmes <mike.holmes@linaro.org

>         <mailto:mike.holmes@linaro.org>>

>         >> ---

>         >> v2:

>         >>    Include all the executables in the scan (Maxim)

>         >>

>         >>   .travis.yml | 20 +++++++++++++++++++-

>         >>   1 file changed, 19 insertions(+), 1 deletion(-)

>         >>

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

>         >> index 0d1add3..afcc353 100644

>         >> --- a/.travis.yml

>         >> +++ b/.travis.yml

>         >> @@ -1,8 +1,17 @@

>         >>   # Copyright (c) 2016, Linaro Limited

>         >>   # All rights reserved.

>         >>   # SPDX-License-Identifier:  BSD-3-Clause

>         >> +#

>         >> +# Please update xxxx for your coverity token and

>         notification email if

>         >> required

>         >> +# pushing to github/master will run make check

>         >> +# pushing to github/coverity_scan will also launch a

>         static analysis

>         >> +# See https://scan.coverity.com/travis_ci

>         <https://scan.coverity.com/travis_ci>

>         >>   -

>         >> +env:

>         >> +  global:

>         >> +    # COVERITY_SCAN_TOKEN

>         >> +    # ** specific to your project **

>         >> +    - secure: "xxxx"

>         >>     language: c

>         >>   compiler: clang

>         >> @@ -33,3 +42,12 @@ script:

>         >>           - make check

>         >>           - make doxygen-doc

>         >>           - make distcheck

>         >> +

>         >> +addons:

>         >> +    coverity_scan:

>         >> +        project:

>         >> +            name: "$TRAVIS_REPO_SLUG"

>         >> +        notification_email: xxxx

>         >> +        build_command_prepend: "./bootstrap && ./configure

>         >> --enable-test-cpp --enable-test-vald --enable-test-helper

>         --enable-test-perf

>         >> --enable-user-guides --enable-test-perf-proc

>         --enable-test-example"

>         >

>         >

>         > user-guides might be not needed here....

>         >

>         > Maxim.

>         >

>         >

>         >> +        build_command:   "make"

>         >> +        branch_pattern: coverity_scan

>         >

>         >

>

>

>

>         --

>         Mike Holmes

>         Program Manager - Linaro Networking Group

>         Linaro.org │ Open source software for ARM SoCs

>         "Work should be fun and collaborative, the rest follows"

>

>

>

>

>

> -- 

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

>
diff mbox

Patch

diff --git a/.travis.yml b/.travis.yml
index 0d1add3..afcc353 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -1,8 +1,17 @@ 
 # Copyright (c) 2016, Linaro Limited
 # All rights reserved.
 # SPDX-License-Identifier:     BSD-3-Clause
+#
+# Please update xxxx for your coverity token and notification email if required
+# pushing to github/master will run make check
+# pushing to github/coverity_scan will also launch a static analysis
+# See https://scan.coverity.com/travis_ci
 
-
+env:
+  global:
+    # COVERITY_SCAN_TOKEN
+    # ** specific to your project **
+    - secure: "xxxx"
 
 language: c
 compiler: clang
@@ -33,3 +42,12 @@  script:
         - make check
         - make doxygen-doc
         - make distcheck
+
+addons:
+    coverity_scan:
+        project:
+            name: "$TRAVIS_REPO_SLUG"
+        notification_email: xxxx
+        build_command_prepend: "./bootstrap && ./configure --enable-test-cpp --enable-test-vald --enable-test-helper --enable-test-perf --enable-user-guides --enable-test-perf-proc --enable-test-example"
+        build_command:   "make"
+        branch_pattern: coverity_scan