← Back to team overview

openlp-core team mailing list archive

Re: [Merge] lp:~m2j/openlp/cleanups into lp:openlp

 

Review: Needs Fixing

First the minor:
934,945: You have an extra space after the first =

Secondly the more serious, it wouldn't start for me:

Traceback (most recent call last):
  File "openlp.pyw", line 35, in <module>
    from openlp.core import main
  File "C:\Users\Jonathan\Documents\projects\openlp\cleanups\openlp\__init__.py"
, line 31, in <module>
    import core
  File "C:\Users\Jonathan\Documents\projects\openlp\cleanups\openlp\core\__init_
_.py", line 43, in <module>
    from openlp.core.lib import Receiver, check_directory_exists
  File "C:\Users\Jonathan\Documents\projects\openlp\cleanups\openlp\core\lib\__i
nit__.py", line 361, in <module>
    from dockwidget import OpenLPDockWidget
  File "C:\Users\Jonathan\Documents\projects\openlp\cleanups\openlp\core\lib\doc
kwidget.py", line 36, in <module>
    from openlp.core.ui import ScreenList
  File "C:\Users\Jonathan\Documents\projects\openlp\cleanups\openlp\core\ui\__in
it__.py", line 83, in <module>
    from slidecontroller import SlideController, Controller
  File "C:\Users\Jonathan\Documents\projects\openlp\cleanups\openlp\core\ui\slid
econtroller.py", line 128
    else:
       ^
SyntaxError: invalid syntax

-- 
https://code.launchpad.net/~m2j/openlp/cleanups/+merge/95733
Your team OpenLP Core is subscribed to branch lp:openlp.


References