Skip to content

feat: Overhaul font format into CrossPoint font#589

Draft
daveallie wants to merge 1 commit intomasterfrom
feature/crosspoint-font
Draft

feat: Overhaul font format into CrossPoint font#589
daveallie wants to merge 1 commit intomasterfrom
feature/crosspoint-font

Conversation

@daveallie
Copy link
Member

@daveallie daveallie commented Jan 28, 2026

Summary

Additional Context


AI Usage

While CrossPoint doesn't have restrictions on AI tools in contributing, please be transparent about their usage as it
helps set the right context for reviewers.

Did you use AI tools to help write this code? No

@daveallie daveallie force-pushed the feature/crosspoint-font branch from 7322f72 to 567fa6e Compare January 28, 2026 12:59
@cbix
Copy link
Contributor

cbix commented Feb 10, 2026

@daveallie keep in mind Group5 uses GPL-3.0-or-later (restrictive) while CrossPoint is MIT (permissive), so the resulting binary (if it contains the compiled Group5 lib) could only be distributed under GPL as well. IANAL but you might consider one of these options

  1. making everything GPL3
  2. keep MIT but add separate build targets with/without Group5 so people can build an MIT-compatible firmware
  3. clean-room reimplementation of the format under MIT (feels a bit excessive tho)
  4. it seems Group5 changed its license here, can't tell if the previous license is more or less permissive and would allow use of that older version under MIT
  5. talk to the Group5 author, maybe they can make this library MIT-compatible

Someone with more experience in software licenses should add more context here but I'm just trying to raise the awareness of this potential issue before it's too late :) Also, thanks for all your work, I couldn't be happier with my new e-reader ❤️

@daveallie
Copy link
Member Author

Thanks for your concern @cbix, I did actually talk with @bitbank2 (Group5's author) and confirm that it was ok to include inside of CrossPoint. Perhaps there is a better way to reference this fact through a custom license file.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants