search

xDrivenDevelopment / precommit1c

Набор утилит для автоматической разборки/сборки внешних обработок и отчетов, при помещении (commit) в git
Other
223 stars 60 forks source link

Убрана предварительная очистка исходников перед запуском конфигуратора #128

Closed witales closed 4 years ago

witales commented 5 years ago

Столкнулся с тем, что при неудачном запуске конфигуратора (например, некорректный пароль), ранее разложенные файлы удаляются, потому что чистка каталога исходников происходит до запуска конфигуратора. Предлагаю очищать исходники только после выполнения команды конфигуратора. Изменения касаются только нового режима --use-designer.

witales commented 5 years ago

В ЗапуститьРаспаковкуОбработки тоже есть правки: очистка каталога вынесена после работы конфигуратора. Либо я не понял замечания:)

witales commented 5 years ago

Спасибо, исправил. Также добавил раскладку через промежуточный каталог и для расширений, и для обработок в режиме v8Reader (с ними тоже нарывался на очистку исходников в случае, когда отъезжает лицензия 1С).

zeratulayuris commented 5 years ago

Я бы ещё почистил временные файлы после копирования.

witales commented 5 years ago

Чисткой временных занимается сам движок oscript.

nixel2007 commented 5 years ago

Чисткой временных занимается сам движок oscript.

вы о чем? движок не занимается очисткой временных файлов. Если они были созданы через библиотеку tempfiles, то они удаляются только при вызове метода Менеджера Временных Файлов. Файлы, созданные вручную через ПолучитьИмяВременногоФайла не удаляются вообще.

witales commented 5 years ago

Следил за поведением скрипта, видел, что они по окончании удаляются, но не увидел сразу в конце процедуры ЗапускВКоманднойСтроке вызов ВременныеФайлы.Удалить(). Поэтому и списал это на движок. Дал более корретное сообщение при отладке, а в части логики здесь все и так ок.

zeratulayuris commented 5 years ago

@witales накроете тестами новое поведение? И заодно заберёте правильный бамп от @artbear.

zeratulayuris commented 5 years ago

@witales вы не против, если я заберу ваши изменения и открою новый PR с тестами и исправленными конфликтами?

witales commented 5 years ago

@zeratulayuris не против, конечно забирайте

zeratulayuris commented 4 years ago

Этот PR закрываю, открыл новый.