dCore Import Debian Packages to Mountable SCE extensions > dCore X86

sce-update should have option quit

<< < (2/2)

Jason W:
The "q" quit option does not work with the select menu being used in a loop like some SCE tools use.  So now Ctrl-c is mentioned in the select menu as the quit function and that works as it kills the parent script and not just the select menu.  Sce-import still has the "q" quit function as it works in it's case.  In the newest RC.

nitram:
Can we remove the Ctrl-c dialogue and simply use this?

--- Code: ---Enter selection ( 1 - 47 ), Enter to proceed or exit, (n)ext, (p)revious:

--- End code ---


Even if items selected for update check, if user changes mind still gets this:

--- Code: ---Xprogs
 
Press Enter to update check above SCE(s), (q)uit to exit: q

--- End code ---

Just adding 'or exit' so it's bullet proof intuitive but avoids use of Ctrl-c dialogue. I plan to submit revamped UI updates, one goal will be to replace any Ctrl-c dialogues with a more eloquent (q)uit. As Ctrl-c is a built-in break, usually used for misbehaving programs, to me it's intentional use in a properly functioning script should be avoided.

Jason please don't issue new RC for this, just explaining my viewpoint. Eventually as time permits i will send udpates, thanks.

Jason W:
Yeah, I agree with the Ctrl-c thing and not promoting it, though a "q" in the middle of the menu function that is a loop will not work as in this case, unless I have a q choice do a 'kill parent process' thing which I don't think is a good idea to accommodate a quit function.  One was asked if they want to proceed or quit after the selection(s) are made and that is sufficient. 

Adding the Ctrl-c text required some awk newline code, but the changes are there in git so reverting to one line should be no issue.  Since you are going to submit updates and I am not planning changes aside from bug reports that may come up, I will wait for you to send in updates.  Thanks.

sm8ps:
Tested the current solution with Ctrl-C and it does what it should. I fully agree with Nitram that avoiding Ctrl-C would be nicer. I would prefer the proposed solution over the current one.

@Nitram: It would be great, if you could improve the UI further. Thanks for your efforts!

Navigation

[0] Message Index

[*] Previous page