Skip to content

Update README.md - #1

Merged
devoncarew merged 1 commit into
mainfrom
devoncarew-patch-1
Mar 21, 2023
Merged

devoncarew merged 1 commit into
mainfrom
devoncarew-patch-1

Conversation

@devoncarew

Copy link
Copy Markdown
Contributor
  • fix a comment in the readme

@devoncarew
devoncarew requested a review from dcharkes March 21, 2023 18:18
@devoncarew
devoncarew enabled auto-merge (squash) March 21, 2023 20:39
@devoncarew
devoncarew merged commit cdb380b into main Mar 21, 2023
@devoncarew
devoncarew deleted the devoncarew-patch-1 branch March 21, 2023 20:56
HosseinYousefi pushed a commit that referenced this pull request Nov 16, 2023
HosseinYousefi pushed a commit that referenced this pull request Nov 16, 2023
HosseinYousefi pushed a commit that referenced this pull request Nov 16, 2023
mosuem added a commit that referenced this pull request Oct 9, 2026
…very OS

macOS and iOS have the same problem as Windows: `-u` makes the link fail if
the symbol isn't defined (ld64: "must be defined for the link to succeed").
On Linux and Android, `-u` of an undefined symbol leaves an undefined dynamic
symbol in the library, which fails at load time with `-z now`.

readArchiveSymbols now also reads the BSD symbol table of macOS and iOS
archives (`__.SYMDEF [SORTED]`, `__.SYMDEF_64 [SORTED]`, with the `#1/<len>`
member names of Apple's tools), checked against llvm-ar --format=darwin and
llvm-nm --print-armap, and the filter applies to the `-u` flags, the exported
symbols list and the version script, matching the leading underscore of
Mach-O. The tree-shaking tests ask for an undefined symbol on every OS.

Fixes #3737
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