Summary: | W3C test importer breaks svg files. | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Carlos Alberto Lopez Perez <clopez> | ||||
Component: | Tools / Tests | Assignee: | Carlos Alberto Lopez Perez <clopez> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | cdumez, clopez, ews-watchlist, glenn, jbedard, webkit-bug-importer, youennf | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
See Also: | https://bugs.webkit.org/show_bug.cgi?id=137968 | ||||||
Attachments: |
|
Description
Carlos Alberto Lopez Perez
2019-10-10 10:43:38 PDT
Created attachment 380658 [details]
Patch
Comment on attachment 380658 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=380658&action=review > Tools/Scripts/webkitpy/w3c/test_importer.py:514 > + _log.info("Rewriting: %s" % new_filepath) Can we use .format strings? % encoding doesn't work in Python 3 (In reply to Jonathan Bedard from comment #2) > Comment on attachment 380658 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=380658&action=review > > > Tools/Scripts/webkitpy/w3c/test_importer.py:514 > > + _log.info("Rewriting: %s" % new_filepath) > > Can we use .format strings? % encoding doesn't work in Python 3 I think it does work in python3 without issue. $ python3 -c 'var="world"; print("Hello %s" % var)' Hello world (In reply to Carlos Alberto Lopez Perez from comment #3) > (In reply to Jonathan Bedard from comment #2) > > Comment on attachment 380658 [details] > > Patch > > > > View in context: > > https://bugs.webkit.org/attachment.cgi?id=380658&action=review > > > > > Tools/Scripts/webkitpy/w3c/test_importer.py:514 > > > + _log.info("Rewriting: %s" % new_filepath) > > > > Can we use .format strings? % encoding doesn't work in Python 3 > > I think it does work in python3 without issue. > > $ python3 -c 'var="world"; print("Hello %s" % var)' > Hello world You're totally right...not sure why I thought that didn't work in Python3. Comment on attachment 380658 [details] Patch Clearing flags on attachment: 380658 Committed r250992: <https://trac.webkit.org/changeset/250992> All reviewed patches have been landed. Closing bug. |