Skip to content

Conversation

@witales
Copy link

@witales witales commented Jan 28, 2019

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

@witales witales changed the base branch from master to develop January 28, 2019 12:45

Процедура ЗапуститьРаспаковкуРасширения(Знач Файл, Знач ПапкаИсходников)

Лог.Отладка("Обеспечиваем пустой каталог: " + ПапкаИсходников.ПолноеИмя);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

пустой каталог еще требовался для ЗапуститьРаспаковкуОбработки. В PR запуск только для расширения.

Copy link
Member

@nixel2007 nixel2007 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

.

@witales
Copy link
Author

witales commented Jan 28, 2019

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

Параметры.Добавить(КоманднаяСтрокаV8Reader);

КонецЕсли;
Конфигуратор.ВыполнитьКоманду(Параметры);
Copy link

@zeratulayuris zeratulayuris Jan 28, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Вот здесь пустой каталог не обеспечивается независимо от того, макет разбираем или не макет. Для не-макетов нужно обеспечивать пустой каталог.

@witales
Copy link
Author

witales commented Jan 28, 2019

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

@zeratulayuris
Copy link

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

@witales
Copy link
Author

witales commented Jan 29, 2019

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

@nixel2007
Copy link
Member

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

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

@witales
Copy link
Author

witales commented Jan 29, 2019

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

@zeratulayuris
Copy link

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

@zeratulayuris zeratulayuris added this to the 2.2.2 milestone Apr 11, 2019
@zeratulayuris
Copy link

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

@witales
Copy link
Author

witales commented Apr 12, 2019

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

@zeratulayuris
Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants