Skip to content

Fix cp1252 build #4751

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
Merged
merged 2 commits into from
May 12, 2021
Merged

Fix cp1252 build #4751

merged 2 commits into from
May 12, 2021

Conversation

jepler
Copy link

@jepler jepler commented May 12, 2021

@ladyada this is a stab in the dark at your build problems reported in #4750.

jepler added 2 commits May 11, 2021 21:48
@jepler
gen_display_resources: Always treat the sample file as utf-8.
35f1da4 
This may help resolve the build problems reported at
adafruit#4750.
@jepler
makeqstrdata.py: Remove a problematic print
45dc095 
.. it contained non-ASCII characters, even when building the standard
English translation.

This may help resolve the build problems reported at adafruit#4750.
@ladyada
Copy link
Member

ladyada commented May 12, 2021 

$ make BOARD="adafruit_qt2040_trinkey" -j 32
Use make V=1, make V=2 or set BUILD_VERBOSE similarly in your environment to increase build verbosity.
QSTR not updated

648392 bytes used, 396088 bytes free in flash firmware space out of 1044480 bytes (1020.0kB).
14884 bytes used, 247260 bytes free in ram for stack and heap out of 262144 bytes (256.0kB).

Converting to uf2, output size: 1292800, start address: 0x10000000
Wrote 1292800 bytes to build-adafruit_qt2040_trinkey/firmware.uf2

@ladyada ladyada merged commit fa485aa into adafruit:main May 12, 2021
@jepler jepler deleted the fix-cp1252-build branch November 3, 2021 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ladyada ladyada ladyada approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jepler @ladyada