Нужна помощь с моим скриптом bash

Question

Нужна помощь с моим сценарием. Я хочу конвертировать файлы ISO в UTF-8. Проблема в том, что я не знаю, как написать IF:

if [ `file -b {}` = "$UTF8" ] \ 

правильно и как можно сказать программе sed - что она игнорирует # комментарии?

Вот мой сценарий:

#!/bin/bash

clear

echo -e '\E[37mThis script encodes recursively files from \E[31mISO-8859-1 \E[37mto         \E[31mUTF-8 \E[37musing iconv.'
echo "Files of the following coded character sets will be encode: "
echo -e '\E[32m'

a='*.java'
b='*.txt'
c='*.php'
d='*.html'
e='*.aj'
f='*.patch'
g='*.css'
h='*.js'
i='*.conf'
j='*.jsp'
k='*.sh'
l='*.py'
m='*.pl'
n='*.rb'

for x in "$a" "$b" "$c" "$d" "$e" "$f" "$g" "$h" "$i" "$j" "$k" "$l" "$m" "$n" 
do
   echo $x
done

echo
tput sgr0

#
# TODO: COMMENTS aren't ignored
# TOOD: IF-THEN aren't working right
#


for y in  "$a" "$b" "$c" "$d" "$e" "$f" "$g" "$h" "$i" "$j" "$k" "$l" "$m" "$n"  
  do
    echo -e "\E[37mencoding all <\E[32m$y\E[37m> files ..."
    find . -name "$y" -exec sh -c "( \
      UTF=".*UTF-8 Unicode.*" \
      FILE={} \
      if [ `file -b {}` = "$UTF8" ] \ 
        then  \   
          iconv -f latin1 -t UTF-8 {} -o {}.iconv ; \
          sed -n '
              { 
                s/^ *#/#/#.*//g;
                s/ä/0xE4;/g;
                s/Ä/0xC4;/g;
                s/ü/0xFC;/g;
                s/Ü/0xDC;/g;
                s/ö/0xF6;/g;
                s/Ö/0xD6;/g;
                s/ß/0xDF;/g;
                p; 
                }  {}.iconv > {}.iconv_sed \ '
       mv {}.iconv_sed {} && rm {}.iconv ; \
    else \
      echo "$FILE is a UTF8 file. " \ 
  fi \
)" \;
      echo -e '\E[33m*** done ***'
done

echo 
tput sgr0

exit 0

Спасибо

Answer 1

Кажется, в вашем скрипте есть несколько неправильных вещей (например, я не вижу нигде определенной переменной "UTF8"), но вы сильно усложнили ее с точки зрения отладки. Если бы это был я, я бы:

1. положить все находки sh -c "... дерьмо в отдельном скрипте, чтобы вы могли проверить его отдельно

2. if [ "`file -b $1`" = ...
   

3. вероятно, положить sed вещи в отдельной функции и проверить, что

4. не использовать sed -n а затем явно p; каждая строка, это глупо
5. правильно процитировать сценарий sed; Я полагаю, что вы пытаетесь сделать перенаправление внутри него

... пять предложений должно быть достаточно, чтобы начать. Предложение 0 "напишите более конкретный заголовок для вашего вопроса"