FPDB-for-OSX icon indicating copy to clipboard operation
FPDB-for-OSX copied to clipboard

Replace Tkinter code with pyQt code in fpdb_prerun.py

Open astephane opened this issue 8 years ago • 2 comments

Replace Tkinter code with pyQt code in fpdb_prerun.py in order to remove Tkinter dependency.

astephane avatar Jan 19 '17 19:01 astephane

I think the point of fpdb_prerun is to check that dependencies are installed. Dependencies like pyQt. (of course, it is not safe to assume the tkinter works, either, but I didn't write this stuff.)

philroberts avatar Jan 19 '17 20:01 philroberts

Sure, it's like a booststrap mechanism: no Qt UI packages are present, another Tk UI package is used to display in a window (usefull on Windows not so on Linux because we could simply trace to the console).

A better way to remove Tkinter could be to trace to the error log file and/or simply let Python module exception be raised and stop execution.

In deed, I installed Tkinter and I think this issue is not urgent.

astephane avatar Jan 20 '17 11:01 astephane