24251 2010-10-09 15:58:25 +0400 False positive "Unterminated quoted string" error 2010-10-12 20:10:13 +0400 1 1 4 Development Sisyphus checkbashisms unstable all Linux NEW P3 normal --- 1 kurakin nobody qa-sisyphus oldest_to_newest 113367 0 kurakin 2010-10-09 15:58:25 +0400 Скрипт i2myspell из одноименного пакета: $ checkbashisms /usr/bin/i2myspell possible bashism in /usr/bin/i2myspell line 61 ([^] should be [!]): s/\([^ ]*\).>.\([^,]*\)$/0 \2 \1/ possible bashism in /usr/bin/i2myspell line 62 ([^] should be [!]): s/\([^ ]*\).>.-\([^,]*\),\(\-\)$/\2 0 \1/ possible bashism in /usr/bin/i2myspell line 63 ([^] should be [!]): s/\([^ ]*\).>.-\([^,]*\),\(.*\)$/\2 \3 \1/' | error: /usr/bin/i2myspell: Unterminated quoted string found, EOF reached. Wanted: <'> Удалось выяснить, что ошибка с определением закрывающей кавычки происходит на строке 46 проверяемого скрипта, а всё остальное -- это уже последствия. Ошибочно распознаваемая конструкция выглядит так: sed 's#/# #g s# #/#' Запись конструкции в одну строку (sed 's#/# #g;s# #/#') проблемы не решает. Зато проблема решается заменой "#" на любой другой разделитель. Причем, достаточно заменить "#" только во втором шаблоне: sed 's#/# #g s| |/|' Такая конструкция проблем не вызывает и проверяется успешно. 113607 1 lav 2010-10-12 17:38:19 +0400 Если вы оформите предложение по исправлению checkbashisms в виде патча сюда и/или в mainstream, то я смогу его применить в новой сборке пакета. 113618 2 4607 kurakin 2010-10-12 20:06:41 +0400 Created attachment 4607 checkbashisms fails on this script 113619 3 4608 kurakin 2010-10-12 20:08:32 +0400 Created attachment 4608 checkbashisms processes this script successfully 113620 4 kurakin 2010-10-12 20:10:13 +0400 Был бы патч -- не было бы вопросов. Пока могу предоставить только testcase. 4607 2010-10-12 20:06:41 +0400 2010-10-12 20:06:41 +0400 checkbashisms fails on this script checkbashisms_fail application/octet-stream 49 kurakin IyEvYmluL3NoCmVjaG8gc29tZXRoaW5nIHwgc2VkICcKcyMvIyNnCnMjICMvIycKCg== 4608 2010-10-12 20:08:32 +0400 2010-10-12 20:08:32 +0400 checkbashisms processes this script successfully checkbashisms_success application/octet-stream 49 kurakin IyEvYmluL3NoCmVjaG8gc29tZXRoaW5nIHwgc2VkICcKcyMvIyNnCnN8IHwvfCcKCg==