control: reopen -1 control: retitle -1 RFS: python-argh/0.26.1-1.2 [NMU] -- simple argparse wrapper control: severity -1 normal
Hello, On Fri, Aug 12, 2016 at 02:22:09PM +0200, Jakub Wilk wrote: > Both the original and the new patch look wrong to me: > * C is not the only locale in which the test fails. > * You can't determine the current locale just by looking at the LANG > variable. (It can by overridden by various LC_* variables.) > > Ideally the test should be fixed upstream[0], but until then it's probably > best to drop this half-baked patch, and instead force UTF-8 locale in > debian/rules (export LC_ALL=C.UTF-8). I agree -- thanks for your feedback. I'm reopening this RFS for another NMU (this time, I would like to suggest an upload to DELAYED/10). New changelog: * Non-maintainer upload. * Drop parts of 0001-fix-unit-tests.patch touching test_interaction.py. * Fix the Unicode encoding test by setting LC_ALL in d/rules. This fix is more general as the LANG var can be overridden by LC_* vars (see locale(7)). Thanks to Jakub Wilk for pointing this out and suggesting the improved fix. Download from mentors: dget -x http://mentors.debian.net/debian/pool/main/p/python-argh/python-argh_0.26.1-1.2.dsc -- Sean Whitton
signature.asc
Description: PGP signature