1572618 - Allow rlbox Wasm library sandboxes to be linked statically

Status: RESOLVED FIXED

(Firefox Build System :: General, enhancement, P3)

Comment 1

[Mike Hommey [:glandium]]

Attachment: Bug 1572618 - Statically link the rlbox library to libxul.

We still rely on dynamic loading to find the symbols, but since we get
them from libxul, we don't need to load the library before activating
the process sandbox anymore.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0a70349bb19a
Statically link the rlbox library to libxul. r=firefox-build-system-reviewers,nika,andi

Comment 3

[Shravan Narayan]

Attachment: Bug 1572618 - Allow rlbox Wasm library sandboxes to be linked statically r=glandium

Comment 4

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/0a70349bb19a

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/95c98eb80002
adjust Thunderbird package-manifest.in for "Allow rlbox Wasm library sandboxes to be linked statically". rs=me

Comment 6

[Shravan Narayan]

I believe this is now backed out

Comment 7

[Mike Hommey [:glandium]]

I believe this is now backed out

No it's not.

Comment 8

[Shravan Narayan]

ah sorry for the confusion. You are correct.
TLDR - This bug tracks the moving of wasm-compiled code into xul, which landed. A separate bug will track the static function invocation change.

Comment 9

[Phabricator Automation]

Comment on attachment 9246081 [details]
Bug 1572618 - Allow rlbox Wasm library sandboxes to be linked statically r=glandium

Revision D128566 was moved to bug 1737733. Setting attachment 9246081 [details] to obsolete.