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

Исправление бага движка при запуске процесса с передачей переменных среды #743

Merged
merged 1 commit into from Apr 20, 2018

Conversation

Projects
None yet
5 participants
@artbear
Copy link
Collaborator

artbear commented Apr 20, 2018

@@ -244,6 +244,7 @@ public static ProcessContext Create(string cmdLine, string currentDir = null, bo
{
foreach (var kv in env)
{
sInfo.EnvironmentVariables.Remove(kv.Key.AsString());

This comment has been minimized.

@dmpas

dmpas Apr 20, 2018

Collaborator

не будет ругаться, если там нет такого ключа?

This comment has been minimized.

@artbear

artbear Apr 20, 2018

Author Collaborator

Эх, ты успел раньше вопрос написать, я как раз на всякий случай ответ вставлял :)

@artbear

This comment has been minimized.

Copy link
Collaborator Author

artbear commented Apr 20, 2018

На всякий случай -
StringDictionary.Remove - метод

https://msdn.microsoft.com/ru-ru/library/system.collections.specialized.stringdictionary.remove(v=vs.90).aspx

Если в словаре StringDictionary нет элемента с указанным ключом, словарь StringDictionary не изменяется. Исключение не выдается.

@dmpas

This comment has been minimized.

Copy link
Collaborator

dmpas commented Apr 20, 2018

тесты где? :trollface:

@artbear

This comment has been minimized.

Copy link
Collaborator Author

artbear commented Apr 20, 2018

@artbear

This comment has been minimized.

Copy link
Collaborator Author

artbear commented Apr 20, 2018

и сборка отвалилась :( но не по моей вине :)
http://build.oscript.io/job/1Script/view/change-requests/job/PR-743/1/console

artbear added a commit that referenced this pull request Apr 20, 2018

artbear added a commit that referenced this pull request Apr 20, 2018

Merge pull request #744 from EvilBeaver/process-with-env-743
Тесты для дополнения #743
@artbear

This comment has been minimized.

Copy link
Collaborator Author

artbear commented Apr 20, 2018

А вот и тесты #744

@artbear

This comment has been minimized.

Copy link
Collaborator Author

artbear commented Apr 20, 2018

@dmpas И почему тесты упали?
Два новых и 2 старых, на которых мой PR не влияет :(

@dmpas

This comment has been minimized.

Copy link
Collaborator

dmpas commented Apr 20, 2018

в тестах косяк.
-version output version string - вот отличие выхлопе.

artbear added a commit that referenced this pull request Apr 20, 2018

@artbear artbear merged commit 9cfaf08 into develop Apr 20, 2018

1 of 2 checks passed

continuous-integration/jenkins/pr-merge This commit has test failures
Details
sonarqube SonarQube reported no issues

@artbear artbear deleted the artbear-patch-1 branch Apr 20, 2018

@artbear

This comment has been minimized.

Copy link
Collaborator Author

artbear commented Apr 20, 2018

Тесты упали, но другие, не имеющие отношения к моему исправлению.
http://build.oscript.io/job/1Script/view/change-requests/job/PR-743/6/console
Мержу

@artbear artbear added this to the v1.0.21 milestone Apr 20, 2018

@artbear artbear self-assigned this Apr 20, 2018

@nixel2007

This comment has been minimized.

Copy link
Collaborator

nixel2007 commented Apr 20, 2018

Мне одному предложенное решение кажется странным?..

@nixel2007

This comment has been minimized.

Copy link
Collaborator

nixel2007 commented Apr 20, 2018

А, вижу обсуждение в соседнем репо ( :( )
Если косяк в платформе, то претензий нет

@EvilBeaver

This comment has been minimized.

Copy link
Owner

EvilBeaver commented Apr 21, 2018

@artbear поправь заодно может и упавший тест с version? Бойскауты и все такое...

@artbear

This comment has been minimized.

Copy link
Collaborator Author

artbear commented Apr 21, 2018

@EvilBeaver а что там поправлять-то?
вроде все поправлено и в девелоп теперь зеленые тесты
http://build.oscript.io/job/1Script/job/develop/

@EvilBeaver

This comment has been minimized.

Copy link
Owner

EvilBeaver commented Apr 22, 2018

Вижу в сборке на графике тестов резкое уменьшение их количества. Может поэтому сборка и зеленая?

@allustin

This comment has been minimized.

Copy link
Contributor

allustin commented Apr 25, 2018

Это товарищи

ИНФОРМАЦИЯ - Проверка необходимости выгрузки файла D:\Gherkin\GIT\МояТестоваяВнешняяОбработка.epf в каталог D:\Gherkin\GIT\src, корень D:\Gherkin\GIT
ОШИБКА - {Модуль C:\Program Files (x86)\OneScript\lib\1commands\src\Команда.os / Ошибка в строке: 310 / Внешнее исключение (System.ArgumentException): Элемент уже добавлен. Ключ в словаре: "HOME"  Добавляемый ключ: 'home'

как бы падать то продолжает ;-), надо бы как-то захотфиксить

@artbear

This comment has been minimized.

Copy link
Collaborator Author

artbear commented Apr 25, 2018

Релиз-то не выпущен, пока только в ночной сборке.
Видимо, пока придется сделать исправление в 1коммандс.

@artbear

This comment has been minimized.

Copy link
Collaborator Author

artbear commented Apr 27, 2018

Выпущен релиз 1коммандс с обходом этой проблемы для движков с версией < 1.0.21

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment