WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
199738
[GTK][WPE] run-webkit-tests should run with a temporal homedir
https://bugs.webkit.org/show_bug.cgi?id=199738
Summary
[GTK][WPE] run-webkit-tests should run with a temporal homedir
Carlos Alberto Lopez Perez
Reported
2019-07-11 17:30:17 PDT
On
https://bugs.webkit.org/show_bug.cgi?id=169916#c7
I discovered some tests give different visual output depending if you run the tests with on an empty $HOME or not. I guess this is related to some configuration of some library. In order to make the output of the tests more predictable I think run-webkit-tests should run with a temporal new homedir.
Attachments
Patch
(4.19 KB, patch)
2019-07-17 10:06 PDT
,
Carlos Alberto Lopez Perez
no flags
Details
Formatted Diff
Diff
Patch
(4.19 KB, patch)
2019-07-17 18:09 PDT
,
Carlos Alberto Lopez Perez
no flags
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Adrian Perez
Comment 1
2019-07-12 01:19:13 PDT
(In reply to Carlos Alberto Lopez Perez from
comment #0
)
> On
https://bugs.webkit.org/show_bug.cgi?id=169916#c7
I discovered some tests > give different visual output depending if you run the tests with on an empty > $HOME or not.
>
> I guess this is related to some configuration of some library.
Ouch. It could be something like some configuration being present (or not) in $HOME for the hinting and/or subpixel rendering settings for fonts, for example.
> In order to make the output of the tests more predictable I think > run-webkit-tests should run with a temporal new homedir.
Yes, or even have a separate home directory for each test case being run. Or point $HOME to an empty, non-writable directory (not sure if that would work). We may want to also set $XDG_CONFIG_HOME, $XDG_CACHE_DIR, and so on.
Carlos Alberto Lopez Perez
Comment 2
2019-07-17 09:53:42 PDT
(In reply to Adrian Perez from
comment #1
)
> (In reply to Carlos Alberto Lopez Perez from
comment #0
) > > On
https://bugs.webkit.org/show_bug.cgi?id=169916#c7
I discovered some tests > > give different visual output depending if you run the tests with on an empty > > $HOME or not. > > > > I guess this is related to some configuration of some library. > > Ouch. It could be something like some configuration being present > (or not) in $HOME for the hinting and/or subpixel rendering settings > for fonts, for example. > > > In order to make the output of the tests more predictable I think > > run-webkit-tests should run with a temporal new homedir. > > Yes, or even have a separate home directory for each test case > being run.
I will try to use a separate home directory per worker.
> Or point $HOME to an empty, non-writable directory (not > sure if that would work). We may want to also set $XDG_CONFIG_HOME, > $XDG_CACHE_DIR, and so on.
XDG_CONFIG_HOME and XDG_CACHE_DIR are filtered out from the user environment by the webkitpy driver for tests. So I think is enough with just setting $HOME because the value of this two variables default to $HOME/.confing and $HOME/.cache
Carlos Alberto Lopez Perez
Comment 3
2019-07-17 10:06:26 PDT
Created
attachment 374302
[details]
Patch
Carlos Alberto Lopez Perez
Comment 4
2019-07-17 10:09:19 PDT
Comment on
attachment 374302
[details]
Patch Ok. Not ready for review still due to revert of
r247512
Adrian Perez
Comment 5
2019-07-17 12:35:51 PDT
(In reply to Carlos Alberto Lopez Perez from
comment #2
)
> (In reply to Adrian Perez from
comment #1
) > > (In reply to Carlos Alberto Lopez Perez from
comment #0
) > > > On
https://bugs.webkit.org/show_bug.cgi?id=169916#c7
I discovered some tests > > > give different visual output depending if you run the tests with on an empty > > > $HOME or not. > > > > > > I guess this is related to some configuration of some library. > > > > Ouch. It could be something like some configuration being present > > (or not) in $HOME for the hinting and/or subpixel rendering settings > > for fonts, for example. > > > > > In order to make the output of the tests more predictable I think > > > run-webkit-tests should run with a temporal new homedir. > > > > Yes, or even have a separate home directory for each test case > > being run. > > I will try to use a separate home directory per worker.
Wonderful, thanks.
> > Or point $HOME to an empty, non-writable directory (not > > sure if that would work). We may want to also set $XDG_CONFIG_HOME, > > $XDG_CACHE_DIR, and so on. > > XDG_CONFIG_HOME and XDG_CACHE_DIR are filtered out from the user environment > by the webkitpy driver for tests. > So I think is enough with just setting $HOME because the value of this two > variables default to $HOME/.confing and $HOME/.cache
Good point, I had forgotten about the env var filter. Setting $HOME will do.
Carlos Alberto Lopez Perez
Comment 6
2019-07-17 18:09:13 PDT
Created
attachment 374359
[details]
Patch
Carlos Alberto Lopez Perez
Comment 7
2019-07-19 02:56:50 PDT
Comment on
attachment 374359
[details]
Patch Clearing flags on attachment: 374359 Committed
r247642
: <
https://trac.webkit.org/changeset/247642
>
Carlos Alberto Lopez Perez
Comment 8
2019-07-19 02:56:58 PDT
All reviewed patches have been landed. Closing bug.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug