[webkit-dev] location in the tree for third party python code?

David Levin levin at chromium.org
Thu Jan 21 18:11:43 PST 2010


I didn't see a web page about it, but when you submit a patch, every single
bullet mentions only BSD or LGPL 2.1 is accepted.

If you are sending in a patch to existing WebKit code, you agree by clicking
below that your changes are licensed under the existing license terms of the
file you are modifying (i.e., BSD license or GNU Lesser General Public
License v.2.1, LGPL v. 2.1). Please also add your copyright (name and year)
to the relevant files for changes that are more than 10 lines of code.
If you are sending in a new file for inclusion in WebKit (no code copied
from another source), the preferred license is BSD, but LGPL 2.1 is an
option as well. Please include your copyright (name and year) and license
preference (BSD or LGPL 2.1). By clicking below you agree that your file is
licensed under either the BSD license or LGPL 2.1, as indicated in your
file.
If you aren't the author of the patch, you agree to include the original
copyright notices and licensing terms with it, to the extent that they
exist. If there wasn't a copyright notice or license, please make a note of
it. Generally we can only take in patches that are BSD- or LGPL-licensed in
order to maintain license compatibility within the project.



On Thu, Jan 21, 2010 at 6:01 PM, Adam Barth <abarth at webkit.org> wrote:

> According to that site, the package is available under:
>
> License :: OSI Approved :: MIT License
>
> Is that compatible with being landed in the svn.webkit.org tree?
>
> Adam
>
>
> On Thu, Jan 21, 2010 at 5:58 PM, Dirk Pranke <dpranke at chromium.org> wrote:
> > Autoinstall appears to be pretty flaky for me at the moment, and these
> > aren't the sort of scripts that can be flaky :) I'll install
> > simplejson into WebKitTools/simplejson as part of the patch and we can
> > clean it up once things are stable.
> >
> > -- Dirk
> >
> > On Thu, Jan 21, 2010 at 2:45 PM, Eric Seidel <eric at webkit.org> wrote:
> >> Be aware, there are a couple known issues with our current autoinstall
> setup:
> >> https://bugs.webkit.org/show_bug.cgi?id=33632
> >> https://bugs.webkit.org/show_bug.cgi?id=33365
> >>
> >> On Thu, Jan 21, 2010 at 2:35 PM, Adam Barth <abarth at webkit.org> wrote:
> >>> This page makes it looks like we can just autoinstall simplejson:
> >>>
> >>> http://pypi.python.org/pypi/simplejson/
> >>>
> >>> IMHO, that's better than checking it in.
> >>>
> >>> Adam
> >>>
> >>>
> >>> On Thu, Jan 21, 2010 at 1:58 PM, Eric Seidel <eric at webkit.org> wrote:
> >>>> We also have webkitpy/autoinstall.py which knows how to download
> >>>> modules on-demand.  This is useful in the case that you're using code
> >>>> with an incompatible license.
> >>>>
> >>>> see webkitpy/__init__.py for an example of how we pull in mechanize
> >>>> (which is a HUGE module, with a compatible license for most files, but
> >>>> includes a bit of code which is not compatible).
> >>>>
> >>>> On Thu, Jan 21, 2010 at 1:39 PM, David Levin <levin at google.com>
> wrote:
> >>>>> So far, things have been put in place with other files (like the
> python
> >>>>> websockets code) but I think that is confusing for a number of
> reasons:
> >>>>> 1. The level of review needed varies imo between 3rd party code that
> is
> >>>>> being used vs new code added to wk.
> >>>>> 2. The style never seems to match what is done for the rest of
> WebKit.
> >>>>> 3. It becomes unclear how to update it because there aren't always
> good
> >>>>> concise instructions about this.
> >>>>> Personally, I'd much prefer a ThirdParty directory to check these
> things
> >>>>> into which we could use to help address these things.
> >>>>> dave
> >>>>>
> >>>>> On Thu, Jan 21, 2010 at 1:30 PM, Dirk Pranke <dpranke at chromium.org>
> wrote:
> >>>>>>
> >>>>>> Hi all,
> >>>>>>
> >>>>>> I'm about to upload a patch that depends on third-party python code
> >>>>>> (simplejson). The patch is a bunch of scripts that'll live under
> >>>>>> WebKitTools/Scripts . Is there an appropriate place for the
> simplejson
> >>>>>> code? In the absence of a better location, I'll probably check it in
> >>>>>> under WebKitTools.
> >>>>>>
> >>>>>> -- Dirk
> >>>>>> _______________________________________________
> >>>>>> webkit-dev mailing list
> >>>>>> webkit-dev at lists.webkit.org
> >>>>>> http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
> >>>>>
> >>>>>
> >>>>> _______________________________________________
> >>>>> webkit-dev mailing list
> >>>>> webkit-dev at lists.webkit.org
> >>>>> http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
> >>>>>
> >>>>>
> >>>> _______________________________________________
> >>>> webkit-dev mailing list
> >>>> webkit-dev at lists.webkit.org
> >>>> http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
> >>>>
> >>>
> >> _______________________________________________
> >> webkit-dev mailing list
> >> webkit-dev at lists.webkit.org
> >> http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
> >>
> >
> _______________________________________________
> webkit-dev mailing list
> webkit-dev at lists.webkit.org
> http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20100121/fdc4351b/attachment.html>


More information about the webkit-dev mailing list