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.

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

build: extend GLUON_DEBUG with values 0, 1 and 2 #3295

Merged
merged 2 commits into from
Jun 27, 2024

Conversation

neocturne
Copy link
Member

Debugging binaries in images that are built with sstrip is extremely
annoying, as not even objdump will work on them.

Switching from sstrip to standard strip increases the ath79 image size
(for some usual Archer C7 v2 configuration I used for testing) by ~70KiB,
or 1.3% of the rootfs size (~1% of the total image size), which is
rarely worth the decreased debuggability.

Extend GLUON_DEBUG to not only distinguish 0 and 1, such that:

  • 0 uses sstrip same as before
  • 1 switches to regular strip; this is the new default
  • 2 includes all debug info, like GLUON_DEBUG=1 did before

Debugging binaries in images that are built with sstrip is extremely
annoying, as not even objdump will work on them.

Switching from sstrip to standard strip increases the ath79 image size
(for some usual Archer C7 v2 configuration I used for testing) by ~70KiB,
or 1.3% of the rootfs size (~1% of the total image size), which is
rarely worth the decreased debuggability.

Extend GLUON_DEBUG to not only distinguish 0 and 1, such that:

- 0 uses sstrip same as before
- 1 switches to regular strip; this is the new default
- 2 includes all debug info, like GLUON_DEBUG=1 did before
Unlike Markdown, RST uses double backticks for monospace text; single
backticks result in italic text. Use monospace for commands like the
surrounding paragraphs.
@neocturne neocturne added 0. type: enhancement The changeset is an enhancement 2. status: waiting-on-review Awaiting review from the assignee but also interested parties. 3. topic: build This is about the build system labels Jun 27, 2024
@github-actions github-actions bot added 3. topic: docs Topic: Documentation 3. topic: hardware Topic: Hardware Support labels Jun 27, 2024
@blocktrron blocktrron merged commit c61456e into freifunk-gluon:main Jun 27, 2024
42 checks passed
@neocturne neocturne deleted the debug-default branch July 4, 2024 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0. type: enhancement The changeset is an enhancement 2. status: waiting-on-review Awaiting review from the assignee but also interested parties. 3. topic: build This is about the build system 3. topic: docs Topic: Documentation 3. topic: hardware Topic: Hardware Support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants