Исправить сумасшедшее число вызовов определения рабочего каталога

[artbear]

например, http://build.oscript.io/job/oscript-library/job/gitsync/view/change-requests/job/PR-73/6/console

ИНФОРМАЦИЯ - Получаем исходники для версии 2, 10/10/2017 9:30:00 AM 
ИНФОРМАЦИЯ - Раскладываем модули по папкам согласно иерархии метаданных 
ОТЛАДКА - РабочийКаталог: C:\Users\JenkinsSlaveOS\AppData\Local\Temp\rero1t4g.dvh.tmp\src 
ОТЛАДКА - Код возврата равен 0 
ОТЛАДКА - РабочийКаталог: C:\Users\JenkinsSlaveOS\AppData\Local\Temp\rero1t4g.dvh.tmp\src 
ОТЛАДКА - Код возврата равен 0 
ОТЛАДКА - РабочийКаталог: C:\Users\JenkinsSlaveOS\AppData\Local\Temp\rero1t4g.dvh.tmp\src 
ОТЛАДКА - Код возврата равен 0 
ОТЛАДКА - РабочийКаталог: C:\Users\JenkinsSlaveOS\AppData\Local\Temp\rero1t4g.dvh.tmp\src 
ОТЛАДКА - Код возврата равен 0 
ОТЛАДКА - РабочийКаталог: C:\Users\JenkinsSlaveOS\AppData\Local\Temp\rero1t4g.dvh.tmp\src 
ОТЛАДКА - Код возврата равен 0 
ОТЛАДКА - РабочийКаталог: C:\Users\JenkinsSlaveOS\AppData\Local\Temp\rero1t4g.dvh.tmp\src 
ОТЛАДКА - Код возврата равен 0 
ОТЛАДКА - РабочийКаталог: C:\Users\JenkinsSlaveOS\AppData\Local\Temp\rero1t4g.dvh.tmp\src 
ОТЛАДКА - Код возврата равен 0

/cc @nixel2007 @khorevaa

это же пипец какой-то
я ИМХО месяца 3 назад об этой проблеме гитраннера или гитсинка вам с @khorevaa писал, а воз и ныне там.
нафига это сумасшедшее ненужное дергание системы, процессов, дисков??

[EvilBeaver]

А че это ваще? В каком месте кода там это?

[artbear]

Я уже писал, зачем нужны эти строки кода в супер-популярном методе ПолучитьГитРепозиторий?
Строка 443

Функция ПолучитьГитРепозиторий(Знач КаталогРабочейКопии)

	ГитРепозиторий = Новый ГитРепозиторий;
	ГитРепозиторий.УстановитьРабочийКаталог(КаталогРабочейКопии);
	ГитРепозиторий.УстановитьНастройку("core.quotepath","false",РежимУстановкиНастроекGit.Локально); // ВОТ ЗДЕСЬ 
	ГитРепозиторий.УстановитьНастройку("merge.ours.driver","true",РежимУстановкиНастроекGit.Локально); // ВОТ ЗДЕСЬ
	Возврат ГитРепозиторий;

КонецФункции // ПолучитьГитРепозиторий()

нафига лишний вызов аж 2 команд гита ??

[artbear]

Предлагаю добавить кеширование рабочего каталога

Перем РабочийКаталогСохр;
Перем ГитРепозиторийСохр;

Функция ПолучитьГитРепозиторий(Знач КаталогРабочейКопии)

	ФайлКаталога = Новый Файл(КаталогРабочейКопии);
	Если ФайлКаталога.ПолныйПуть = РабочийКаталогСохр Тогда
		ГитРепозиторий = ГитРепозиторийСохр;
	Иначе
		ГитРепозиторий = Новый ГитРепозиторий;
		ГитРепозиторий.УстановитьРабочийКаталог(КаталогРабочейКопии);
		ГитРепозиторий.УстановитьНастройку("core.quotepath","false",РежимУстановкиНастроекGit.Локально);
		ГитРепозиторий.УстановитьНастройку("merge.ours.driver","true",РежимУстановкиНастроекGit.Локально);

		РабочийКаталогСохр = ФайлКаталога.ПолныйПуть;
		ГитРепозиторийСохр = ГитРепозиторий;

	КонецЕсли;

	Возврат ГитРепозиторий;

КонецФункции // ПолучитьГитРепозиторий()

@nixel2007 Что скажешь?

[EvilBeaver]

Рацпредложение:

• Смысл этого логирующего сообщения - дать понять, что каталог есть, и он равен Х.
• Стало быть, при каждой команде нам это знать не надо. Если каталога нет - будет исключение. А если он есть, то нам не надо про это знать постоянно.
• Посему, логирование "Рабочий каталог равен Х" вынести в метод УстановитьРабочийКаталог. Тогда задача из первого пункта будет выполнена. В логах мы увидим, чему РК был равен.

[artbear]

Выполнено.

[khorevaa]

Про вафлил задачу. Работа затянула.. Но всегда есть кто поможет и улучшит -:)