В информационный лог добавлен вывод используемого для запуска команд исполняемого файла платформы #131
Conversation
…исполняемого файла платформы
…, то процедура ИспользоватьВерсиюПлатформы не вызывается
src/v8runner.os
Outdated
|
|
||
|
|
||
| ПутьКПлатформе1С(ПолучитьПутьКВерсииПлатформы("8.3", РазрядностьПлатформы.x64x86)); | ||
| Лог.Информация("Ищу платформу " + ВерсияПлатформы); |
There was a problem hiding this comment.
Только не надо "Информация". Это же библиотека, она используется разными приложениями. Автор приложения может не хотеть выводить эту строку, но не сможет повлиять. Пусть будет Отладка
There was a problem hiding this comment.
В отладке масса сообщений от плагинов :( Среди них теряется. Может быть какой-нибудь параметр? Или это еще сложнее?
There was a problem hiding this comment.
Можно конфигурировать лог так, чтобы выводить только конкретное имя лога. Подробнее в ридми logos. Но в целом я тоже рекомендую не добавлять в либы какой-либо "информационный" вывод.
There was a problem hiding this comment.
Я посмотрю и отпишусь, спасибо за наводку.
There was a problem hiding this comment.
Да, отладочные логи можно фильтровать по имени лога, а вот информационные от библиотек засоряют вывод использующего приложения.
There was a problem hiding this comment.
Вы у себя можете сделать отдельный logos-аппендер для лога v8runner и в нем это перехватить, например.
There was a problem hiding this comment.
Да, но я же могу отбор делать только по уровню лога, правильно? Или мне прямо надо написать аналог ВыводЛогаВКонсоль и в нем уже фильтровать сообщения?
Пока остановился на простом set LOGOS_CONFIG=logger.oscript.lib.v8runner=DEBUG
И что собственно делать с PR? :)
There was a problem hiding this comment.
В logos appender расковырял, но это сложнее, чем две строчки в v8runner поменять :)
There was a problem hiding this comment.
И что собственно делать с PR? :)
Исправить на Отладка - и вливаем
src/v8runner.os
Outdated
| Лог.Информация("Ищу платформу " + ВерсияПлатформы); | ||
| ПутьКПлатформе1С(ПолучитьПутьКВерсииПлатформы(ВерсияПлатформы, РазрядностьПлатформы.x64x86)); | ||
| Если Не ПустаяСтрока(мПутьКПлатформе1С) Тогда | ||
| Лог.Информация("Использую платформу " + мПутьКПлатформе1С); |
There was a problem hiding this comment.
Аналогично тут. А "Ошибка" ниже - пусть остается
This reverts commit 5264f8f.
В информационном логе при запуске команд отсутствует вывод используемой версии платформы. Предлагаю исправить эту неинформативность :)