On 09/30/2010 01:50 PM, Luca Bruno wrote: > Luca Bruno scrisse: > >> ep93xx and at91rm9200 are conditionally built only on arm and were not >> updated to reflect changes in command registration handler. >> This patch makes them properly compile again, fixing a build failure >> experienced on Debian armel. > > Before integrating this patch in our downstream packaging, I'd like to > hear some comments from openocd developers and (preferably) having a > copy (or improved version) of it in upstream git, to avoid divergence... > > Any comments on this? > I forgot to say, the bug report which triggered this patch is at > http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=597932 > (the armel part)
Since I was the one that did that work and overlooked these files, I have reviewed and committed the fixes that you provided. Sorry for the game delay. For posterity's sake, I consciously stopped fixing those files during the command handling upgrade, because I could not compile them and began to question whether anyone else could either. Thanks for submitting a patch to keep them alive. -- Zach Welch CodeSourcery zwe...@codesourcery.com (650) 331-3385 x743 _______________________________________________ Openocd-development mailing list Openocd-development@lists.berlios.de https://lists.berlios.de/mailman/listinfo/openocd-development
