Hi,
On Sat, Oct 26, 2013 at 11:52:08PM +1100, Ben Finney wrote:
> package git-buildpackage
> tags 727763 + patch
> thanks
> 
> On 25-Oct-2013, Daniel Dehennin wrote:
> > Traceback (most recent call last):
> >   File "setup.py", line 64, in <module>
> >     'console_scripts': [ 'gbp = gbp.scripts.supercommand:supercommand' ],
> >   […]
> >   File "/usr/lib/python2.7/unittest/main.py", line 95, in __init__
> >     self.runTests()
> >   […]
> >   File "/usr/lib/python2.7/dist-packages/coverage/control.py", line 658, in 
> > html_report
> >     reporter = HtmlReporter(self, self.config)
> 
> What is the point of generating an HTML report for the coverage, when
> building ‘git-buildpackage’?

Well, it's 2013 and having not only machine parseable reports is nice to
help development. I'd be o.k. to disable this for "release builds" like 

#if RELEASE_BUILD
...
#endif 

but I don't think setup.cfg can cope with that unfortunately. Disabling
it in general is not an option.
Cheers,
 -- Guido

> 
> To avoid this coupling to the HTML reporting feature, the Debian package
> for ‘git-buildpackage’ can disable the HTML report feature for Coverage:
> 
> 
> --- old/setup.cfg     2013-10-26 11:46:09.000000000 +0000
> +++ new/setup.cfg     2013-10-26 12:41:10.333638255 +0000
> @@ -4,8 +4,6 @@
>  with-xunit=1
>  cover-package=gbp
>  cover-erase=1
> -cover-html=1
> -cover-html-dir=build/cover/
>  cover-min-percentage=68
>  
>  [epydoc]
> 
> 
> -- 
>  \       己所不欲、勿施于人。 (What is undesirable to you, do not do to others.) |
>   `\                                —孔夫子 Confucius (551 BCE – 479 BCE) |
> _o__)                                                                  |
> Ben Finney <b...@benfinney.id.au>


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to