Skip to content

fix static linking on windows - #3936

Merged
sbiscigl merged 1 commit into
mainfrom
fix-static-windows
Sep 23, 2026
Merged

sbiscigl merged 1 commit into
mainfrom
fix-static-windows

Conversation

@sbiscigl

Copy link
Copy Markdown
Collaborator

Issue #, if available:

#3935

Description of changes:

fixing a bug where a new macro SMITHY_TEMPLATE_EXPORT was introduced instead of using SMITHY_API to correctly handle windows export semantics

Check all that applies:

  • Did a review by yourself.
  • Added proper tests to cover this PR. (If tests are not applicable, explain.)
  • Checked if this PR is a breaking (APIs have been changed) change.
  • Checked if this PR will not introduce cross-platform inconsistent behavior.
  • Checked if this PR would require a ReadMe/Wiki update.

Check which platforms you have built SDK on to verify the correctness of this PR.

  • Linux
  • Windows
  • Android
  • MacOS
  • IOS
  • Other Platforms

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@sbiscigl
sbiscigl marked this pull request as ready for review September 22, 2026 14:47
@sbiscigl
sbiscigl force-pushed the fix-static-windows branch 5 times, most recently from 1b3f486 to 7c08476 Compare September 23, 2026 13:11
@sbiscigl
sbiscigl added this pull request to the merge queue Sep 23, 2026
Merged via the queue into main with commit e16521c Sep 23, 2026
5 of 6 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.

3 participants