NEW301469
webkitscmpy.program.command shouldn't hardcode "Canonical link"
https://bugs.webkit.org/show_bug.cgi?id=301469
Summary webkitscmpy.program.command shouldn't hardcode "Canonical link"
Sam Sneddon [:gsnedders]
Reported 2025-10-25 12:39:13 PDT
As part of bug 266028, we need to migrate everything to supporting actual git trailers. We have: ``` Tools/Scripts/libraries/webkitscmpy/webkitscmpy/program/command.py 79: NO_FILTER_RES = [re.compile(r' Canonical link:'), re.compile(r' git-svn-id:')] ``` I'm not totally sure what the goal of this is, because making the list empty still has all the tests passing, and trying to run e.g. `git-webkit log --hash` just makes the functionality seem pretty broken. Thinking about this in abstract, I don't know when it expects it to be indented v. when not, as for `git-webkit log` this depends on the `--format` (etc.) option. Either way, we should at the least look at `identifier_template` rather than hardcoding this.
Attachments
Radar WebKit Bug Importer
Comment 1 2025-10-25 12:39:19 PDT
Note You need to log in before you can comment on or make changes to this bug.