Return-Path: X-Original-To: apmail-cordova-dev-archive@www.apache.org Delivered-To: apmail-cordova-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id A4DD41022B for ; Mon, 21 Oct 2013 14:41:31 +0000 (UTC) Received: (qmail 31916 invoked by uid 500); 21 Oct 2013 14:41:31 -0000 Delivered-To: apmail-cordova-dev-archive@cordova.apache.org Received: (qmail 31896 invoked by uid 500); 21 Oct 2013 14:41:31 -0000 Mailing-List: contact dev-help@cordova.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@cordova.apache.org Delivered-To: mailing list dev@cordova.apache.org Received: (qmail 31888 invoked by uid 99); 21 Oct 2013 14:41:30 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 21 Oct 2013 14:41:30 +0000 X-ASF-Spam-Status: No, hits=1.5 required=5.0 tests=HTML_MESSAGE,RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (nike.apache.org: domain of braden@google.com designates 209.85.214.48 as permitted sender) Received: from [209.85.214.48] (HELO mail-bk0-f48.google.com) (209.85.214.48) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 21 Oct 2013 14:41:24 +0000 Received: by mail-bk0-f48.google.com with SMTP id my13so781643bkb.21 for ; Mon, 21 Oct 2013 07:41:04 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=mime-version:sender:in-reply-to:references:date:message-id:subject :from:to:content-type; bh=iC4A9B9OypCARsQ38QV9ATW4lIJmXOolxrAYwR2rwLc=; b=FKVkVatiwu8IIIOcC2YrVc6n018fCi/itDbstZLa+OX4k58O7cx2zcVLj4RDDUDspe iiFd+qclVHMoxTqDtc7dKR1c/vDD4aZsYCFSzc+5eVkWUiO3rV4JPSU7Ei6hqSWXl9Li BqLcGnDuBus437v3OohPZJ2HF/jeR5PkRvAu8zagTs/706jbch0/MRdJQ/2WAJHXkOc/ OwzOE9OT3vEf/MEvJtd0LMcDlOzjABqsjUJXv55XMqqKB/GxBB2mFw5v3jECFnTJHzXs XB81ays0N2dJiCVKU3A/x78qcWXgpJw0WH/IVwXpA2JVIUdg5jCjY7+mMQnhJb3wHb+x dQIQ== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=mime-version:sender:in-reply-to:references:date:message-id:subject :from:to:content-type; bh=iC4A9B9OypCARsQ38QV9ATW4lIJmXOolxrAYwR2rwLc=; b=GadLafo7aPMUK/UzUU05uAu4LrknbsyFTIb1B5BWwSGCDoNze65bGv/km7v00IcrGa gwU6Fgmro/jTBY85zW4uEb9z/g53F7fxpiAE59aR2wzrRJJwmHmXpXV/QJuy7f7Uh+GA vzn6Zg67mxwxdlV3+Y5XF0J0CNNLD38iibAEw= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:sender:in-reply-to:references:date :message-id:subject:from:to:content-type; bh=iC4A9B9OypCARsQ38QV9ATW4lIJmXOolxrAYwR2rwLc=; b=Gk4aSDjKn7m/xr24ejywVikTMcsueIgZoD45kMZDKZ2r/1vZI1VG/CioPN0rQfovps eAGloDuaD7FeqpDIgkBvyX7p4rwWOu3fajNvXblhFOTlPX3ohPFAZhQBu626dgEuZWwe YiI7s+solJABKPUmxW3yGQV6lTaDIpWz3e4qXJNV0SJG+TdHc9v3Mgu73Gj8BX45oR+h fB8rlMdi5wdq5ekdBEspPKPiFY5FIOeqEI4FX6041divbsL4BW+9LC3oPOE5TLGFACRU UF3zIxSEIz2NcJ4I4+3pOpe0QkxyZz7mkgmvslOoOWJhtTPZzuh+6oDhk6kuOph7RjiK jwDA== X-Gm-Message-State: ALoCoQmpdXFgiSFhMZi9F3NjLWS5RqOhRlCuVVzg7I85afTAXKcLCriXwwUxJvx4PQFsA1fTSvI1eEnnI5nIYKgM4liue95BwFuGs/wnNcH+TRhKz46EfwDMxcU3QS6PTcECl9xxcu8QCgTmqlMSkHamBv6Ib/pgGygx5HIbCdyaUZ4/ebn8/4VcaH9ApXjwYF+DViNtHOl+Og2bOyKfZ+3ce3P40nmTdQ== MIME-Version: 1.0 X-Received: by 10.204.69.202 with SMTP id a10mr1727414bkj.36.1382366464221; Mon, 21 Oct 2013 07:41:04 -0700 (PDT) Sender: braden@google.com Received: by 10.204.51.203 with HTTP; Mon, 21 Oct 2013 07:41:04 -0700 (PDT) In-Reply-To: <44C501F2-4776-43BF-AE9E-397D67BC41C1@redhat.com> References: <1CB028F5-CB9B-479F-B19B-D97FC65C8561@gmail.com> <057FEA9F-E67A-49F2-8BF0-95058096D4DB@redhat.com> <6612DC39-909E-4CD0-AA06-BFAF8D4017A0@redhat.com> <44C501F2-4776-43BF-AE9E-397D67BC41C1@redhat.com> Date: Mon, 21 Oct 2013 10:41:04 -0400 X-Google-Sender-Auth: 4tnWNIcMgsWv9QrJZRtAjw9S9yU Message-ID: Subject: Re: new meta data for plugin.xml From: Braden Shepherdson To: "dev@cordova.apache.org" Content-Type: multipart/alternative; boundary=001a1132ebe80cdd4304e9414693 X-Virus-Checked: Checked by ClamAV on apache.org --001a1132ebe80cdd4304e9414693 Content-Type: text/plain; charset=ISO-8859-1 I'd be happier if it were JSON, but it's not, and the XML doesn't cause enough pain to be worth making the switch. Ian explained it accurately; it's mostly a historical accident that we're using XML. Braden On Mon, Oct 21, 2013 at 10:09 AM, Lucas Holmquist wrote: > yea, this is understandable. wasn't really sure the reasoning, but it > looks like diminishing returns here > On Oct 21, 2013, at 10:06 AM, Michal Mocny wrote: > > > XML is also buying us a couple of small but nice features, such as > > optionally wrapping tags with a tag or (potentially) a > > tag, etc. That functionality would not be expressed as cleanly with > JSON, > > so its not a pure win to move away from XML. > > > > Add to that the fact that we are already perceived to change stuff way > too > > often for no due cause, I just really don't see the value. > > > > > > On Mon, Oct 21, 2013 at 9:28 AM, Ian Clelland > wrote: > > > >> I suspect that it is because plugin.xml was derived (intellectually, if > not > >> literally) from config.xml, which was an XML file because of the W3C > >> Widgets spec, which we tried to adhere to. > >> > >> Whether that spec is still relevant (there doesn't seem to be a lot of > >> vendor interest in it (speaking as an Apache member, *not* as a vendor > >> representative)) is definitely up for debate. There probably are some > gains > >> to be made in switching to a JSON config format, given how much of the > >> project is JavaScript these days, but it might not be worth all of the > work > >> it would take to do it. > >> > >> Ian > >> > >> > >> On Mon, Oct 21, 2013 at 8:35 AM, Lucas Holmquist >>> wrote: > >> > >>> Perhaps this has been brought up before, but why are we using an xml > >>> file? why not make it a json file. > >>> > >>> Plugman is written in node( js ) so why not have the plugin "config" > file > >>> in it's native format. This will probably save a bit of code since the > >> xml > >>> is converted to an object to manipulate anyway. > >>> > >>> > >>> i know this is a little off topic. > >>> > >>> thoughts? > >>> > >>> On Oct 18, 2013, at 5:31 PM, Steven Gill > wrote: > >>> > >>>> I have created an issue to keep track of the registry refactor. > >>>> https://issues.apache.org/jira/browse/CB-5130 > >>>> > >>>> > >>>> On Fri, Oct 18, 2013 at 2:04 PM, Anis KADRI > >>> wrote: > >>>> > >>>>> I added some validation for plugin names (to follow > >>>>> reverse-domain-name convention) a couple of weeks ago but there needs > >>>>> to be more of it for sure. > >>>>> > >>>>> On Fri, Oct 18, 2013 at 11:59 AM, Steven Gill < > stevengill97@gmail.com > >>> > >>>>> wrote: > >>>>>> I have created an issue to track the meta tag addition. > >>>>>> https://issues.apache.org/jira/browse/CB-5128 > >>>>>> > >>>>>> I agree with doing validation with plugman during publish time. We > >>> should > >>>>>> decide soon which ones are going to be mandatory and which ones will > >> be > >>>>>> optional. Probably update the plugin spec + our docs around creating > >>>>>> plugins as well. > >>>>>> > >>>>>> > >>>>>> > >>>>>> On Fri, Oct 18, 2013 at 11:54 AM, Shazron > wrote: > >>>>>> > >>>>>>> Perhaps either plugman or the registry should do some validation, > >> and > >>>>> have > >>>>>>> some "required" fields? I know that PhoneGap Build when you try to > >>>>> submit a > >>>>>>> plugin they error out if you are missing some fields that they > >>> require. > >>>>>>> > >>>>>>> > >>>>>>> On Fri, Oct 18, 2013 at 11:50 AM, Gorkem Ercan < > >>> gorkem.ercan@gmail.com > >>>>>>>> wrote: > >>>>>>> > >>>>>>>> +1 for adding metadata but should more of the metadata be > >> compulsory? > >>>>>>>> > >>>>>>>> JBoss tools plugin discovery uses the cordova.io registry and > some > >>>>> of > >>>>>>> the > >>>>>>>> plugins are missing a lot to. http://snag.gy/aAxjL.jpg is a > >>>>> screenshot > >>>>>>>> that shows how the case. http://snag.gy/J8rl6.jpg is a screenshot > >> of > >>>>> a > >>>>>>> few > >>>>>>>> plugins that has most of its data. As you can see with the missing > >>>>>>>> descriptions etc. it is not possible to do an informed decision on > >>>>>>> whether > >>>>>>>> to use a plugin or not. Although information such as keywords does > >>> not > >>>>>>> seem > >>>>>>>> like important it becomes quite useful when you are trying to find > >> a > >>>>>>>> certain plugin. > >>>>>>>> -- > >>>>>>>> Gorkem > >>>>>>>> > >>>>>>>> > >>>>>>>> On Fri, Oct 18, 2013 at 1:12 PM, Michal Mocny < > mmocny@chromium.org > >>> > >>>>>>> wrote: > >>>>>>>> > >>>>>>>>> +1 to repo / issue / website / docs etc metadata > >>>>>>>>> > >>>>>>>>> -1 *for now* to dependencies at specific versions, and testing > >>>>> related > >>>>>>>>> changes like , just because its not clear what the right > >>>>> solution > >>>>>>>> to > >>>>>>>>> these problems is. We do need to address it, but those topics > >> will > >>>>>>>> likely > >>>>>>>>> move to separate discussions. > >>>>>>>>> > >>>>>>>>> > >>>>>>>>> On Fri, Oct 18, 2013 at 12:24 PM, Lucas Holmquist < > >>>>> lholmqui@redhat.com > >>>>>>>>>> wrote: > >>>>>>>>> > >>>>>>>>>> i was just thinking the same thing :) > >>>>>>>>>> On Oct 18, 2013, at 12:06 PM, Carlos Santana < > >>>>> csantana23@gmail.com> > >>>>>>>>> wrote: > >>>>>>>>>> > >>>>>>>>>>> plugin.xml metadata is looking more and more like a > package.json > >>>>>>>> (i.e. > >>>>>>>>>> npm) > >>>>>>>>>>> ;-p > >>>>>>>>>>> > >>>>>>>>>>> > >>>>>>>>>>> > >>>>>>>>>>> > >>>>>>>>>>> > >>>>>>>>>>> On Fri, Oct 18, 2013 at 11:40 AM, Steve Gill < > >>>>>>> stevengill97@gmail.com > >>>>>>>>> > >>>>>>>>>> wrote: > >>>>>>>>>>> > >>>>>>>>>>>> Yes I meant plugins.xml > >>>>>>>>>>>> > >>>>>>>>>>>> > >>>>>>>>>>>> > >>>>>>>>>>>>> On Oct 18, 2013, at 5:43 AM, Lucas Holmquist < > >>>>>>> lholmqui@redhat.com> > >>>>>>>>>>>> wrote: > >>>>>>>>>>>>> > >>>>>>>>>>>>> > >>>>>>>>>>>>>> On Oct 17, 2013, at 7:54 PM, Steven Gill < > >>>>>>> stevengill97@gmail.com> > >>>>>>>>>>>> wrote: > >>>>>>>>>>>>>> > >>>>>>>>>>>>>> So looks like want to to start including more data on > >>>>>>>>>>>>>> http://plugins.cordova.io. > >>>>>>>>>>>>>> > >>>>>>>>>>>>>> Repo tag -> points to repo where plugin lives > >>>>>>>>>>>>>> Issue tag -> points to issue tracker (with component for > >>>>> jira) > >>>>>>>>>>>>>> > >>>>>>>>>>>>>> Testing related (can get discussed more in testing thread > >>>>>>>>>>>>>> Mode tag -> to differentiate between testing mode and normal > >>>>>>> mode > >>>>>>>>>>>>>> JS module tag for test module > >>>>>>>>>>>>>> > >>>>>>>>>>>>>> Dependency related > >>>>>>>>>>>>>> adding version number to dependency tags so they don't just > >>>>> grab > >>>>>>>>>> latest > >>>>>>>>>>>>>> always. Multiple approaches were discussed and this > >>>>> discussion > >>>>>>>>> should > >>>>>>>>>>>>>> probably happen in a new thread. > >>>>>>>>>>>>>> > >>>>>>>>>>>>>> Thoughts on above? Suggestions for other meta data we should > >>>>>>> look > >>>>>>>>> into > >>>>>>>>>>>>>> adding to config.xml? > >>>>>>>>>>>>> did you mean plugin.xml? > >>>>>>>>>>>>> > >>>>>>>>>>>> > >>>>>>>>>>> > >>>>>>>>>>> > >>>>>>>>>>> > >>>>>>>>>>> -- > >>>>>>>>>>> Carlos Santana > >>>>>>>>>>> > >>>>>>>>>> > >>>>>>>>>> > >>>>>>>>> > >>>>>>>> > >>>>>>> > >>>>> > >>> > >>> > >> > > --001a1132ebe80cdd4304e9414693--