Исправление бага движка при запуске процесса с передачей переменных среды#743
Исправление бага движка при запуске процесса с передачей переменных среды#743
Conversation
| { | ||
| foreach (var kv in env) | ||
| { | ||
| sInfo.EnvironmentVariables.Remove(kv.Key.AsString()); |
There was a problem hiding this comment.
не будет ругаться, если там нет такого ключа?
There was a problem hiding this comment.
Эх, ты успел раньше вопрос написать, я как раз на всякий случай ответ вставлял :)
|
На всякий случай - Если в словаре StringDictionary нет элемента с указанным ключом, словарь StringDictionary не изменяется. Исключение не выдается. |
|
тесты где? |
|
и сборка отвалилась :( но не по моей вине :) |
|
А вот и тесты #744 |
|
@dmpas И почему тесты упали? |
|
в тестах косяк. |
|
Тесты упали, но другие, не имеющие отношения к моему исправлению. |
|
Мне одному предложенное решение кажется странным?.. |
|
А, вижу обсуждение в соседнем репо ( :( ) |
|
@artbear поправь заодно может и упавший тест с version? Бойскауты и все такое... |
|
@EvilBeaver а что там поправлять-то? |
|
Вижу в сборке на графике тестов резкое уменьшение их количества. Может поэтому сборка и зеленая? |
|
Это товарищи как бы падать то продолжает ;-), надо бы как-то захотфиксить |
|
Релиз-то не выпущен, пока только в ночной сборке. |
|
Выпущен релиз 1коммандс с обходом этой проблемы для движков с версией < 1.0.21 |
|
Ошибка осталась. Удаление дублирующихся ключей переменных среды не помогает. Такое ощущение что СоздатьПроцесс игнорирует параметр ПеременныеСредыПроцесса и получает их там внутри заново. Причём эта ошибка появляется и при 'git commit' из командной строки. Windows Server 2008 |
Как у вас получилось запустить оскрипт 1.9, у которого таргет на .net 4.6, на .net 4.0? |
|
И попробуйте oscript 2.0, там уже dotnet, а не .net framework. В целом ошибка была в самом дотнете, а не в оскрипте как таковом. |
5 participants
fix artbear/1commands#33