-
Notifications
You must be signed in to change notification settings - Fork 13
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
fix #38 Запуска команды и получения процесса выполнения #39
Conversation
/ping @artbear |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
есть предложения и дополнения
src/Команда.os
Outdated
// Выполнение произвольной команды | ||
// | ||
// Возвращаемое значение: | ||
// Число - код возврата команды |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
в описании возвращается Число - код возврата команды
а в коде возвращается Процесс :(
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Косяк! ))) Поправлю
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
еще остались замечания
packagedef
Outdated
@@ -4,6 +4,7 @@ | |||
.ВерсияСреды("1.0.17") | |||
.ЗависитОт("logos") | |||
.ЗависитОт("asserts") | |||
.ЗависитОт("semver") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Зависимость от semver теперь же не нужна!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Убрал зависимость
src/Команда.os
Outdated
// строку. Добавляются после исполняемого файла. | ||
// | ||
// Возвращаемое значение: | ||
// Число - Код возврата приложения |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
число вместо процесса так и осталось :(
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Поправил
Тесты вроде как упали |
@artbear упали т.к. и не работали (наверное) я хз как починить странный тест |
@@ -375,14 +397,14 @@ | |||
|
|||
Если ИспользуемКомандныйПроцессор Тогда | |||
Если ЭтоWindows Тогда | |||
ШаблонЗапуска = "cmd /c ""%1"""; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@khorevaa А вот с удаленной тобой строкой "cmd /c ""%1""";
что-то важное было связано.
сначала вроде бы не было кавычек, а затем они понадобились.
@nixel2007 @EvilBeaver не помните, почему они стояли?
#38
/cc @artbear