Skip to content

Change sys.path to ['', '/', '.frozen', '/lib'] to address memory on builds with frozen libs #1310

Description

@kattni

The issue with memory allocation failures on M0 builds with frozen libraries is coming up constantly now. The most reasonable solution is to switch the path to ['', '/', '.frozen', '/lib'] to resolve this.

While it would be ideal to address the underlying issue instead, it's not reasonable right now to fix all of the memory issues we're seeing in the M0 builds. There is no quick answer to fixing the underlying issue, and we need something in place now.

I discussed it again with @caternuson because he's a perfect example of the support people who are dealing with this. If we make this change, the thing that will have to happen is for users to upgrade to the latest version of CircuitPython, a step which is much simpler to identify and explain than explaining frozen modules and the reason for deleting the library from the /lib folder.

Activity

  1. changed the title [-]Change `sys.path` to `['', '/', '.frozen', '/lib']` to address memory issues[/-] [+]Change `sys.path` to `['', '/', '.frozen', '/lib']` to address memory on builds with frozen libs[/+] on Oct 31, 2018
  2. dhalbert commented on Oct 31, 2018

    @dhalbert
    Collaborator

    I agree with this. We can make a 3.1.0 release and also incorporate into master. Happy to do the work.

  3. tannewt commented on Oct 31, 2018

    @tannewt
    Member

    I'm ok with this too. I think its a good compromise because it doesn't let the use-case specific builds creep out into the bundle.

  4. kattni commented on Oct 31, 2018

    @kattni
    Author

    Excellent. I'm going to update the issue on the bundle and close it in favor of this one.

  5. added this to the 3.x milestone on Oct 31, 2018
  6. self-assigned this
    on Oct 31, 2018
  7. kevinjwalters commented on Nov 2, 2018

    @kevinjwalters

    I just made a summary of the 3.0.x situation to make current situation more visible and readable by the masses: https://forums.adafruit.com/viewtopic.php?f=60&t=143160

    It's possible I've not fully understood the situation, please comment/edit as necessary.

  8. dhalbert commented on Nov 2, 2018

    @dhalbert
    Collaborator

    Fixed by #1314. Released in 3.1.1.

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

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions