Skip to content
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 PS1-based bash shell integration #151170

Merged
merged 2 commits into from Jun 2, 2022
Merged

Fix PS1-based bash shell integration #151170

merged 2 commits into from Jun 2, 2022

Conversation

Tyriar
Copy link
Member

@Tyriar Tyriar commented Jun 2, 2022

Fixes #150877

Fixes #150877

Co-Authored-By: Megan Rogge <merogge@microsoft.com>
@Tyriar Tyriar added this to the May 2022 milestone Jun 2, 2022
@Tyriar Tyriar requested a review from meganrogge June 2, 2022 22:21
@Tyriar Tyriar self-assigned this Jun 2, 2022
@Tyriar
Copy link
Member Author

Tyriar commented Jun 2, 2022

macOS bash PS1:
Screen Shot 2022-06-02 at 3 25 24 pm

macOS bash PROMPT_COMMAND:
Screen Shot 2022-06-02 at 3 26 17 pm

macOS bash debug trap/starship (warning is unrelated):
Screen Shot 2022-06-02 at 3 26 56 pm

@Tyriar
Copy link
Member Author

Tyriar commented Jun 2, 2022

Linux bash PS1:

image

I think it's safe to assume it works in other cases on Linux too, the bash PS1 problem reproduced on macOS as well, we just didn't notice.

@Tyriar Tyriar enabled auto-merge June 2, 2022 22:36
@Tyriar Tyriar merged commit 7b264cb into main Jun 2, 2022
@Tyriar Tyriar deleted the tyriar/si_bash branch June 2, 2022 22:37
@meganrogge
Copy link
Contributor

also fixes #150892

@github-actions github-actions bot locked and limited conversation to collaborators Jul 17, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bash debug trap doesn't work with shell integration for multi-word functions
2 participants