Thanks for the patch. I need to check whether readline is able to detect when it cannot be used, maybe it can check whether /dev/{stdin,stdout} are terminals, or pipes, ...
Brice Miroslaw Zalewski wrote: > Package: lltag > Version: 0.14.2-1 > Severity: important > > lltag may use Term::ReadLine to improve it's interactive mode. But it > makes lltag completely unusable when not run in terminal. This happens > especially when you use lltag in some script, which is fired by other > program - e.g. file manager. The simpliest way to check it is opening > some "run" program (I used Fluxbox's fbrun) and typing in: > > lltag -S /path/to/any.mp3 >/tmp/somefile 2>&1 > > Content of /tmp/somefile will be something like > "Cannot open /dev/tty for read at /usr/share/perl5/Lltag/Misc.pm line > 108" instead of listed file's ID3 tags (try to do the same in term). > > Removing libterm-readline-gnu-perl from system fixes this. > > I think that problem is that lltag tries to init readline library even > when it's not needed - i.e. when -S or --yes options are given. I attach > patch that makes lltag usable in batch mode even when Term::ReadLine is > installed. I'm not sure if it doesn't break anything, but it seems safe. > > -- System Information: > Debian Release: squeeze/sid > APT prefers testing > APT policy: (990, 'testing'), (500, 'unstable') > Architecture: i386 (i686) > > Kernel: Linux 2.6.30-2-486 > Locale: LANG=pl_PL.UTF-8, LC_CTYPE=pl_PL.UTF-8 (charmap=UTF-8) > Shell: /bin/sh linked to /bin/dash > > Versions of packages lltag depends on: > ii perl 5.10.1-5 Larry Wall's Practical > Extraction > > Versions of packages lltag recommends: > ii flac 1.2.1-2+b1 Free Lossless Audio Codec - > comman > ii libmp3-tag-perl 1.11-1 Module for reading tags of MP3 > aud > ii libterm-readline-gnu-perl 1.19-2 Perl extension for the GNU > Readlin > ii libwww-perl 5.833-1 Perl HTTP/WWW client/server > librar > ii mp3info 0.8.5a-1 An MP3 technical info viewer and > I > ii vorbis-tools 1.2.0-6 several Ogg Vorbis tools > > lltag suggests no packages. > > -- no debconf information > -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org