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
Create a unique & temporary IPYTHONDIR for each testing group. #2202
Conversation
Test results for commit bd7abfb merged into master
Not available for testing: python2.6 |
env['IPYTHONDIR'] = IPYTHONDIR | ||
# print >> sys.stderr, '*** CMD:', ' '.join(self.call_args) # dbg | ||
subp = subprocess.Popen(self.call_args, env=env) | ||
self.pids.append(subp.pid) |
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.
We should probably wait for #2204 to be merged first, and then update this to use that API, right?
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.
Yes.
Yes, exactly. Rebased and force pushed. |
Test results for commit 0b4fd4c merged into master
Not available for testing: python2.6 |
OK, all tests good, merging now. Thanks! |
Create a unique & temporary IPYTHONDIR for each testing group. Following #2148 (unification of Windows / Unix code in iptest), and in progress towards #1880 (Add parallelism to iptest & test_pr), this pull request launches each iptest test group with a unique & temporary `IPYTHONDIR`. This has two benefits: * Insulates the test suite from any craziness in your own configuration. (Try adding `import sys; sys.exit()` to your config file...). * Allows multiple test suites to be launched in parallel without the worry of conflicts.
Create a unique & temporary IPYTHONDIR for each testing group. Following ipython#2148 (unification of Windows / Unix code in iptest), and in progress towards ipython#1880 (Add parallelism to iptest & test_pr), this pull request launches each iptest test group with a unique & temporary `IPYTHONDIR`. This has two benefits: * Insulates the test suite from any craziness in your own configuration. (Try adding `import sys; sys.exit()` to your config file...). * Allows multiple test suites to be launched in parallel without the worry of conflicts.
Following #2148 (unification of Windows / Unix code in iptest), and
in progress towards #1880 (Add parallelism to iptest & test_pr), this
pull request launches each iptest test group with a unique & temporary
IPYTHONDIR
.This has two benefits:
(Try adding
import sys; sys.exit()
to your config file...).conflicts.