On 9/06/2011, at 12:21 AM, Erwan de FERRIERES wrote: > Le 08/06/2011 14:04, Scott Gray a écrit : >> Hi Erwan, >> >> Ignore this paragraph since I wrote it will you were making the changes that >> solved the problem: >> I agree with Jacopo, I really think the best solution would be to just have >> users that want cobertura run the separate ant task that downloads it >> instead of doing it automatically for them. We are allowed optional >> dependencies of non-compatible libraries but I think the automatic download >> when running tests crosses the line in a mandatory dependency. >> >> Also another thing to keep in mind for any feature is that a change that >> makes life easier for you won't necessarily make life easier for everyone. >> For example, I never look at the cobertura reports when running tests but >> now I have to wait while the library is downloaded (not sure what happens >> when I'm offline) and also put up with longer run times because of the >> additional overhead of the instrumentation. >> >> I'm not complaining though, just wanted to take the opportunity to again >> highlight that sometimes certain additional features can be as much of a >> hindrance to some as they can be a help to others. >> >> Regards >> Scott >> > Hi Scott, > > i'm not using the cobertura feature either. The only reason I make cobertura > downloadable is to remove the errors at the build time.
Well in that case this is a good example of making changes without any thought to the impact of those changes. But all the same, thanks for fixing the issue. Regards Scott
smime.p7s
Description: S/MIME cryptographic signature
