bugzilla-tool should not require users to install mechanize First step is to centralize our handling of mechanize imports. Second step is to make a magical way to have it auto-downloaded. (Another alternative would be to check in a copy of mechanize, but it's *huge* and we don't have a ThirdParty directory to check it in to.) This lazy-download approach is slightly more error-prone than checking in a copy, but should still work.
Created attachment 45020 [details] Step one
style-queue ran check-webkit-style on attachment 45020 [details] without any errors.
Comment on attachment 45020 [details] Step one I'll re-open the bug after the cq closes it. :)
Comment on attachment 45020 [details] Step one Clearing flags on attachment: 45020 Committed r52228: <http://trac.webkit.org/changeset/52228>
All reviewed patches have been landed. Closing bug.
I'm going to be a hypocrite and re-use this bug for the second step if I can ever get it working...
So looks like step two could be done via easy_install with a --prefix directory. Something like this: PYTHONPATH=./webkit_module_cache/lib/python2.5/site-packages easy_install --prefix ./webkit_module_cache mechanize and then we would edit sys.path to include ./webkit_module_cache
I think we can use http://pypi.python.org/pypi/autoinstall/0.1a2 to solve this problem. I've emailed the author about licensing.
Created attachment 45215 [details] work in progress patch
Comment on attachment 45215 [details] work in progress patch Sorry, forgot --no-review
I've spoken with the autoinstall.py author. His intent was to make it clear that it is available under a BSD license, however he's agreed to release a new version which makes that explicit. I also have a local patch to send him as well.
A new autoinstall module has been released: http://pypi.python.org/pypi/autoinstall/0.2 which clarifies that it's BSD licensed code. This should be easy to finish, I just need to take a few moments and do it.
Created attachment 45777 [details] Patch
Created attachment 45779 [details] Patch
Comment on attachment 45779 [details] Patch This sauce is awesome
Committed r52718: <http://trac.webkit.org/changeset/52718>
Committed r52723: <http://trac.webkit.org/changeset/52723>