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

Incompatible with oh-my-posh #61

Open
FWest98 opened this issue Jul 2, 2022 · 3 comments
Open

Incompatible with oh-my-posh #61

FWest98 opened this issue Jul 2, 2022 · 3 comments

Comments

@FWest98
Copy link

FWest98 commented Jul 2, 2022

I have little understanding of what the code in this module does, but I do know that it is incompatible with the oh-my-posh prompt. To be specific, for some reason the variable $omp_status_cache seems to be empty, causing an invalid argument to be passed to the OMP executable here: https://github.com/JanDeDobbeleer/oh-my-posh/blob/main/src/shell/scripts/omp.fish#L32

When I try to set async_prompt_inherit_variables all, I get even more issues (set: Tried to change the read-only variable “status_generation”). Would you maybe have an idea what is happening here?

@acomagu
Copy link
Owner

acomagu commented Feb 25, 2023

The error Tried to change the read-only variable “status_generation” is fixed at fcc8c25. Cloud you update the plugin and try async_prompt_inherit_variables all again?

If any problem still remains, please reopen. Thank you!

@acomagu acomagu closed this as completed Feb 25, 2023
@FWest98
Copy link
Author

FWest98 commented Mar 5, 2023

I still experience some issues with oh-my-posh. To be specific:

  • I get the following error message regularly: Error: invalid argument "--execution-time" for "-e, --error" flag: strconv.ParseInt: parsing "--execution-time": invalid syntax
  • The transient prompt feature does not work properly anymore

@acomagu acomagu reopened this Mar 6, 2023
@acomagu
Copy link
Owner

acomagu commented Mar 6, 2023

oh-my-posh seems to be using global variables. fish-async-prompt doesn't currently support such prompt functions, as we have not come up with a way to make it work. I'm sorry.

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

No branches or pull requests

2 participants