search

keymanapp / keyman

Keyman cross platform input methods system running on Android, iOS, Linux, macOS, Windows and mobile and desktop web
https://keyman.com/
Other
398 stars 112 forks source link

[kmlmi] Improve error message when author URL undefined #2370

Closed darcywong00 closed 4 years ago

darcywong00 commented 4 years ago

For keymanapp/lexical-models#60 I was stuck because the lexical model package compiled fine in Developer, but the build script failed with the error

Building model sample/example/example.ta.wordlist/
Cannot read property 'match' of undefined
Unable to merge .model_info file
Aborting with error 1

This was cause kmlmi was looking for an author URL in the .kps file. It would be nice to have a clearer error.

mcdurdin commented 4 years ago

Fix should be to allow kps without author url at all. kps spec does not require author.