JustinTimperio / pacback

Advanced Version Control for Arch Linux
MIT License
148 stars 4 forks source link

KeyboardInterrupt error is not handled nicely #18

Closed Gr3q closed 4 years ago

Gr3q commented 4 years ago

Describe the Bug:\ App does not handle KeyboardInterrupts too nicely. Not an urgent matter

Output of Command It looks like this

Do You Want to Remove These Packages From Your System? (y/n):^CTraceback (most recent call last):
  File "/usr/bin/pacback", line 115, in <module>
    rb.rollback_to_rp(version, args.rollback)
  File "/usr/share/pacback/core/rollback_rp.py", line 162, in rollback_to_rp
    if PS.YN_Frame('Do You Want to Remove These Packages From Your System?') is True:
  File "/usr/share/pacback/core/python_scripts/ps_defuns.py", line 58, in YN_Frame
    yn = input('\033[1m' + prompt + ' (y/n):' + '\033[0m')
KeyboardInterrupt

Machine (please fill in the following information):

To Reproduce Steps to reproduce the behavior:

  1. Do a rollback
  2. Press Ctrl+C

Expected behavior Exit without errors

JustinTimperio commented 4 years ago

Sorry for the slow response. I think I was expecting people to input 'no' instead of 'ctrl+c'. I will take a look at this since I expect there are a number of places where the program doesn't exit cleanly.

JustinTimperio commented 4 years ago

This has not been forgotten! This will be fixed in the full release which hopefully will be in the next week or two.

JustinTimperio commented 4 years ago

This feature is now present in the latest release