Skip to content

Trim debuginfo in dev builds - #491

Merged
sox8502 merged 3 commits into
mainfrom
daniel/dev-line-tables-debuginfo
Oct 1, 2026
Merged

sox8502 merged 3 commits into
mainfrom
daniel/dev-line-tables-debuginfo

Conversation

@sox8502

@sox8502 sox8502 commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

Shrinks dev/test build output: our crate keeps line tables only, third-party crates get no debuginfo. Release/dist profiles are unchanged; no version bump.

  • Panics and backtraces in our code still show file:line.
  • Lost: debugger variable inspection, and line numbers for panics inside dependency code.

Clean cargo build + cargo test --no-run on the same tree (macOS arm64):

Setting target/ Build time
Full debuginfo (old default) 2.62 GiB 111 s
line-tables-only 2.05 GiB (−22%) 89 s
+ deps debug = false (this PR) 1.86 GiB (−29%) 88 s

Test plan

  • cargo fmt --all --check, cargo clippy --all-targets -- -D warnings, cargo build --locked, cargo test --locked (1010 passed) on the merged tree
  • Size/time comparison above
  • CI green

Full debuginfo made each worktree's target/ grow to several GB. File:line
backtraces are kept; only debugger variable inspection is lost.

Co-Authored-By: Claude <noreply@anthropic.com>
@sox8502

sox8502 commented Oct 1, 2026

Copy link
Copy Markdown
Contributor Author

@greptileai review

@greptile-apps

greptile-apps Bot commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

[Low risk] Adjusts debug symbol output in local builds.

The PR appears safe to merge.

Summary

The PR reduces dev and test build output by disabling debuginfo for third-party crates. The application retains line-table debuginfo; release and dist profiles are unchanged.

Reviews (2) · Last reviewed commit: "Merge remote-tracking branch 'origin/mai..."

@sox8502 sox8502 changed the title Use line-tables-only debuginfo for dev builds Trim debuginfo in dev builds Oct 1, 2026
@sox8502

sox8502 commented Oct 1, 2026

Copy link
Copy Markdown
Contributor Author

@greptileai review

@sox8502
sox8502 merged commit f22d5b5 into main Oct 1, 2026
18 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.

1 participant