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.

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

fix(developer): debug flag for compiling keyboards #9901

Merged
merged 2 commits into from
Nov 2, 2023

Conversation

mcdurdin
Copy link
Member

Fixes #9885.

The 'Include Debug Info' flag was being ignored by the kmc wrapper.

User Testing

  • TEST_DEBUG_FLAG_ON: In Keyman Developer, open a keyboard project and open the keyboard source file. Ensure that 'Keyboard'/'Include Debug Information' is checked. Compile the keyboard and press F5. The debugger should open.

  • TEST_DEBUG_FLAG_OFF: In Keyman Developer, open a keyboard project and open the keyboard source file. Ensure that 'Keyboard'/'Include Debug Information' is not checked. Compile the keyboard and press F5. The debugger should not open, and instead an error 'You must include debug information to start the debugger...' should be displayed.

Fixes #9885.

The 'Include Debug Info' flag was being ignored by the kmc wrapper.
@mcdurdin mcdurdin requested a review from darcywong00 as a code owner October 31, 2023 02:42
@keymanapp-test-bot keymanapp-test-bot bot added has-user-test user-test-required User tests have not been completed labels Oct 31, 2023
@keymanapp-test-bot
Copy link

keymanapp-test-bot bot commented Oct 31, 2023

User Test Results

Test specification and instructions

  • TEST_DEBUG_FLAG_ON (PASSED): Tested with the attached PR build (Keyman Developer 17.0.203-alpha-test-9901) on Windows 11 OS (VM) and here is my observation: 1. Confirmed that the debugger opens after compiling the keyboard when the 'Include Debug Information' option is enabled. (notes)
  • TEST_DEBUG_FLAG_OFF (PASSED): 1. Ensured that 'Keyboard'/'Include Debug Information' is not checked. 2. Compiled the keyboard and confirmed that the debugger is closed; however, it is displaying an error message on the screen instead. (notes)

@keymanapp-test-bot keymanapp-test-bot bot added this to the A17S25 milestone Oct 31, 2023
@mcdurdin mcdurdin linked an issue Oct 31, 2023 that may be closed by this pull request
8 tasks
@bharanidharanj
Copy link

Test Results

  • TEST_DEBUG_FLAG_ON (PASSED): Tested with the attached PR build (Keyman Developer 17.0.203-alpha-test-9901) on Windows 11 OS (VM) and here is my observation: 1. Confirmed that the debugger opens after compiling the keyboard when the 'Include Debug Information' option is enabled.

  • TEST_DEBUG_FLAG_OFF (PASSED): 1. Ensured that 'Keyboard'/'Include Debug Information' is not checked. 2. Compiled the keyboard and confirmed that the debugger is closed; however, it is displaying an error message on the screen instead.

@keymanapp-test-bot keymanapp-test-bot bot removed the user-test-required User tests have not been completed label Oct 31, 2023
… fix/developer/9885-debug-flag-not-working-in-tike
Base automatically changed from fix/developer/9889-compiler-crash-without-project to master November 2, 2023 03:49
@mcdurdin mcdurdin merged commit 9dad461 into master Nov 2, 2023
2 checks passed
@mcdurdin mcdurdin deleted the fix/developer/9885-debug-flag-not-working-in-tike branch November 2, 2023 03:49
@keyman-server
Copy link
Collaborator

Changes in this pull request will be available for download in Keyman version 17.0.204-alpha

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug(developer): Cannot test within Keyman Developer
4 participants