#openteacher - 2017-04-29
[09:33:42] <lordnoid> commandoline: works locally until I get to the openteacher.py step, then it's lots of errors :P
[09:51:50] <lordnoid> okay it builds! crazy amount of warnings but at least there's a zip and msi
[09:53:23] <lordnoid> replaced pywin32 by pypiwin32, then gui.py line 349 "float(platform.win32_ver()[0]) >= 6.0" instead of what it says now, and add C:\Python36\Scripts to the PATH ("set PATH=C:\Python36;C:\Python36\Scripts;%PATH%
[10:07:00] <lordnoid> for msi it needs
[10:07:28] <lordnoid> https://wix.codeplex.com/releases/view/60102 and "wb" -> "w" on windowsMsi.py:219 and the correct path to wix on 219
[16:48:50] <commandoline> Nice work lordnoid, I'll make sure to incorporate those changes. :D
[16:50:43] <commandoline> I also somewhat managed to fix the qtwebkit problem by abstracting out the concept of a web engine into a separate module. Depending on what's available, QtWebKit or QWebEngine will be used.
[18:32:45] <commandoline> lordnoid: does it really need such an old WiX?
[19:36:20] <lordnoid> commandoline: well the version is hardcoded in OT, but you could change it and see what happens
[19:37:55] <commandoline> ah, ok. Just checking if there was any other reason. I'll probably try to make that somewhat more flexible so it's not hardcoded to a version.
[19:39:16] <lordnoid> yes and I had to make the env var ProgramFiles(x86) because I'm on 64-bit Windows
[19:39:23] <lordnoid> But I assume appveyor is on 32 bit
[19:39:49] <commandoline> you can choose. I presume using 32 bit is best as it would be more likely to work on both, but unsure.
[19:40:18] <lordnoid> yes there are still 32-bit windows versions out there
[19:41:18] <lordnoid> and our top download country was Algeria last week.. not sure how the 64-bit adoption is there
[19:47:09] <commandoline> ok, let me check if that's the default then.
[19:48:11] <commandoline> probably 64-bit but with all kinds of 32 bit software installed or something
[19:51:31] <lordnoid> can always test it on a 32-bit vm
[19:53:22] <commandoline> yeah, seems like the build system is 64 bit, but almost all software has both versions installed, so PyInstaller should be able to make a 32-bit version.
[19:54:09] <commandoline> & for testing, yes, indeed.
[20:33:49] <commandoline> hmm, seems like the build for windows crashes on pyttsx. Did you just remove it, or did it work for you, lordnoid?
[20:34:38] <commandoline> (i.e. https://ci.appveyor.com/project/openteacher/openteacher-ci)
[20:36:42] <commandoline> hmm, seems like you can use Remote Desktop with an AppVeyor worker, I'll need to try that sometime, should make debugging easier.
[20:56:37] <lordnoid> commandoline: oh right, removed the .decode('UTF-8')
[20:57:29] <lordnoid> I haven't started OT and it will probably crash on startup but at least it builds
[22:43:33] <commandoline> Perhaps, but if it does it'll probably be TTS related, as that's the only platform specific code. Disabling that is probably worth fixing the topo bug.
[22:47:06] <commandoline> hmm, if I remove that .decode() it'll crash the linux TTS. I think I'm just going to disable TTS on windows for now, too much of the bugs we encountered (pypiwin32 too) trace back to it.
[23:13:51] <commandoline> oh, seems like pyttsx is now breaking the arch build too. That's it. rm -rf pyttsx.
[09:51:50] <lordnoid> okay it builds! crazy amount of warnings but at least there's a zip and msi
[09:53:23] <lordnoid> replaced pywin32 by pypiwin32, then gui.py line 349 "float(platform.win32_ver()[0]) >= 6.0" instead of what it says now, and add C:\Python36\Scripts to the PATH ("set PATH=C:\Python36;C:\Python36\Scripts;%PATH%
[10:07:00] <lordnoid> for msi it needs
[10:07:28] <lordnoid> https://wix.codeplex.com/releases/view/60102 and "wb" -> "w" on windowsMsi.py:219 and the correct path to wix on 219
[16:48:50] <commandoline> Nice work lordnoid, I'll make sure to incorporate those changes. :D
[16:50:43] <commandoline> I also somewhat managed to fix the qtwebkit problem by abstracting out the concept of a web engine into a separate module. Depending on what's available, QtWebKit or QWebEngine will be used.
[18:32:45] <commandoline> lordnoid: does it really need such an old WiX?
[19:36:20] <lordnoid> commandoline: well the version is hardcoded in OT, but you could change it and see what happens
[19:37:55] <commandoline> ah, ok. Just checking if there was any other reason. I'll probably try to make that somewhat more flexible so it's not hardcoded to a version.
[19:39:16] <lordnoid> yes and I had to make the env var ProgramFiles(x86) because I'm on 64-bit Windows
[19:39:23] <lordnoid> But I assume appveyor is on 32 bit
[19:39:49] <commandoline> you can choose. I presume using 32 bit is best as it would be more likely to work on both, but unsure.
[19:40:18] <lordnoid> yes there are still 32-bit windows versions out there
[19:41:18] <lordnoid> and our top download country was Algeria last week.. not sure how the 64-bit adoption is there
[19:47:09] <commandoline> ok, let me check if that's the default then.
[19:48:11] <commandoline> probably 64-bit but with all kinds of 32 bit software installed or something
[19:51:31] <lordnoid> can always test it on a 32-bit vm
[19:53:22] <commandoline> yeah, seems like the build system is 64 bit, but almost all software has both versions installed, so PyInstaller should be able to make a 32-bit version.
[19:54:09] <commandoline> & for testing, yes, indeed.
[20:33:49] <commandoline> hmm, seems like the build for windows crashes on pyttsx. Did you just remove it, or did it work for you, lordnoid?
[20:34:38] <commandoline> (i.e. https://ci.appveyor.com/project/openteacher/openteacher-ci)
[20:36:42] <commandoline> hmm, seems like you can use Remote Desktop with an AppVeyor worker, I'll need to try that sometime, should make debugging easier.
[20:56:37] <lordnoid> commandoline: oh right, removed the .decode('UTF-8')
[20:57:29] <lordnoid> I haven't started OT and it will probably crash on startup but at least it builds
[22:43:33] <commandoline> Perhaps, but if it does it'll probably be TTS related, as that's the only platform specific code. Disabling that is probably worth fixing the topo bug.
[22:47:06] <commandoline> hmm, if I remove that .decode() it'll crash the linux TTS. I think I'm just going to disable TTS on windows for now, too much of the bugs we encountered (pypiwin32 too) trace back to it.
[23:13:51] <commandoline> oh, seems like pyttsx is now breaking the arch build too. That's it. rm -rf pyttsx.