Hi,

On Dienstag, 4. November 2014, Ximin Luo wrote:
> OK, I've uploaded a release candidate to mentors, same address as above.

I took a look, and there are two or three problems, though the fix itself is 
mostly fine I think :-)

1. bumping the standards version now is often perceived as unwanted noise by 
those reviewing the changes to decide whether to let it enter jessie. leave it 
now, but next time please only include non-cosmetic changes _if_ you add non 
ron-RC fixes at all.

2. your fix for #765156 looks good to me, I just wonder whether in the 
following it really should only be 20 and not 40 or 100... are you sure that 
safe enough now and in 5 years?

 ++        for i in xrange(0, 20):

(I think so, as hw gets faster but...  maybe 40 is still better as there could 
be even slower hw??)

3. there are lot of changes in debian/rules between 1.7-1 and 1.7-2 and 
there's no mentioning of those in debian/changelog at all. Is there a bug# for 
the problem they are fixing?

> Get well soon!

thanks, I'm on it..! :-)


cheers,
        Holger

Attachment: signature.asc
Description: This is a digitally signed message part.

Reply via email to