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.

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

Fixes clean target for applications with large number of fils + P2 PSRAM size calculation issues #2661

Merged
merged 2 commits into from Jun 8, 2023
Merged

Fixes clean target for applications with large number of fils + P2 PSRAM size calculation issues #2661

merged 2 commits into from Jun 8, 2023

Conversation

avtolstoy
Copy link
Member

Problem

  1. Applications with large number of sources can trigger Argument list too long when Make calls shell + rm + list of files to remove
  2. PSRAM size calculations for RTL872x-based platforms do not account for .dynalib section, which also resides in PSRAM

@avtolstoy avtolstoy added the bug label Jun 8, 2023
@scott-brust scott-brust added this to the 5.4.0 milestone Jun 8, 2023
@avtolstoy avtolstoy merged commit 6934511 into develop Jun 8, 2023
12 checks passed
@avtolstoy avtolstoy deleted the fix/p2-psram-section-size-clean-target branch June 8, 2023 18:04
@scott-brust scott-brust mentioned this pull request Jun 13, 2023
Merged
@technobly technobly mentioned this pull request Sep 21, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@scott-brust scott-brust scott-brust approved these changes

@XuGuohui XuGuohui Awaiting requested review from XuGuohui

Assignees
No one assigned
Labels
4.x 5.x bug
Projects
None yet
Milestone
5.4.0
3 participants
@avtolstoy @scott-brust @technobly