Always call glyrc with LC_ALL=C
authorSteve McIntyre <steve@einval.com>
Thu, 2 Aug 2018 13:56:48 +0000 (14:56 +0100)
committerSteve McIntyre <steve@einval.com>
Thu, 2 Aug 2018 13:56:48 +0000 (14:56 +0100)
Thanks to Andreas Vögele for the report and the suggested fix. Closes
Issue 83: https://abcde.einval.com/bugzilla/show_bug.cgi?id=50

abcde
changelog

diff --git a/abcde b/abcde
index 12678d2..6c90a27 100755 (executable)
--- a/abcde
+++ b/abcde
@@ -3252,7 +3252,7 @@ do_getalbumart()
                # use glyrc
                if [ ! -s "${ABCDETEMPDIR}/$ALBUMARTFILE" ]; then
                        vecho "trying to get cover with glyrc for $ARTISTFILE / $ALBUMFILE" >&2
-                       $GLYRC cover --artist "$ARTISTFILE" --album "$ALBUMFILE" --write "${ABCDETEMPDIR}/$ALBUMARTFILE" $GLYRCOPTS
+                       env LC_ALL=C $GLYRC cover --artist "$ARTISTFILE" --album "$ALBUMFILE" --write "${ABCDETEMPDIR}/$ALBUMARTFILE" $GLYRCOPTS
                        if [ $? -ne 0 ]; then
                                vecho "could not download cover with glyrc" >&2
                        else
index 24793a5..eed165d 100644 (file)
--- a/changelog
+++ b/changelog
@@ -1,6 +1,9 @@
 abcde 2.9.2
 
  * Fix up lookup code to deal with spaces etc. in directory names
+ * Always call glyrc with LC_ALL=C. Thanks to Andreas Vögele for the
+   report and the suggested fix. Closes Issue 83:
+   https://abcde.einval.com/bugzilla/show_bug.cgi?id=50
 
 abcde 2.9.1