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

Crash whith invalid working-dir #890

Open
silentbat opened this issue Jan 9, 2025 · 1 comment
Open

Crash whith invalid working-dir #890

silentbat opened this issue Jan 9, 2025 · 1 comment
Labels
bug Something isn't working

Comments

@silentbat
Copy link

silentbat commented Jan 9, 2025

Error:

---------------------------
Rio: Runtime Error
---------------------------
panicked at frontends\rioterm\src\screen\mod.rs:1501:22:
attempt to calculate the remainder with a divisor of zero

Press Ctrl-C to Copy
---------------------------
OK   
---------------------------

How to replicate:

  • on win10
  • set working-dir in the config to something invalid ("~" for example)
  • start rio (and get the error message)
  • try to close with Alt-F4
  • then instead of pressing enter, press esc (to continue)
  • takes a bit, then error

sometimes i needed to press escape multiple times, but always get this error.

Sorry if this information is insufficient, please tell me what you need. I just started using rio and have no idea whether there is logging.

Nice work on a new terminal, I like it, thank you!


edit:

line of code from the panic: https://github.com/raphamorim/rio/blob/v0.2.2/frontends/rioterm/src/screen/mod.rs#L1501

version from rio -V is rioterm 0.2.2

@raphamorim
Copy link
Owner

Yea, it's a bad behaviour on Rio. Ideally Rio should check if the path exists and then allow to proceed otherwise trigger an error. Marking it as a bug

@raphamorim raphamorim added the bug Something isn't working label Jan 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants