Skip to content

Model100: Add the OneShotConfig to our plugin list#33

Merged
obra merged 1 commit intomasterfrom
f/OneShotConfig
Oct 16, 2022
Merged

Model100: Add the OneShotConfig to our plugin list#33
obra merged 1 commit intomasterfrom
f/OneShotConfig

Conversation

@algernon
Copy link
Copy Markdown
Contributor

No description provided.

@algernon algernon added the enhancement New feature or request label Oct 16, 2022
@algernon algernon requested a review from obra October 16, 2022 00:47
@algernon algernon self-assigned this Oct 16, 2022
@obra obra merged commit 026a600 into master Oct 16, 2022
@obra obra deleted the f/OneShotConfig branch October 16, 2022 21:29
@cxw42
Copy link
Copy Markdown

cxw42 commented Oct 22, 2022

I just got my Model100 yesterday and get compile errors that OneShotConfig is not declared:

/home/cxw/proj/Chrysalis-Firmware-Bundle/Keyboardio/Model100/Model100.ino: In static member function 'static kaleidoscope::EventHandlerResult kaleidoscope_internal::EventDispatcher::apply(Args__&& ...)':
Model100:576:3: error: 'OneShotConfig' was not declared in this scope; did you mean 'EscapeOneShotConfig'?
  576 |   OneShotConfig,
      |   ^~~~~~~~~~~~~

Can this repo be used standalone, or does it have to be a submodule of something else? E.g., I recall there was a relationship between the Model01-Firmware and Kaleidoscope repos. Thanks!

Edit If I check out 50a7fbb, it builds fine. I'm enjoying the feel so far and am looking forward to getting my customizations ported from the Model01 to the Model100! Thanks!

@algernon
Copy link
Copy Markdown
Contributor Author

This repo needs Kaleidoscope, and a recent enough version that has the OneShotConfig plugin.

I think the easiest way to get everything up to date is to nuke the .kaleidoscope and .arduino directories, and run tools/bootstrap again. Unless you use your own copy of Kaleidoscope, and have used the KALEIDOSCOPE_DIR environment variable to point this repo's Makefile to it. In that case, you'll want to update that repo.

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

Labels

enhancement New feature or request

Projects

Development

Successfully merging this pull request may close these issues.

3 participants