keymanapp
[cs_pinyin] CS-Pinyin keyboard initial public version
This is a public version of the CS-Pinyin keyboard (from keyman-keyboards-internal). It contains all the files necessary to build the keyboard and the DLLs that go with it.
~~I will be checking the build agent to ensure that it can build the DLLs.~~ We have checked in builds of the DLLs so that we don't introduce a Windows dependency for building the keyboards repo.
- .cfx file has the wrong name (should match keyboard base name).
- Even when .cfx is named correctly, the keyboard isn't working.
When I try to build I get this error:
cs_pinyin.kps: Warning: Warning: File ...\keyboards\cs_pinyin\source\..\build\KeymnIMX.dll does not exist.
cs_pinyin.kps: Warning: Warning: File ...\keyboards\cs_pinyin\source\..\build\imxconfig.exe does not exist.
cs_pinyin.kps: Warning: Warning: File ...\keyboards\cs_pinyin\source\..\build\KeymnIMX.x64.dll does not exist.
I think the .kps is looking in the wrong place for the .dlls?
A lot of the documentation still talks about Tavultesoft and Keyman 6.
I've just added the work-in-progress label to clarify that this keyboard is not yet really ready for review. The content was put online to give another developer access to the source for their own purposes and I will continue to prepare this for release as time permits.
Do you think we should close the PR and reopen it later? Or just leave it open as a hint to yours truly that something should be done about it?
Missing files in the package have been copied from the legacy .kmp and put in the /build folder. Those files are: imxconfig.exe, KeymnIMX.dll and KeymnIMX.x64.dll.
The tweaks fix the errors, but the keyboard doesn't function well yet. It needs further investigation.
Also, the content of the README.md and CS-Pinyin ReadMe.html have been updated.
Note: those files are supposed to be built during the keyboard build process but the build script needs updating to make it work with the new build.sh infrastructure.
On Thu, 26 Oct 2017 at 2:03 pm, Makara [email protected] wrote:
Missing files in the package have been copied from the legacy .kmp and put in the /build folder. Those files are: imxconfig.exe, KeymnIMX.dll and KeymnIMX.x64.dll.
The tweaks fix the errors, but the keyboard doesn't function well yet. It needs further investigation.
Also, the content of the README.md and CS-Pinyin ReadMe.html have been updated.
— You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/keymanapp/keyboards/pull/64#issuecomment-339571651, or mute the thread https://github.com/notifications/unsubscribe-auth/AESjvc2h0L0jMWY56Klvp3rl9jraPeEGks5swC7BgaJpZM4OkzN2 .
@mcdurdin Two years ago you wrote: "Do you think we should close the PR and reopen it later? Or just leave it open as a hint to yours truly that something should be done about it?" How well has that hint been working? ;)
@DavidLRowe ask me again in two years. Currently my to do list is probably about 5 years long and getting longer.