Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

bug(developer): kmc KM04003 error doesn't specify what file is missing #12846

Open
darcywong00 opened this issue Dec 21, 2024 · 1 comment · May be fixed by #12879
Open

bug(developer): kmc KM04003 error doesn't specify what file is missing #12846

darcywong00 opened this issue Dec 21, 2024 · 1 comment · May be fixed by #12879
Assignees
@mcdurdin
Labels
bug developer/compilers/ developer/
Milestone
A18S19

Comments

@darcywong00
Copy link
Contributor

From https://community.software.sil.org/t/building-exe-not-working/9572

A Community Site user asked for help troubleshooting why the kmc building .exe wasn't working:

kmc build windows-package-installer source\keyboard.kps --msi keymandesktop.msi --exe setup-redist.exe --out-file keyboard.exe

The error: error KM04003: File does not exist. also does not specify what file does not exist so it’s hard to debug.

@LornaSIL found that kmc 17 required --license .\LICENSE.md
@ermshiperete ermshiperete changed the title fix(developer): kmc KM04003 error doesn't specify what file is missing bug(developer): kmc KM04003 error doesn't specify what file is missing Jan 6, 2025
@ermshiperete
Copy link
Contributor

See also #12854

@ermshiperete ermshiperete marked this as a duplicate of #12854 Jan 6, 2025
@ermshiperete ermshiperete mentioned this issue Jan 6, 2025
Closed
8 tasks
@mcdurdin mcdurdin added this to the A18S19 milestone Jan 13, 2025
@mcdurdin mcdurdin removed the fix label Jan 13, 2025
@mcdurdin mcdurdin moved this to In Progress in Keyman Jan 13, 2025
mcdurdin added a commit that referenced this issue Jan 13, 2025
@mcdurdin
fix(developer): ensure license parameter is required and add tests
dad1df3 
Some additional items arose from this:

* Adds a command-line-tests unit test which does some basic sanity tests
  on `kmc`.
* Cleans up some of the error reporting infrastructure in `kmc` to
  ensure that thrown errors are handled better.
* Removes redundant subshell from `builder_run_action`.

Fixes: #12846
@mcdurdin mcdurdin linked a pull request Jan 13, 2025 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mcdurdin mcdurdin

Labels
bug developer/compilers/ developer/
Projects
Keyman
Status: In Progress
Milestone
A18S19
Development

Successfully merging a pull request may close this issue.

fix(developer): ensure license parameter is required and add tests keymanapp/keyman
3 participants
@ermshiperete @mcdurdin @darcywong00