tags 580725 + patch thanks Hi Florian,
Re: Nikolaus Schulz 2010-05-08 <20100508024739.5908.94278.reportbug@penelope.zusammrottung.local> > currently tree uses plain isprint() to determine if a character in the > filename is printable. > > This is because the cpp-macro LINUX isn't defined, so tree thinks > MB_CUR_MAX cannot be used and substitutes a default value of 1. With > this default value it uses isprint() instead of iswprint() do determine > if a character is printable. > > The following patch fixes this. > > Nikolaus Schulz > > diff --git a/debian/rules b/debian/rules > index b86cf85..320b43d 100755 > --- a/debian/rules > +++ b/debian/rules > @@ -9,7 +9,7 @@ MANDIR = $(PREFIX)/share/man/man1 > # Leave LDFLAGS empty so symbol table and relocation information won't be > # removed from the executable; dh_strip takes care of this when needed > LDFLAGS = > -CFLAGS = -g -Wall -fomit-frame-pointer -D_LARGEFILE64_SOURCE > -D_FILE_OFFSET_BITS=64 > +CFLAGS = -g -Wall -fomit-frame-pointer -DLINUX -D_LARGEFILE64_SOURCE > -D_FILE_OFFSET_BITS=64 > ifneq (,$(findstring noopt, $(DEB_BUILD_OPTIONS))) > CFLAGS += -O0 > else I can confirm that Nikolaus' patch works, and is the right way to fix this problem, given the way the code works with multibyte characters. Did you just overlook the patch, or are you not using tree anymore? If you like, I can NMU the package, or take over maintenance. Otherwise, an upload would be very much appreciated. Thanks, Christoph -- c...@df7cb.de | http://www.df7cb.de/
signature.asc
Description: Digital signature
