On 09-08-17 08:12, Илья Шипицин wrote:
> 2017-08-09 10:47 GMT+05:00 Илья Шипицин <chipits...@gmail.com
> <mailto:chipits...@gmail.com>>:
>     2017-08-09 10:41 GMT+05:00 Илья Шипицин <chipits...@gmail.com
>     <mailto:chipits...@gmail.com>>:
>         2017-08-08 20:55 GMT+05:00 Steffan Karger
>         <steffan.kar...@fox-it.com <mailto:steffan.kar...@fox-it.com>>:
>             Enable coverity analysis for the release/2.4 branch.
>             We can only do a limited number of coverity scans per week
>             with our FOSS
>             account, but since we only occasionally push commits, that
>             should work out
>             fine.  But this limit is the reason we don't use the
>             standard travis addon,
>             because that would cause the coverity script to run on all
>             of our matrix
>             builds.  That would cause us to reach our limit faster, and
>             waste travis'
>             resources.
>             Since our FOSS coverity account doesn't handle multiple
>             branches very well,
>             we have to pick one branch to run coverity on.  I think it's
>             best to use
>             the most recent stable branch for that (i.e. for now,
>             release/2.4).
>             Though for ease of maintenance, it's probably best to apply
>             the patch to
>             both master and release/2.4.
>         I would refactor it like that
>         https://gist.github.com/chipitsine/8dcae4ff1d59eb43df39f6015c6106fd
>         <https://gist.github.com/chipitsine/8dcae4ff1d59eb43df39f6015c6106fd>
>         however, your is ok as well
>     maybe, "script" would be better here
> https://gist.github.com/chipitsine/8dcae4ff1d59eb43df39f6015c6106fd#file-gistfile1-txt-L74
> <https://gist.github.com/chipitsine/8dcae4ff1d59eb43df39f6015c6106fd#file-gistfile1-txt-L74>
>     than "before_script"
> oops, travis does not support braches within matrix
> https://gist.github.com/chipitsine/8dcae4ff1d59eb43df39f6015c6106fd#file-gistfile1-txt-L70-L72
> sorry for that

I like your idea, and toyed with it a bit.  But in the end, it needs too
much fiddling to get right.  So I prefer my own suggested approach.


Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
Openvpn-devel mailing list

Reply via email to