<div dir="ltr"><div>On Wed, Dec 10, 2014 at 4:51 PM, Norbert Thiebaud <<a href="mailto:nthiebaud@gmail.com">nthiebaud@gmail.com</a>> wrote:<br>><br>> On Wed, Dec 10, 2014 at 9:45 AM, Maarten Hoes <<a href="mailto:hoes.maarten@gmail.com">hoes.maarten@gmail.com</a>> wrote:<br>> ><br>> > If still interested (and my tests completes as expected), I could submit the<br>> > contents of my .tb directory somewhere, but im not too sure if that's still<br>> > needed or relevant anymore with your current modifications/version of the<br>> > script:<br>><br>> I'm not using tb. I did the modification to be able to run the lcov<br>> script stand alone triggerred by jenkins.<br>><br></div>Aawwwhhh... Now *I* wanted to do that task (with the required hand holding, of course ;)<br><div><br><br>> <br>> But that is a special case... making this available under tb is still<br>> a valuable thing to make it easier for people running such tb-box to<br>> include code coverage in the mix.<br>><br></div><div>Now Im confused. Why would you want to run the same code coverage on a tb, when those results are already provided by a Jenkins build ? Doesnt it make more sense to 'just' have one box that (in this case, triggered by Jenkins), takes care of all your code coverage needs ?<br></div><div><br></div><div>Anyway, I guess I could, once I verified that it works as intended, put the contents of my .tb directory in a subdirectory of buildbot.git/lcov-report/tinderbox, accompanied with a small README explaining how you could go about integrating the two ?<br></div><div><br></div><div>And the one lacking feature of the current way is of course that the reports gets generated, but you still have to take care of the uploading them somewhere useful part. (assuming youve done that with Jenkins, and not by modifying the lcov-reports.sh script).<br></div><div><br><br>><br>> Let me emphasis: I made some modification/extenstion to fit better my<br>> use case.. that does not imply that the model you used was wrong or<br>> needed change<br>><br></div><div>I guess it's just that I dont see the 'tinderbox use cases' as clearly as you anymore, especially when now youve already got code coverage working in the Jenkins case.<br></div><div><br><br><br></div><div>- Maarten<br><br></div></div>