Skip to content

fix(qnx): reorder user_link_flags after libraries_to_link - #128

Merged
nradakovic merged 1 commit into
mainfrom
nira_fix_gcov
Aug 26, 2026
Merged

fix(qnx): reorder user_link_flags after libraries_to_link#128
nradakovic merged 1 commit into
mainfrom
nira_fix_gcov

Conversation

@nradakovic

Copy link
Copy Markdown
Member

Fixes undefined reference errors (e.g. __gcov_dump) caused by user linkopts appearing before the objects/libraries that need them on the QNX link command line.
Matches the Linux template's ordering.

Fixes undefined reference errors (e.g. __gcov_dump) caused by
user linkopts appearing before the objects/libraries that need them
on the QNX link command line.
Matches the Linux template's ordering.
@nradakovic nradakovic self-assigned this Aug 25, 2026
Copilot AI lite review requested due to automatic review settings August 25, 2026 18:19
@nradakovic nradakovic added the bug Something isn't working label Aug 25, 2026
@nradakovic nradakovic added template Add or update template toolchain configurations p2 Medium/High - key service impacted - needing rapid resolution labels Aug 25, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the QNX C++ toolchain config template to ensure user-provided link flags (linkopts) are applied after Bazel’s libraries_to_link arguments, preventing link-order–dependent undefined reference errors (e.g., __gcov_dump) and aligning QNX behavior with the Linux template.

Changes:

  • Reordered user_link_flags_feature to come after libraries_to_link_feature in the QNX toolchain features list.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@nradakovic
nradakovic merged commit 6680def into main Aug 26, 2026
19 of 21 checks passed
@nradakovic
nradakovic deleted the nira_fix_gcov branch August 26, 2026 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working p2 Medium/High - key service impacted - needing rapid resolution template Add or update template toolchain configurations

Projects

Development

Successfully merging this pull request may close these issues.

3 participants