Add option for enable gui event loop when not using matplotlib

[chubei-oppen]

Closes #782

[ghost]

All CLA requirements met.

[int19h]

This bit shouldn't be necessary - debugOptions is a legacy property for boolean flags in debug config that we had before deciding to just use separate boolean properties; it is really only kept for backwards compatibility.

[chubei-oppen]

I was wondering what those optoins are for... Removed in f07013d

[fabioz]

@chubei-oppen I was thinking that in the current way, to get it to work users would need to do additional work (to call something as: pydevd_ipython.inputhook.enable_gui('qt5') or pydevd_ipython.inputhook.set_inputhook(callback)) and if matplotlib is indeed imported afterwards it'd override that value...

So, what do you think about making guiEventLoop a string and if it matches matplotlib (default), nothing would change, otherwise one of the gui constants in pydevd_ipython.inputhook would be expected, then it'd call pydevd_ipython.inputhook.enable_gui with that constant, but if it still doesn't match it'd be expected to map to a callable which could be then be imported/set in pydevd_ipython.inputhook.set_inputhook.

Does that make sense?

Also, note that if that setting is not matplotlib, init_matplotlib_support should be a no-op (so that we don't have the override issue).

[chubei-oppen]

@fabioz Sounds good. I'll find time later for the implementaion.

[chubei-oppen]

@fabioz I force pushed the new implementation.

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

[fabioz]

It seems good to me, but I want to do some additional manual testing before merging (I'm finishing another issue related to gevent and as soon as I finish that I'll do the final checks/integrate this pull request, sorry for the delay here...)

[fabioz]

Closing in favor of #800 (which includes the changes here and has some additional fixes/tests on top of it).