Hi!

I am working on packaging the development version of the eagle-usb
driver & tools for Debian and have found a little problem with the
"setvars" script concerning the automatic language selection.

Currently it goes:

[...]
if [ "$EU_LANG" = "auto" ] ; then
        EU_LANG=${LANG:0:2}
fi
# fr is usually the most up-to-date language
# => it is better to display some french sentences than nothing!
. $EU_LANG_DIR/fr
# now read the real language-file
if [ -e $EU_LANG_DIR/$EU_LANG ] ; then
        . $EU_LANG_DIR/$EU_LANG
else
        . $EU_LANG_DIR/en
fi
[...]

A problem occurs when the "LANG" environment variable is not set. In
this case, EU_LANG gets set to an empty variable, so that the test 
[ -e $EU_LANG_DIR/$EU_LANG ] ends up testing the existence of the
language files *directory*! A better test would be to test if
$EU_LANG_DIR/$EU_LANG is a regular file like this:

[ -f $EU_LANG_DIR/$EU_LANG ] 

Regards,
Jeremy

--  
http://www.jerryweb.org/         : JerryWeb.org
http://sailcut.sourceforge.net/  : Sailcut CAD
http://mpf70.sourceforge.net/    : MPman MP-F70 support for Linux

Reply via email to