issue-276 #331
issue-276 #331
Conversation
<a href="https://github.com/TouchInstinct/BuildScripts/issues/276">issue-276</a> Перед экспортом репозитория проверить файлы на невалидные символы в именах
scripts/export_src.sh
Outdated
| ERR_PATH_ARR=($(find . -name "*[<>:\\|?*]*" | xargs -I %s echo "- %s")) | ||
| unset IFS | ||
|
|
||
| if [ ${#ERR_PATH_ARR[@]} != 0 ] | ||
| then | ||
| echo "Export aborted! Invalid characters found in file or directories name(s):" | ||
| printf "%s\n" "${ERR_PATH_ARR[@]}" |
There was a problem hiding this comment.
массивы это bash расширение, в чистом sh надо на пустую строку или return code проверять
я предлагаю после вызова find проверить return code или строку на пустоту, потом вывести первую строки, а затем пройтись for по оставшимся результатам, тогда и xargs вроде не надо
scripts/export_src.sh
Outdated
| read -d $'\0' err_path | ||
| } | ||
|
|
||
| find . -name "*[<>:\\|?*]*" -print0 | |
There was a problem hiding this comment.
вот тут я сломался. что тут происходит, куда уходит результат?
There was a problem hiding this comment.
Результат читается построчно в функции read_err_path
There was a problem hiding this comment.
не ну это жесть какая-то. ищем в одной строчке, читаем в функции, а используем вообще дальше
давай попытаемся сделать как в нормальных языках программирования: переменные, зона видимости и вот это всё
scripts/export_src.sh
Outdated
| read_err_path() { | ||
| read -d $'\0' err_path | ||
| } |
There was a problem hiding this comment.
а нам точно нужна отдельная функция? можем этот код перенести к while?
There was a problem hiding this comment.
Вынесено в функцию, чтобы дублирования не было
scripts/export_src.sh
Outdated
| do | ||
| echo "- $err_path" | ||
| done | ||
| exit |
There was a problem hiding this comment.
exit без кода что вернёт?
There was a problem hiding this comment.
Можно exit 1 написать, раз завершение с ошибкой, но разница будет ли
issue-276 Перед экспортом репозитория проверить файлы на невалидные символы в именах