Skip to content

Bug 2042327 - glean-sym: Enable it on iOS by using this library#3471

Merged
badboy merged 1 commit into
mainfrom
glean-sym-enable-it-on-ios-by-using-this-library/nxqmqplwqmpk
Jun 18, 2026
Merged

Bug 2042327 - glean-sym: Enable it on iOS by using this library#3471
badboy merged 1 commit into
mainfrom
glean-sym-enable-it-on-ios-by-using-this-library/nxqmqplwqmpk

Conversation

@badboy

@badboy badboy commented May 28, 2026

Copy link
Copy Markdown
Member

This is the equivalent of using dlsym(RTLD_DEFAULT, name), that is:

looking for the desired symbols using the default shared object search order.
The search will include global symbols in the executable and its

(from the dlsym(3) man page)

@badboy badboy force-pushed the glean-sym-enable-it-on-ios-by-using-this-library/nxqmqplwqmpk branch 2 times, most recently from 57e0f08 to ded3bed Compare June 15, 2026 14:04
@badboy badboy marked this pull request as ready for review June 15, 2026 14:04
@badboy badboy requested a review from a team as a code owner June 15, 2026 14:04
@badboy badboy requested review from jeddai and removed request for a team June 15, 2026 14:04
@badboy

badboy commented Jun 15, 2026

Copy link
Copy Markdown
Member Author

I tested this out with a local build of appservices and this Glean change in Firefox iOS. It's able to load the function pointers.
E.g. when adding some panic to check what it does:

Loaded Glean symbols. ffi_glean_core_rustbuffer_from_bytes: Symbol "ffi_glean_core_rustbuffer_from_bytes"@0x108aa91e8 from "/Users/jer/Library/Developer/Xcode/DerivedData/Client-ggbdmvoodhczijcievulgqgnjkrv/Build/Products/Fennec-iphonesimulator/Glean.framework/Glean"

This is the equivalent of using `dlsym(RTLD_DEFAULT, name)`, that is:

> looking for the desired symbols using the default shared object search order.
> The search will include global symbols in the executable and its

(from the dlsym(3) man page)
@badboy badboy force-pushed the glean-sym-enable-it-on-ios-by-using-this-library/nxqmqplwqmpk branch from ded3bed to a4fee80 Compare June 18, 2026 13:01
@badboy badboy requested a review from travis79 June 18, 2026 13:40
@badboy badboy enabled auto-merge (rebase) June 18, 2026 13:43

@travis79 travis79 left a comment

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.

Let's be sure and eventually document this difference wherever the user facing docs for glean-sym will be

@badboy badboy merged commit 0042f2c into main Jun 18, 2026
29 of 30 checks passed
@badboy badboy deleted the glean-sym-enable-it-on-ios-by-using-this-library/nxqmqplwqmpk branch June 18, 2026 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants