Возможность указания каталога temp #7
Conversation
|
В export такая штука не нужна? |
| СоздатьКаталог(БазовыйКаталог); | ||
| КонецЕсли; | ||
|
|
||
| ВременныеФайлы.БазовыйКаталог = БазовыйКаталог; |
There was a problem hiding this comment.
Не согласен с переустановкой свойства у глобальной переменной ВременныеФайлы.
Может повлиять на работу различных скриптов, которые могут вызывать данный код - например, тестраннер или 1бдд или еще что-нибудь
ИМХО для таких случаев нужно юзать собственный экземпляр класса ВременныеФайлы и в своем коде юзать только его.
There was a problem hiding this comment.
Т.е. для этого изменения нужно во всем коде исправить вызов ВременныеФайлы, заменив на собственный объект :)
There was a problem hiding this comment.
@EvilBeaver Помнишь, как-то говорили об этом в гиттере?
There was a problem hiding this comment.
Тогда другой вопрос: подключение библиотеке в скрипт не создает ее отдельный экземпляр?
Если при одновременном выполнении скриптов используются одинаковое пространство для выполнения, тогда вообще объявление любой глобальной переменной может привести к коллапсу в другом скрипте, что на мой взгляд мало вероятно. Вообще вопрос больше к движку скриптов и как он работает...
There was a problem hiding this comment.
Вообще логично.. Тогда и использование глобальной переменной будет возможно :) В общем добавлять или нет решать Вам, но это РАБОТАЕТ!!
There was a problem hiding this comment.
Но вопрос к @khorevaa у меня остался - что побудило менять основной темп?
@EvilBeaver Автор выше написал:
Но вообще указанная доработка решила мою проблему с переполнением диска C:\
Вопрос - почему бы весь пользовательский темп не перенаправить в другое место в этом случае? но в целом я не против сценария автора - меняем только для конкретного сценария.
Я с решаемой задачей (смена временного каталога) согласен, сценарий имеет место быть.
Я с реализацией не очень согласен.
@EvilBeaver Если у тебя нет возражений по задаче и ее реализации, ИМХО нужно принимать.
There was a problem hiding this comment.
Поясню дополнительно. у меня есть терминальный сервер (1шт ) на котором гоняется gitsync в 6 экземплярах.... соответсвенно хотелось бы чтобы они складывали свой temp в отдельные папочки... В случае падения всегда можно посмотреть остаточный temp.
There was a problem hiding this comment.
Давным-давно я имел такую же проблему, как @khorevaa . Тогда я добавил такой функционал в v83unpack (гитсинка, как и 1Скрипта, еще не существовало). Но после портирования обработки на 1Скрипт возможность установки временного каталога выпилили. Причем намерянно.
Я уже задавал вопрос про возможность указания временного каталога для гитсинка. И мне ответили, что достаточно в батнике, который вызывает gitsync вызвать
SET TEMP = "нужный каталог"
SET TMP = "нужный каталог"
В общем меня вроде бы как устраивает.
Изменения в работе приложения под новое API библиотеки
5 participants
No description provided.