| Summary: | [webkitperl] nativeArchitecture() needs to support remote devices | ||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Jonathan Bedard <jbedard> | ||||||||||||||||||||
| Component: | Tools / Tests | Assignee: | Jonathan Bedard <jbedard> | ||||||||||||||||||||
| Status: | RESOLVED FIXED | ||||||||||||||||||||||
| Severity: | Normal | CC: | aakash_jain, guijemont, saam, webkit-bug-importer, zhifei_fang | ||||||||||||||||||||
| Priority: | P2 | Keywords: | InRadar | ||||||||||||||||||||
| Version: | WebKit Nightly Build | ||||||||||||||||||||||
| Hardware: | Unspecified | ||||||||||||||||||||||
| OS: | Unspecified | ||||||||||||||||||||||
| See Also: | https://bugs.webkit.org/show_bug.cgi?id=213487 | ||||||||||||||||||||||
| Attachments: |
|
||||||||||||||||||||||
|
Description
Jonathan Bedard
2020-06-29 07:23:38 PDT
Created attachment 403066 [details]
Patch
Created attachment 403073 [details]
Patch
Comment on attachment 403066 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=403066&action=review The changes look sensible at first sight. I'll try to run a couple tests locally, though the arm EWS should pick it up soon. > Tools/Scripts/webkitdirs.pm:131 > +my %nativeArchitectureMap = (); Do we need this to be a map? It looks like we would always have exactly one element in it, or are there some nativeArchitecture() calls without arguments that are left there on purpose? (In reply to Guillaume Emont from comment #4) > Comment on attachment 403066 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=403066&action=review > > The changes look sensible at first sight. I'll try to run a couple tests > locally, though the arm EWS should pick it up soon. > > > Tools/Scripts/webkitdirs.pm:131 > > +my %nativeArchitectureMap = (); > > Do we need this to be a map? It looks like we would always have exactly one > element in it, or are there some nativeArchitecture() calls without > arguments that are left there on purpose? It's hard to get rid of all the nativeArchitecture() calls without arguments since other places in webkitdirs may call it. Seemed better to just cache the results on a per-host basis. Comment on attachment 403073 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=403073&action=review > Tools/Scripts/webkitdirs.pm:362 > + if ($target eq 'localhost') { Just realized this won't work, embedded devices will target localhost on a different port. Created attachment 403080 [details]
Patch
Created attachment 403109 [details]
Patch
Created attachment 403115 [details]
Patch
Created attachment 403127 [details]
Patch
Created attachment 403132 [details]
Patch
Created attachment 403141 [details]
Patch
Created attachment 403160 [details]
Patch
The patch makes sense to me, I could locally check that it works with --remote, and the jsc-armv7 ews confirms that it fixes the issues introduced in r263569. I am not a reviewer, but I think that this patch should be landed ASAP as it will unbreak the armv7 and mips bots (EWS and post-commit). Comment on attachment 403160 [details]
Patch
rs=me
Committed r263742: <https://trac.webkit.org/changeset/263742> All reviewed patches have been landed. Closing bug and clearing flags on attachment 403160 [details]. |