Добавлен проброс переменных среды, установленных на уровне процесса #32
Conversation
Требует принятия artbear/1commands#32 Как побочный эффект теперь нет привязки к приложениям, расположенным в alias/bin
| @@ -305,9 +305,9 @@ | |||
| ПерехватыватьПотоки = Истина; | |||
|
|
|||
| Если КодировкаВывода = Неопределено Тогда | |||
| Процесс = СоздатьПроцесс(СтрокаЗапуска, РабочийКаталог, ПерехватыватьПотоки, ПерехватыватьПотоки); | |||
| Процесс = СоздатьПроцесс(СтрокаЗапуска, РабочийКаталог, ПерехватыватьПотоки, ПерехватыватьПотоки, , ПеременныеСреды()); | |||
There was a problem hiding this comment.
А может быть не утяжелять АПИ этой функции, а в свойства класса "Процесс" это перетащить? Как думаете?
There was a problem hiding this comment.
ну то есть утяжелить АПИ самого 1коммандс? :)
Мне просто кажется, что проброс переменных среды процесса - это ожидаемое поведение.
Но глобально я не против.
@artbear ?
There was a problem hiding this comment.
Я исключительно про функцию СоздатьПроцесс в движке. Сейчас в ней же нет последнего параметра про переменные окружения?
There was a problem hiding this comment.
@EvilBeaver эээ, Андрюх) Есть :) Это ж движковая функция, я бы не стал вызывать чего не попадя. Мало того, она даже РАБОТАЕТ :D
There was a problem hiding this comment.
There was a problem hiding this comment.
Правильно ли я понимаю, что предлагается добавить возможность настройки передачи переменных среды в создаваемый процесс?
по умолчанию - передача есть
но спец.метод может включить/отключить передачу, верно?
Вроде бы были сценарии, когда передачи переменных среды не нужна в новом процессе, но навскидку их не вспомню.
There was a problem hiding this comment.
@artbear для пущей гибкости, да можно добавить сеттер. Но думаю, что по дефолту их можно пробрасывать. Или забить на сеттер и потом, когда вспомнится кейс, уже добавить :) а сейчас просто смержить PR
There was a problem hiding this comment.
Мало того, она даже РАБОТАЕТ
@nixel2007 дыа?! И даже blame говорит, что я должен про нее знать ))
Ну тогда вопросов не имею.
|
Релиз 1.3.2 выпущен |
Эта штука позволяет пробрасывать переменные среды, установленные на всех уровнях (в том числе на уровне процесса и/или на уровне пользователя/системы, установленные в рамках этого же процесса), в новый процесс cmd/sh.
Это позволяет делать вот такие любопытные вещи:
т.к. позволяет по-нормальному переопределить, например, PATH.