|Summary:||webkitpy: Pass dynamic libraries and frameworks to devices for install|
|Product:||WebKit||Reporter:||Jonathan Bedard <jbedard>|
|Component:||Tools / Tests||Assignee:||Jonathan Bedard <jbedard>|
|Severity:||Normal||CC:||ap, buildbot, dbates, glenn, lforschler, webkit-bug-importer|
|Version:||WebKit Nightly Build|
Description Jonathan Bedard 2017-03-29 14:09:33 PDT
If a device is not based on the host machine running webkitpy, the dynamic libraries and frameworks needed for the driver to run need to be installed along with the app. Pass these in an array to install_app.
Comment 3 Alexey Proskuryakov 2017-03-29 14:20:39 PDT
Comment 5 Daniel Bates 2017-04-03 14:47:00 PDT
You did not address Alexey's remark on the liability of this approach in commen 3. You simply updated the patch to include the webrtc dylib that Alexey pointed out was missing. Please attempt another solution or explain why your proposed patch is the only solution or the best of the worst solutions.
Comment 6 Daniel Bates 2017-04-03 14:48:13 PDT
If you cannot think of another solution then ask
Comment 7 Jonathan Bedard 2017-04-03 15:30:58 PDT
(In reply to Daniel Bates from comment #6) > If you cannot think of another solution then ask I briefly discussed this with Alexey and Simon last week, although have yet to talk with them about this approach. The trouble is, we don't have a list of dependent dynamic libraries anywhere in webkitpy. Such a list would be an unfortunate burden to maintain, but the alternative solution would be to install every .framework and .dylib in the build directory, which would increase the amount of time and space taken up by an install (neither of which is trivial on devices). The newer patch attempts to address the problem of the lack of a list of dependent dynamic libraries by making it clear that this is the responsibility of each port to maintain, if it is needed. I'm still not convinced this is the best approach, if anyone has any other ideas.
Comment 8 Jonathan Bedard 2017-04-04 10:27:51 PDT
It turns out that just copying every framework and dylib in the build directory takes an extra 15 seconds, and that's only if binaries are not already there. I think that's the best solution here.