Add dev profile to strip symbols and disable debug info (ref #3610) #3611
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This significantly speeds up builds: - with strip symbols and debug 0: 43s - without: 169s
Nutomic
changed the title
|
Would be good to try with just debug=0 because those symbols are really useful for stack traces (e.g. anyhow errors, the use of tracing spans isn't consistent everywhere) and i think they are stripped only after the compilation process anyways? also a comment would be good referencing the issue so no one tries to remove it again like i would |
|
Okay updated the measurements. Somehow Im still getting inconsistent results with the same build command taking sometimes 2 minutes and then 5 minutes for no reason. Anyway it looks like only debug 0 is needed, so I updated it like that and added a comment. |
dessalines
approved these changes
Jul 13, 2023
Nutomic
added a commit
to cetra3/lemmy
that referenced
this pull request
Jul 19, 2023
…#3610) (LemmyNet#3611) * Add dev profile to strip symbols and disable debug info (ref LemmyNet#3610) This significantly speeds up builds: - with strip symbols and debug 0: 43s - without: 169s * add comment, no strip symbols
Nutomic
added a commit
that referenced
this pull request
Jul 21, 2023
Nutomic
added a commit
that referenced
this pull request
Jul 21, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This significantly speeds up builds: