Skip to content

Bug: Fix linking with stdc++ #91

Open
apet2929 wants to merge 1 commit intomainfrom
bug/fix_link_std_c++
Open

Bug: Fix linking with stdc++ #91
apet2929 wants to merge 1 commit intomainfrom
bug/fix_link_std_c++

Conversation

@apet2929
Copy link
Copy Markdown
Collaborator

@apet2929 apet2929 commented Apr 6, 2026

Moves library flags (-l) after objects in linker command.
Manual test plan: Add stdc++ to the Megaton.toml config field build.libraries, build a mod with it and see that the check step passes successfully.

@apet2929
Copy link
Copy Markdown
Collaborator Author

apet2929 commented Apr 6, 2026

Manual test plan passed successfully using branch TestMod of new-example-mod. Without "stdc++" in the library flags, the symbol "_ZNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEE15_M_replace_coldEPcmPKcmm" was missing. With "stdc++" in the library flags, no error occurred. Before these changes, this error would occur regardless.

@apet2929 apet2929 requested a review from lwbuchanan April 6, 2026 18:58
@apet2929 apet2929 added the bug Something isn't working label Apr 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant