Review: Needs Fixing
If we were to do it on other platforms, I'm guessing it would be unlikely for 
it to be a .chm file as this is a Microsoft format file. So I think an 
operating system check here would be good to avoid an unnecessary file system 
lookup.

Would it be possible to change the windows build to create the file as 
"OpenLP.chm" and change the corresponding casing here too on line 10?
-- 
https://code.launchpad.net/~smpettit/openlp/windows-help/+merge/65976
Your team OpenLP Core is subscribed to branch lp:openlp.

_______________________________________________
Mailing list: https://launchpad.net/~openlp-core
Post to     : openlp-core@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openlp-core
More help   : https://help.launchpad.net/ListHelp

Reply via email to