Re: [Openocd-development] checkpatch enabled

2011-10-29 Thread Antonio Borneo
On Sat, Oct 29, 2011 at 9:14 PM, Spencer Oliver wrote: > ... > I have also tweaked the build script to echo the args used: > http://openocd.zylin.com/jenkins/job/openocd/lastBuild/consoleFull Perfect! Thank you Antonio ___ Openocd-development mailing l

Re: [Openocd-development] scan-build and gerrit rant

2011-10-29 Thread Peter Stuge
Andreas Fritiofson wrote: > >clang: fix warning by adding assert that shows that a variable is used .. > > > > + assert(e == JIM_OK); > > + .. > I'm not very fond of the idea of merging patches with the sole > purpose of fixing scan-build false positives. +1 > if we start tweaking per

Re: [Openocd-development] openocd patch: d171eb8 target config files: Fix whitespace issues.

2011-10-29 Thread Peter Stuge
ger...@openocd.zylin.com wrote: > +++ b/tcl/target/atmega128.cfg > @@ -1,12 +1,12 @@ > # for avr > > - set _CHIPNAME avr > - set _ENDIAN little > + set _CHIPNAME avr > + set _ENDIAN little Why keep indent here? > # jtag speed > adapter_khz 4500 > > -reset_config srst_only > +res

[Openocd-development] openocd patch: 2533760 interface configs: Add missing URLs and names.

2011-10-29 Thread gerrit
This is an automated email from Gerrit. Uwe Hermann (u...@hermann-uwe.de) just uploaded a new patch set to Gerrit, which you can find at http://openocd.zylin.com/138 -- gerrit commit 2533760c5219527a8887c560babc75087f8b6074 Author: Uwe Hermann Date: Sun Oct 30 01:44:18 2011 +0200 interf

[Openocd-development] openocd patch: b2c9a43 interface configs: Fix whitespace and other issues.

2011-10-29 Thread gerrit
This is an automated email from Gerrit. Uwe Hermann (u...@hermann-uwe.de) just uploaded a new patch set to Gerrit, which you can find at http://openocd.zylin.com/139 -- gerrit commit b2c9a43fba25257c692ff78101613aa309b76923 Author: Uwe Hermann Date: Sun Oct 30 01:45:11 2011 +0200 interf

[Openocd-development] scan-build and gerrit rant (Was: Re: openocd patch: 620ba7e clang: fix warning by adding assert that shows that a variable is used)

2011-10-29 Thread Andreas Fritiofson
On Sat, Oct 29, 2011 at 10:56 AM, wrote: > This is an automated email from Gerrit. > > ?yvind Harboe (oyvindhar...@gmail.com) just uploaded a new patch set to > Gerrit, which you can find at http://openocd.zylin.com/134 > > -- gerrit > > commit 620ba7e6cd57c951fafa0f1ffab2db102fe2a60f > Author: Ø

[Openocd-development] openocd patch: 6ada46d config files: Drop incorrect comments.

2011-10-29 Thread gerrit
This is an automated email from Gerrit. Uwe Hermann (u...@hermann-uwe.de) just uploaded a new patch set to Gerrit, which you can find at http://openocd.zylin.com/136 -- gerrit commit 6ada46d8639bb44800e526111291172506122c0d Author: Uwe Hermann Date: Sat Oct 29 22:58:12 2011 +0200 config

[Openocd-development] openocd patch: ccbf006 gdb_server: clarify w/assert impossible condition on malloc()

2011-10-29 Thread gerrit
This is an automated email from Gerrit. ?yvind Harboe (oyvindhar...@gmail.com) just uploaded a new patch set to Gerrit, which you can find at http://openocd.zylin.com/135 -- gerrit commit ccbf006b0af960d6c8462774e76ddc7df933c2b8 Author: Øyvind Harboe Date: Sat Oct 29 17:37:42 2011 +0200

Re: [Openocd-development] checkpatch enabled

2011-10-29 Thread Spencer Oliver
On 29/10/2011 12:38, Antonio Borneo wrote: Hi Spen, I would like to view the parameters passed by jenkins to configure during the build. Are the scripts used by jenkins for OpenOCD available somewhere? --enable-maintainer-mode --enable-ft2232_libftdi --enable-usb_blaster_libftdi --enable-pre

Re: [Openocd-development] checkpatch enabled

2011-10-29 Thread Øyvind Harboe
On Sat, Oct 29, 2011 at 1:38 PM, Antonio Borneo wrote: > Hi Spen, > > I would like to view the parameters passed by jenkins to configure > during the build. > Are the scripts used by jenkins for OpenOCD available somewhere? Our intention is not for everybody to be *required* to have all these too

Re: [Openocd-development] checkpatch enabled

2011-10-29 Thread Antonio Borneo
Hi Spen, I would like to view the parameters passed by jenkins to configure during the build. Are the scripts used by jenkins for OpenOCD available somewhere? Best Regards Antonio On Fri, Oct 28, 2011 at 9:17 PM, Spencer Oliver wrote: > Hi, > > All patches submitted to gerrit will be verified u

[Openocd-development] openocd patch: 620ba7e clang: fix warning by adding assert that shows that a variable is used

2011-10-29 Thread gerrit
This is an automated email from Gerrit. ?yvind Harboe (oyvindhar...@gmail.com) just uploaded a new patch set to Gerrit, which you can find at http://openocd.zylin.com/134 -- gerrit commit 620ba7e6cd57c951fafa0f1ffab2db102fe2a60f Author: Øyvind Harboe Date: Sat Oct 29 10:55:02 2011 +0200