[webkit-reviews] review requested: [Bug 33791] check-webkit-style: Checking patch does not work when script not run from source root : [Attachment 46809] Proposed patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon Jan 18 03:51:03 PST 2010
Chris Jerdonek <cjerdonek at webkit.org> has asked for review:
Bug 33791: check-webkit-style: Checking patch does not work when script not run
from source root
https://bugs.webkit.org/show_bug.cgi?id=33791
Attachment 46809: Proposed patch
https://bugs.webkit.org/attachment.cgi?id=46809&action=review
------- Additional Comments from Chris Jerdonek <cjerdonek at webkit.org>
[Serial patch: this patch assumes the patch from bug 33775 has been checked in
(awaiting review). As a result, the status bots may not work.]
In this patch, in addition to the one-line fix, I'm also experimenting with a
new way to manage intra-package references. As a sibling to the style/ folder,
I'm adding a style_references.py module to house external dependencies. The
reasoning behind this is more fully explained in the code comments at the
beginning of style_references.py.
As a side benefit, this also allows us to remove the __path__ hack that I added
a while back to __init__.py. See, for instance--
https://bugs.webkit.org/show_bug.cgi?id=32971#c3
https://bugs.webkit.org/show_bug.cgi?id=32971#c6
More information about the webkit-reviews
mailing list