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 the usePTY feature #3054
fix the usePTY feature #3054
Conversation
@@ -248,7 +248,7 @@ def buildCommandKwargs(self, warnings): | |||
|
|||
# check for the usePTY flag | |||
if 'usePTY' in kwargs and kwargs['usePTY'] is not None: | |||
if self.workerVersionIsOlderThan("svn", "2.7"): | |||
if self.workerVersionIsOlderThan("shell", "2.7"): |
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.
This shall not change anything as buildbot worker advertise versions for '*', just fix it for coherency
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.
actually, reporter says that this change did matter.
because there is no "svn" command anymore in the worker package
I looked for other lines with the same problem, and there is none.
some people have reported the usePTY feature to not work. This is a unit test and integration test to make sure it actually works test is only for posix
Codecov Report
@@ Coverage Diff @@
## master #3054 +/- ##
=========================================
- Coverage 88.28% 88% -0.29%
=========================================
Files 314 314
Lines 32899 32899
=========================================
- Hits 29046 28952 -94
- Misses 3853 3947 +94
Continue to review full report at Codecov.
|
|
||
|
||
# This integration test creates a master and worker environment, | ||
# with one builders and a shellcommand step |
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.
"with one builders"
some people have reported the usePTY feature to not work.
This is a unit test and integration test to make sure it actually works
test is only for posix