[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#834078: RFS: python-argh/0.26.1-1.1 [NMU, RC] -- simple argparse wrapper



* Sean Whitton <spwhitton@spwhitton.name>, 2016-08-11, 12:07:
 * Update 0001-fix-unit-tests.patch for newer sbuild (Closes: #832851).
   The LANG environment variable might not be set.

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).

[0] https://github.com/neithere/argh/issues/85

--
Jakub Wilk


Reply to: