Open
Bug 1435276
Opened 7 years ago
Updated 2 years ago
Use new prefs parser in mozprofile Rust port
Categories
(Testing :: Mozbase Rust, enhancement, P5)
Tracking
(Not tracked)
NEW
People
(Reporter: ato, Unassigned)
References
Details
geckodriver uses a minimal Rust port of mozprofile to parse, read,
and write prefs to profiles prior to starting Firefox. As of bug
1423840 Firefox transition to a preference parser written in Rust.
It lives in-tree under modules/libpref/parser and we should be able
to make mozprofile pick that up by path.
Because libpref is not as far as I know a published crate, moving
to it would mean that we couldn’t release geckodriver from GitHub
anymore. This seems like a minor inconvenience since we want
to move releases onto TaskCluster. So either we need to solve
releasing from TC first or we need to live with the prospect of
manually uploading binaries to GitHub for some transition period.
Comment 1•7 years ago
|
||
Can the perf changes in Rust 1.25 be leveraged here (or elsewhere)?
https://blog.rust-lang.org/2018/03/29/Rust-1.25.html
Reporter | ||
Comment 2•7 years ago
|
||
(In reply to Arthur K. from comment #1)
> Can the perf changes in Rust 1.25 be leveraged here (or elsewhere)?
>
> https://blog.rust-lang.org/2018/03/29/Rust-1.25.html
What specifically are you referring to?
Comment 3•7 years ago
|
||
LLVM6(In reply to Andreas Tolfsen ‹:ato› from comment #2)
> (In reply to Arthur K. from comment #1)
> > Can the perf changes in Rust 1.25 be leveraged here (or elsewhere)?
> >
> > https://blog.rust-lang.org/2018/03/29/Rust-1.25.html
>
> What specifically are you referring to?
LLVM 6.
Reporter | ||
Comment 4•7 years ago
|
||
I’m not sure what effect LLVM 6 will have on porting mozprofile to
use the new Firefox prefs parser, but generally speaking central
is compiled with Rust stable, so whatever performance improvements
we get from upgrading our compiler we will also get for mozprofile
(and geckodriver).
Updated•3 years ago
|
Priority: -- → P5
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•