Added a command line option to workaround the wxWidget locale issue

- The automatic mechanism fails (broken and broken again!)
- Related to #493

Author: set-soft

kicost/__main__.py

@@ -237,6 +237,9 @@ def main_real():
     #                         action='store_true',
     #                         help='Run KiCost on terminal using the parameters in the GUI memory, all passed parameters from'
     #                              ' terminal take priority.')
+    parser.add_argument('--force_en_us',
+                        action='store_true',
+                        help='Workaround for broken wxWidgets locale on some Windows systems.')
     parser.add_argument('--setup',
                         action='store_true',
                         help='Run KiCost integration (with KiCad and OS) configuration script.')
@@ -316,13 +319,13 @@ def main_real():
     if args.gui:
         if not GUI_ENABLED:
             kicost_gui_notdependences()
-        kicost_gui([os.path.abspath(fileName) for fileName in args.gui])
+        kicost_gui(args.force_en_us, [os.path.abspath(fileName) for fileName in args.gui])
         sys.exit(0)
 
     if args.input is None:
         if not GUI_ENABLED:
             kicost_gui_notdependences()
-        kicost_gui()  # Use the user gui if no input is given.
+        kicost_gui(args.force_en_us)  # Use the user gui if no input is given.
         sys.exit(0)
     else:
         # Match the EDA tool formats with the input files.

kicost/kicost_gui.py

@@ -1158,25 +1158,20 @@ def close(self):
         pass
 
 
-def kicost_gui(files=None):
+def kicost_gui(force_en_us=False, files=None):
     ''' @brief Load the graphical interface.
         @param String file file names or list.
         (it will be used for plugin implementation on future KiCad6-Eeschema).
     '''
     app = wx.App(redirect=False)
-    loc = wx.Locale(wx.LANGUAGE_DEFAULT)
-    locale_retry = False
+    loc = wx.Locale(wx.LANGUAGE_DEFAULT if not force_en_us else wx.LANGUAGE_ENGLISH_US)
     if not loc.IsOk():
-        logger.warning(W_LOCFAIL+"Failed to set the default locale")
-        locale_retry = True
+        if not force_en_us:
+            logger.warning(W_LOCFAIL+"Failed to set the default locale, try using `--force_en_us`")
+        else:
+            logger.warning(W_LOCFAIL+"`--force_en_us` doesn't seem to help")
     elif not loc.GetLocale() and not loc.GetName():
-        logger.warning(W_LOCFAIL+"Unsupported locale")
-        locale_retry = True
-    if locale_retry:
-        loc = wx.Locale(wx.LANGUAGE_ENGLISH_US)
-        logger.warning(W_LOCFAIL+"Trying with US english locale")
-    if not loc.IsOk():
-        logger.warning(W_LOCFAIL+"Failed to set the en_US locale")
+        logger.warning(W_LOCFAIL+"Unsupported locale"+(", try using `--force_en_us`" if not force_en_us else ""))
     else:
         logger.debug('wxWidgets locale {} ({}) system: {}'.format(loc.GetLocale(), loc.GetName(), locale.getlocale()))
     frame = formKiCost(None)