Skip to content

Improve test_multiply_defined_libsymbols#26953

Merged
sbc100 merged 2 commits into
emscripten-core:mainfrom
sbc100:test_multiply_defined_libsymbols
May 14, 2026
Merged

Improve test_multiply_defined_libsymbols#26953
sbc100 merged 2 commits into
emscripten-core:mainfrom
sbc100:test_multiply_defined_libsymbols

Conversation

@sbc100
Copy link
Copy Markdown
Collaborator

@sbc100 sbc100 commented May 14, 2026

This test was first added in c735bb6. Hopefully these changes make it more obvious what is being tested, and also test more commonly used flags (-shared vs -r).

Split out from #25930.

This test was first added in c735bb6.
@sbc100 sbc100 changed the title Update test_multiply_defined_libsymbols Improve test_multiply_defined_libsymbols May 14, 2026
@sbc100 sbc100 requested a review from kripken May 14, 2026 20:18
Comment thread test/test_other.py
}
''')

# Check linking libA.so multiple times (both directly and indirectly) does not result
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
# Check linking libA.so multiple times (both directly and indirectly) does not result
# Check linking libA.so multiple times (both directly and indirectly) does not result in

Comment thread test/test_other.py Outdated
Co-authored-by: Alon Zakai <alonzakai@gmail.com>
@sbc100 sbc100 merged commit c3736fb into emscripten-core:main May 14, 2026
3 of 15 checks passed
@sbc100 sbc100 deleted the test_multiply_defined_libsymbols branch May 14, 2026 22:45
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