On Sun, Sep 04, 2016 at 04:28:59PM +0100, Chris Lamb wrote: > [Re-adding 829...@bugs.debian.org to CC, and quoting in full] > > Antonio Terceiro wrote: > > > On Sat, Sep 03, 2016 at 05:32:38PM +0100, Chris Lamb wrote: > > > > I tried that, but with this patch the build failed. > > > > > > Please elaborate? > > > > Tests failed. I don't have the logs anymore, but they did. > > If you happen to apply it again, please retain them. > > > > > IMO all the other changes are not really warranted. > > > > > > Oh? Could you comment on why you believe this to be not necessary? > > > > > > > > > https://github.com/rtomayko/ronn/pull/98/files#diff-a87e9e80c3a7693175aa8c08bdb48452L70 > > > > > > (It seems self-evidently not reproducible, and I almost certainly added > > > it to the patch to fix a particular package...) > > > > ruby-ronn does not do locales: > > > > $ cat debian/coveralls.mkd | ronn | grep 2016 > > .TH "COVERALLS" "1" "September 2016" "" "" > > $ cat debian/coveralls.mkd | LANG=pt_BR.UFT-8 ronn | grep 2016 > > .TH "COVERALLS" "1" "September 2016" "" "" > > "UFT-8"? Again, it seems highly unlikely I patched this without good > reason.
yeah that was a typo, but again, ronn does not do locales (at all), so your changes to remove the usage of %B don't accomplish anything. maybe if I copy and paste the locale names you will believe me :-) $ cat debian/coveralls.mkd | LANG=C ronn | grep 2016 .TH "COVERALLS" "1" "September 2016" "" "" $ cat debian/coveralls.mkd | LANG=pt_BR.UTF-8 ronn | grep 2016 .TH "COVERALLS" "1" "September 2016" "" "" $ cat debian/coveralls.mkd | LANG=fr_FR.UTF-8 ronn | grep 2016 .TH "COVERALLS" "1" "September 2016" "" ""
signature.asc
Description: PGP signature
