Имя пользователя:
Пароль:  
Помощь | Регистрация | Забыли пароль?  

Название темы: [repack] Imagine
Показать сообщение отдельно

Аватара для gora

Ушёл из жизни


Сообщения: 6021
Благодарности: 3491

Профиль | Отправить PM | Цитировать


_syd_, посмотрел Ваш аддон:
1 Пути в этих строках имеют пробел и должны быть заключены в кавычки
Код: Выделить весь код
AutoInstall="%%P:forcenowait:%TD%%P%\\%Name% /assocext"
...
AutoInstall6="%%P:shc0:forcenowait:%TD%%P%\\%Name% /assocext"
AutoInstall7="%%P:shc0:forcenowait:%TD%%P%\\%Name% /regcontextmenu"
2 Имя файла указано без расширения, это допустимо, но в некоторых случаях может неправильно сработать. Лучше этого избегать.
3 "Багофича". (Так я называю прием с переменными типа %TD%%P%)
- работает не всегда, причины не установлены
- может работать в одной строке конфига, но не работать в другой строке этого же конфига. Обязательно требуется проверять работу ВСЕХ строк.
- может работать с одним модулем, но не работать с другим. Отдавать такой конфиг на самостоятельную сборку пользователям, с неведомо каким модулем не рекомендуется.
4 Файл assoc.cmd.
- строка
Код: Выделить весь код
If /I %PROCESSOR_ARCHITECTURE% == amd64 (Set InstPath=%ProgramW6432%\Imagine) Else (Set InstPath=%ProgramFiles%\Imagine)
-- определять разрядность системы по одной переменной %PROCESSOR_ARCHITECTURE% некорректно. Есть более корректные способы, а проще использовать переменную модуля.
-- Вы, видимо не тестируете свои аддоны на х64 системах, иначе бы увидели, что это условие у Вас ни когда не сработает, т.к., пробелы до и после "==" не ставятся.
-- строка, скорее всего, "вылетит" на х64 из-за скобок в составе одной из переменных в этой строке, т.к., она окажется в блоке. Не проверялось...
- назначение этой строки осталось для меня загадкой
Код: Выделить весь код
For %%i In (ani cur ico) Do (Echo %1 ^| Find /I "%%i") && ^
Это сообщение посчитали полезным следующие участники:

Отправлено: 13:30, 11-09-2012 | #17

Название темы: [repack] Imagine