Skip to content

fix: force load all symbol#9337

Merged
hardfist merged 1 commit intomainfrom
yj/fix-link-bug
Feb 17, 2025
Merged

fix: force load all symbol#9337
hardfist merged 1 commit intomainfrom
yj/fix-link-bug

Conversation

@hardfist
Copy link
Copy Markdown
Contributor

@hardfist hardfist commented Feb 17, 2025

Summary

related to #9334, this is a more formal solution thanks to @Brooooooklyn
borrow from https://github.com/toeverything/AFFiNE/blob/57213781a88cea5b8e33d888a0f71419368b820e/.cargo/config.toml#L8 to avoid MacOS linker bug,
related info: https://stackoverflow.com/questions/2906147/what-does-the-all-load-linker-flag-do

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@netlify
Copy link
Copy Markdown

netlify bot commented Feb 17, 2025

Deploy Preview for rspack canceled.

Name Link
🔨 Latest commit f21a80e
🔍 Latest deploy log https://app.netlify.com/sites/rspack/deploys/67b2de9ac1f5b2000818395b

@github-actions github-actions bot added release: bug fix release: bug related release(mr only) team The issue/pr is created by the member of Rspack. labels Feb 17, 2025
@hardfist hardfist enabled auto-merge (squash) February 17, 2025 07:01
@hardfist hardfist disabled auto-merge February 17, 2025 07:27
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Feb 17, 2025

CodSpeed Performance Report

Merging #9337 will not alter performance

Comparing yj/fix-link-bug (f21a80e) with main (3e65b62)

Summary

✅ 6 untouched benchmarks

@hardfist hardfist merged commit bf96618 into main Feb 17, 2025
106 checks passed
@hardfist hardfist deleted the yj/fix-link-bug branch February 17, 2025 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release: bug fix release: bug related release(mr only) team The issue/pr is created by the member of Rspack.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants