Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Добавлена возможность инкрементной конвертации в формат EDT #45

Merged
merged 3 commits into from
Nov 7, 2020

Conversation

yukon39
Copy link
Contributor

@yukon39 yukon39 commented Nov 3, 2020

Closes #44

@yukon39 yukon39 changed the title Добавлена возможность инкрементной конвертации в формат EDT WIP: Добавлена возможность инкрементной конвертации в формат EDT Nov 3, 2020

Параметры = Конфигуратор.ПолучитьПараметрыЗапуска();

Параметры.Добавить(СтрШаблон("/DumpConfigToFiles ""%1""", КаталогВыгрузки));
Copy link
Member

Choose a reason for hiding this comment

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

Что то мне не нравиться выгрузка в тот же каталог?
Может стоит все необходимые манипуляции для edt сделать в отдельном временном каталоге....

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Так это и есть манипуляции для EDT. Содержимое этого каталога все равно потом будет полностью удалено (см. стр. 184):
УдалитьФайлы(КаталогВыгрузки, "*");

@sfaqer
Copy link
Contributor

sfaqer commented Nov 5, 2020

@yukon39 может всё таки инкремент сделать опциональным, а не всегда когда получится, типа опцию добавить?

	КлассРеализации.Опция(
		"I increment", 	
		Ложь,
		"[*edtExport] использовать инкрементальную загрузку")
		.ТБулево()
		.ВОкружении("GITSYNC_EDT_INCREMENT");

Чтоб можно было принудительно полную выгрузку делать если что.

@yukon39
Copy link
Contributor Author

yukon39 commented Nov 5, 2020

может всё таки инкремент сделать опциональным, а не всегда когда получится, типа опцию добавить?

@sfaqer Можно же выключить плагин инкремента для этого. А потом снова включить.

За перещелкивание логики на инкремент отвечает dumplist.txt в корне проекта. А он сам по себе не появляется. Его генерит плагин инкрементной выгрузки.

Изначально, я код по доп.выгрузке размещал в increment.os. Тогда в edtExport вообще было без изменений. Но это ухудшило бы работу инкремента для обычных выгрузок (+1 вызов конфигуратора).

@sfaqer
Copy link
Contributor

sfaqer commented Nov 5, 2020

@yukon39 да, действительно, проморгал, всё ок!

@yukon39 yukon39 changed the title WIP: Добавлена возможность инкрементной конвертации в формат EDT Добавлена возможность инкрементной конвертации в формат EDT Nov 6, 2020
@yukon39
Copy link
Contributor Author

yukon39 commented Nov 6, 2020

Прогон по хранилищу с 2к коммитов проблем не выявил.

@khorevaa khorevaa merged commit 9dd1a4e into oscript-library:develop Nov 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants