Skip to content

fix pexpect usage - #69

Closed
zimbatm wants to merge 1 commit into
takluyver:masterfrom
zimbatm:fix-pexpect
Closed

zimbatm wants to merge 1 commit into
takluyver:masterfrom
zimbatm:fix-pexpect

Conversation

@zimbatm

@zimbatm zimbatm commented Nov 22, 2017

Copy link
Copy Markdown
Contributor

pexpect wasn't finding the right prompt match and the kernel would then
timeout on the jupyter side and get killed.

pexpect wasn't finding the right prompt match and the kernel would then
timeout on the jupyter side and get killed.

Use `bash --login` and assume that it's immediately there instead of
waiting for the first prompt.

Pass the unformatted `prompt_change` to the parent class as it's going
to take care of the formatting by itself. This will make it more
resitant to changes in pexpect.
@zimbatm

zimbatm commented Jan 21, 2021

Copy link
Copy Markdown
Contributor Author

ping?

@kdm9

kdm9 commented Aug 24, 2022 •

Copy link
Copy Markdown
Collaborator

I've recently taken on co-maintaining bash_kernel, and I'm currently closing all old/stale issues and PRs.

I'm closing this PR, as reading the diff I can't discern the purpose. Does the issue you were trying to solve persist with the latest version of bash_kernel? Please feel free to reopen this PR if it is still needed for you.

@kdm9 kdm9 closed this Aug 24, 2022
@zimbatm

zimbatm commented Aug 24, 2022

Copy link
Copy Markdown
Contributor Author

it's been a while, but see the commit message:

pexpect wasn't finding the right prompt match and the kernel would then
timeout on the jupyter side and get killed.

Use `bash --login` and assume that it's immediately there instead of
waiting for the first prompt.

Pass the unformatted `prompt_change` to the parent class as it's going
to take care of the formatting by itself. This will make it more
resitant to changes in expect.

I'm not using bash_kernel anymore so won't pursue this

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants