[Openlp-core] [Bug 1844044] Re: Dev build (2.5) MySQL error during boot, some tables created but not all

2019-09-15 Thread Dave Bevan
And subsequently, I re-ran the 1st-time wizard, with MySQL mode still enabled of course. I selected all plugins for enablement. Caused the following error: Traceback (most recent call last): File

[Openlp-core] [Bug 1844044] Re: Dev build (2.5) MySQL error during boot, some tables created but not all

2019-09-15 Thread Dave Bevan
I should have added: MariaDB [openlp]> show tables; +--+ | Tables_in_openlp | +--+ | authors | | media_files | | metadata | | song_books | | songs| | topics | +--+ 6 rows in set (0.00 sec) -- You

[Openlp-core] [Bug 1844044] Re: Dev build (2.5) MySQL error during boot, some tables created but not all

2019-09-15 Thread Dave Bevan
And also, there's a Database Error dialog box appearing during launch: OpenLP cannot load your database. Database: sqlite:home/bevand10/.local/share/openlp/songs/songs.sqlite This is despite it knowing it's in MySQL mode. -- You received this bug notification because you are a member of

[Openlp-core] [Bug 1844044] Re: Dev build (2.5) MySQL error during boot, some tables created but not all

2019-09-15 Thread Dave Bevan
Hi Tomas. Patch applied. Different error now. Latest log attached. ** Attachment added: "openlp-post-UnicodeText-patch.log" https://bugs.launchpad.net/openlp/+bug/1844044/+attachment/5288872/+files/openlp-post-UnicodeText-patch.log -- You received this bug notification because you are a

[Openlp-core] [Merge] lp:~ic90/openlp/animated-alerts into lp:openlp

2019-09-15 Thread noreply
The proposal to merge lp:~ic90/openlp/animated-alerts into lp:openlp has been updated. Status: Needs review => Merged For more details, see: https://code.launchpad.net/~ic90/openlp/animated-alerts/+merge/369540 -- Your team OpenLP Core is subscribed to branch lp:openlp.

[Openlp-core] [Merge] lp:~tomasgroth/openlp/reveal-css into lp:openlp

2019-09-15 Thread noreply
The proposal to merge lp:~tomasgroth/openlp/reveal-css into lp:openlp has been updated. Status: Needs review => Merged For more details, see: https://code.launchpad.net/~tomasgroth/openlp/reveal-css/+merge/372798 -- Your team OpenLP Core is subscribed to branch lp:openlp.

Re: [Openlp-core] [Merge] lp:~raoul-snyman/openlp/animated-alerts into lp:openlp

2019-09-15 Thread Tim Bentley
Review: Approve -- https://code.launchpad.net/~raoul-snyman/openlp/animated-alerts/+merge/372792 Your team OpenLP Core is subscribed to branch lp:openlp. ___ Mailing list: https://launchpad.net/~openlp-core Post to :

Re: [Openlp-core] [Merge] lp:~tomasgroth/openlp/reveal-css into lp:openlp

2019-09-15 Thread Tim Bentley
Review: Approve -- https://code.launchpad.net/~tomasgroth/openlp/reveal-css/+merge/372798 Your team OpenLP Core is subscribed to branch lp:openlp. ___ Mailing list: https://launchpad.net/~openlp-core Post to : openlp-core@lists.launchpad.net

Re: [Openlp-core] [Merge] lp:~raoul-snyman/openlp/animated-alerts into lp:openlp

2019-09-15 Thread Tomas Groth
Review: Approve Looks ok to me. -- https://code.launchpad.net/~raoul-snyman/openlp/animated-alerts/+merge/372792 Your team OpenLP Core is subscribed to branch lp:openlp. ___ Mailing list: https://launchpad.net/~openlp-core Post to :

[Openlp-core] Linting: Passed

2019-09-15 Thread Raoul Snyman
Linting passed! -- https://code.launchpad.net/~tomasgroth/openlp/reveal-css/+merge/372798 Your team OpenLP Core is requested to review the proposed merge of lp:~tomasgroth/openlp/reveal-css into lp:openlp. ___ Mailing list:

[Openlp-core] macOS Test Results: Passed

2019-09-15 Thread Raoul Snyman
macOS tests passed! -- https://code.launchpad.net/~tomasgroth/openlp/reveal-css/+merge/372798 Your team OpenLP Core is requested to review the proposed merge of lp:~tomasgroth/openlp/reveal-css into lp:openlp. ___ Mailing list:

[Openlp-core] Linux Test Results: Passed

2019-09-15 Thread Raoul Snyman
Linux tests passed! -- https://code.launchpad.net/~tomasgroth/openlp/reveal-css/+merge/372798 Your team OpenLP Core is requested to review the proposed merge of lp:~tomasgroth/openlp/reveal-css into lp:openlp. ___ Mailing list:

[Openlp-core] JavaScript Test Results: Passed

2019-09-15 Thread Raoul Snyman
JavaScript tests passed! -- https://code.launchpad.net/~tomasgroth/openlp/reveal-css/+merge/372798 Your team OpenLP Core is requested to review the proposed merge of lp:~tomasgroth/openlp/reveal-css into lp:openlp. ___ Mailing list:

[Openlp-core] [Merge] lp:~tomasgroth/openlp/reveal-css into lp:openlp

2019-09-15 Thread Tomas Groth
Tomas Groth has proposed merging lp:~tomasgroth/openlp/reveal-css into lp:openlp. Commit message: Forgot to update to latest CSS when updating RevealJS to 3.8.0. Requested reviews: OpenLP Core (openlp-core) For more details, see:

[Openlp-core] [Bug 1844044] Re: Dev build (2.5) MySQL error during boot, some tables created but not all

2019-09-15 Thread Tomas Groth
Looking at the logs, this seems to be the issue: sqlalchemy.exc.CompileError: (in table 'media_files', column 'file_path'): VARCHAR requires a length on dialect mysql We should probably also test is postgresql works. -- You received this bug notification because you are a member of OpenLP

[Openlp-core] [Bug 1844044] [NEW] Dev build (2.5) MySQL error during boot, some tables created but not all

2019-09-15 Thread Dave Bevan
Public bug reported: OpenLP 2.5 dev2902 Ubuntu 18.04.3 LTS Configured for MySQL (happens to be MariaDB 10.1.41-MariaDB- 0ubuntu0.18.4.1) Debug log: (full is attached) MySQL tables after the error: MariaDB [openlp]> show tables; +--+ | Tables_in_openlp | +--+ |