sce-remove occasinally hangs when selecting quit

[nitram]

Noticed this even before 'quit' was added to the select list, didn't get a chance to report.

Occasionally sce-remove hangs after selecting two or more SCEs for removal then selecting (q)uit, 100% processor usage. When it works quit allows continuation almost instantly, otherwise something hangs. Running dCore-jessie, noticed across multiple reboots. Does not appear dependent on a specific SCE. Actually also hangs when selecting only one SCE for removal, then re-running sce-remove to add another item later during session (>1 item in /tmp/.removesce). The items selected do get added to /tmp/.removesce just sce-remove won't exit, thanks.

[Jason W]

Ok, will test it, thanks.

[nitram]

Only hangs when terminator is selected for removal, regardless of whether it's the only SCE chosen for removal, numerous removals or even when selected with lxterminal. May re-import tomorrow to re-test but suspect the most likely cause is that lxterminal.sce.dep file contains terminator. It is the only dependency on my system. So instead of getting a 'can't remove terminator as it will break lxterminal functionality', something hangs.

[Jason W]

I noticed similar at one point as for the hang, I will try to reproduce and find the issue.

But instead of issuing a warning about terminator breaking lxterminal functionality, it smiply puts both on the removal list.

[nitram]

Re-importing terminator didn't help, neither did renaming lxterminal.sce.dep to get it out of the way. When all lxterminal related files were manually removed from /sce directory then sce-remove no longer hung when attempting to remove terminator. Copying all lxterminal files back to /sce, re-attempt to remove terminator and it hangs again. So definitely a dependency issue. Note, terminator gets added to /tmp/.removesce but it hangs. Still think it should not get added to .removesce without warning user it will break another extension. Thanks.

Edit: The lxterminal dependency was added properly through sce-import -d.

[Jason W]

Ok, will look into it further.  I see the possible infinite loop that is occuring, should be simple to fix.