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.

Sign up for GitHub

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

Добавлен проброс переменных среды, установленных на уровне процесса #32

Merged
merged 1 commit into from
Jan 19, 2018
Merged

Добавлен проброс переменных среды, установленных на уровне процесса #32

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions src/Команда.os
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -305,9 +305,9 @@
ПерехватыватьПотоки = Истина;

Если КодировкаВывода = Неопределено Тогда
Процесс = СоздатьПроцесс(СтрокаЗапуска, РабочийКаталог, ПерехватыватьПотоки, ПерехватыватьПотоки);
Процесс = СоздатьПроцесс(СтрокаЗапуска, РабочийКаталог, ПерехватыватьПотоки, ПерехватыватьПотоки, , ПеременныеСреды());
Copy link
Collaborator

Choose a reason for hiding this comment

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

А может быть не утяжелять АПИ этой функции, а в свойства класса "Процесс" это перетащить? Как думаете?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ну то есть утяжелить АПИ самого 1коммандс? :)

Мне просто кажется, что проброс переменных среды процесса - это ожидаемое поведение.
Но глобально я не против.

@artbear ?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Я исключительно про функцию СоздатьПроцесс в движке. Сейчас в ней же нет последнего параметра про переменные окружения?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@EvilBeaver эээ, Андрюх) Есть :) Это ж движковая функция, я бы не стал вызывать чего не попадя. Мало того, она даже РАБОТАЕТ :D

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

https://github.com/EvilBeaver/OneScript/blob/7e84f4d5c51a350679a0e8b1b4c07d8d875a4234/src/ScriptEngine.HostedScript/Library/ProcessContext.cs#L243-L249

Copy link
Owner

Choose a reason for hiding this comment

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

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

по умолчанию - передача есть
но спец.метод может включить/отключить передачу, верно?

Вроде бы были сценарии, когда передачи переменных среды не нужна в новом процессе, но навскидку их не вспомню.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@artbear для пущей гибкости, да можно добавить сеттер. Но думаю, что по дефолту их можно пробрасывать. Или забить на сеттер и потом, когда вспомнится кейс, уже добавить :) а сейчас просто смержить PR

Copy link
Collaborator

Choose a reason for hiding this comment

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

Мало того, она даже РАБОТАЕТ

@nixel2007 дыа?! И даже blame говорит, что я должен про нее знать ))

Ну тогда вопросов не имею.

Иначе
Процесс = СоздатьПроцесс(СтрокаЗапуска, РабочийКаталог, ПерехватыватьПотоки, ПерехватыватьПотоки, КодировкаВывода);
Процесс = СоздатьПроцесс(СтрокаЗапуска, РабочийКаталог, ПерехватыватьПотоки, ПерехватыватьПотоки, КодировкаВывода, ПеременныеСреды());
КонецЕсли;
Процесс.Запустить();

Expand Down Expand Up @@ -470,4 +470,4 @@

Инициализация();

// Лог.УстановитьУровень(УровниЛога.Отладка);
// Лог.УстановитьУровень(УровниЛога.Отладка);