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

Update rev3 to rev4 for clueboard #17134

Closed
wants to merge 1 commit into from
Closed

Update rev3 to rev4 for clueboard #17134

wants to merge 1 commit into from

Conversation

austinwilcox
Copy link

@austinwilcox austinwilcox commented May 17, 2022

When running the rev3 version on Arco Linux, I get build errors. On rev4 it works every time

Description

Types of Changes

  • Core
  • Bugfix
  • New feature
  • Enhancement/optimization
  • Keyboard (addition or update)
  • Keymap/layout/userspace (addition or update)
  • Documentation

Issues Fixed or Closed by This PR

Checklist

  • My code follows the code style of this project: C, Python
  • I have read the PR Checklist document and have made the appropriate changes.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • I have tested the changes and verified that they work and don't break anything (as well as I can manage).

When running the rev3 version on Arco Linux, I get build errors. On rev4 it works every time
@tzarc
Copy link
Member

tzarc commented May 17, 2022

I'd be more inclined to try to work out why rev3 isn't working, tbh.

@daskygit
Copy link
Member

I think with rev3 being AVR and rev4 STM32, it's very likely this bug #17064 (comment).

@zvecr
Copy link
Member

zvecr commented May 17, 2022

rev3: AVR
rev4: ARM

Going to be #17064 as its an Arch based distro.

@zvecr zvecr added the wontfix label May 17, 2022
@zvecr zvecr closed this May 17, 2022
@zvecr
Copy link
Member

zvecr commented May 17, 2022

Closing as this doesn't really address the root issue.

@austinwilcox
Copy link
Author

austinwilcox commented May 17, 2022

So as another note, in the following page of the docs we are then referencing rev4.
https://docs.qmk.fm/#/newbs_building_firmware

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.

4 participants