Skip to content

fix(scons): strip annobin/hardened specs after configuration phase#17

Merged
movingpictures83 merged 1 commit into
FIUBioRG:masterfrom
quinnjr:quinnjr/bugfix/annobin-post-config-strip
Jun 18, 2026
Merged

fix(scons): strip annobin/hardened specs after configuration phase#17
movingpictures83 merged 1 commit into
FIUBioRG:masterfrom
quinnjr:quinnjr/bugfix/annobin-post-config-strip

Conversation

@quinnjr

@quinnjr quinnjr commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Strip RHEL annobin/hardened -specs= flags after the configuration phase, not just before it — Perl's ParseConfig(ccopts/ldopts) re-injects them during configure_perl, causing a gcc-annobin plugin version mismatch error (plugin built for gcc 8.5.0 but run with gcc 9.5.0)
  • Add redhat-hardened-ld to the specs filter (was missing)
  • Filter LINKFLAGS and SHLINKFLAGS in addition to compile-flag variables

Follows up on #16 which stripped the specs only in create_base_environment() — insufficient because ParseConfig re-adds them.

Test plan

  • scons completes successfully with no gcc-annobin errors
  • Compile commands no longer contain -specs=...redhat-annobin-cc1 or -specs=...redhat-hardened-cc1

🤖 Generated with Claude Code

The previous fix only stripped RHEL annobin specs in
create_base_environment(), but Perl's ParseConfig (ccopts/ldopts)
re-injected -specs=redhat-annobin-cc1 and related flags during the
configuration phase, causing gcc-annobin plugin version mismatch
failures (built for gcc 8.5.0, running gcc 9.5.0).

- Add redhat-hardened-ld to the specs filter list
- Filter LINKFLAGS and SHLINKFLAGS in addition to compile flags
- Call _remove_annobin_flag and _strip_lto_flags after config.Finish()
  to catch all flags injected by ParseConfig during configuration

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@movingpictures83 movingpictures83 merged commit c6e124a into FIUBioRG:master Jun 18, 2026
3 of 5 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.

2 participants