Skip to content

Conversation

@guyush1
Copy link
Owner

@guyush1 guyush1 commented Dec 24, 2024

The newer submodule will create smaller static python libraries.

@guyush1 guyush1 force-pushed the reduce-static-python-size branch from 205e70d to a637c40 Compare December 24, 2024 19:18
@guyush1 guyush1 requested a review from roddyrap December 24, 2024 19:18
@guyush1
Copy link
Owner Author

guyush1 commented Dec 24, 2024

This closes #23

@guyush1
Copy link
Owner Author

guyush1 commented Dec 24, 2024

@roddyrap please have a look at this. Also - could you please review the updated deepfrozen.o file, and check if there are more modules we can remove? the binary size is ~32MB still, but reducing it further is always nice :)

@roddyrap
Copy link
Collaborator

Sure, will try my best.

Updated the python submodule.
The newer submodule will create smaller static python libraries.
@guyush1 guyush1 force-pushed the reduce-static-python-size branch from 7e41a1c to aa49ade Compare December 25, 2024 19:35
@guyush1
Copy link
Owner Author

guyush1 commented Dec 25, 2024

@roddyrap i have removed the said modules (now gdb size is 31MB). could you have a look and approve both submodule and this pr if everything looks fine to you?

1 similar comment
@guyush1
Copy link
Owner Author

guyush1 commented Dec 25, 2024

@roddyrap i have removed the said modules (now gdb size is 31MB). could you have a look and approve both submodule and this pr if everything looks fine to you?

@guyush1
Copy link
Owner Author

guyush1 commented Dec 25, 2024

just FYI: i tried tinkering with the binary in order to make it smaller. the best i got is 28MB, but i had to remove some maybe-needed modules and the compilation time went up by a lot since i used lto (i also tried -O3 but dont get me started on that).

@guyush1 guyush1 merged commit 17346ca into develop Dec 25, 2024
6 checks passed
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.

3 participants