RESOLVED FIXED 235334
Speedometer should be locally runnable on Windows
https://bugs.webkit.org/show_bug.cgi?id=235334
Summary Speedometer should be locally runnable on Windows
Ross Kirsling
Reported 2022-01-18 14:36:32 PST
Speedometer should be locally runnable on Windows
Attachments
Patch (1.58 KB, patch)
2022-01-18 14:43 PST, Ross Kirsling
no flags
Patch for landing (1.42 KB, patch)
2022-01-18 14:51 PST, Ross Kirsling
no flags
Ross Kirsling
Comment 1 2022-01-18 14:43:57 PST
Don Olmstead
Comment 2 2022-01-18 14:47:42 PST
Comment on attachment 449427 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=449427&action=review r=me > PerformanceTests/.gitattributes:8 > +Speedometer/resources/todomvc/architecture-examples/emberjs/dist/assets/todomvc-5d3e8eb3d5b3740a33185edcb11eeb57.js -crlf > +Speedometer/resources/todomvc/architecture-examples/emberjs/dist/assets/vendor-820919567eb7bd4d9fac358a90a5aac4.js -crlf My only concern is that the hash would be updated and then this issue would crop up again but that's minor.
Ross Kirsling
Comment 3 2022-01-18 14:51:01 PST
(In reply to Don Olmstead from comment #2) > Comment on attachment 449427 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=449427&action=review > > r=me > > > PerformanceTests/.gitattributes:8 > > +Speedometer/resources/todomvc/architecture-examples/emberjs/dist/assets/todomvc-5d3e8eb3d5b3740a33185edcb11eeb57.js -crlf > > +Speedometer/resources/todomvc/architecture-examples/emberjs/dist/assets/vendor-820919567eb7bd4d9fac358a90a5aac4.js -crlf > > My only concern is that the hash would be updated and then this issue would > crop up again but that's minor. Good call, I think I can use *.js instead.
Ross Kirsling
Comment 4 2022-01-18 14:51:39 PST
Created attachment 449430 [details] Patch for landing
EWS
Comment 5 2022-01-18 15:41:33 PST
Committed r288154 (246149@main): <https://commits.webkit.org/246149@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 449430 [details].
Radar WebKit Bug Importer
Comment 6 2022-01-18 15:42:17 PST
Note You need to log in before you can comment on or make changes to this bug.