cordova-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Filip Maj <...@adobe.com>
Subject Re: About that plugin discovery thing....
Date Thu, 25 Jul 2013 16:54:49 GMT
+1

On 7/25/13 9:51 AM, "Andrew Grieve" <agrieve@chromium.org> wrote:

>+1. should live with the rest of plugman.
>
>
>On Thu, Jul 25, 2013 at 12:32 PM, Brian LeRoux <b@brian.io> wrote:
>
>> agree
>>
>> On Thu, Jul 25, 2013 at 12:24 PM, Anis KADRI <anis.kadri@gmail.com>
>>wrote:
>> > I didn't know that there was a Cordova group. I am new to this review
>> process.
>> > The meat is here [1]. Now that I think about it, I don't think it
>> > should be a separate module. It should be part of plugman itself. I
>> > can work on adding it, if it makes sense.
>> >
>> > [1] https://github.com/imhotep/plugman-registry
>> >
>> > On Thu, Jul 25, 2013 at 9:04 AM, Andrew Grieve <agrieve@chromium.org>
>> wrote:
>> >> Yeah, yet another account to create :(
>> >>
>> >> Anis - looked at both reviews, but it seems like the meat is missing.
>> Where
>> >> is the plugman-registry module coming from?
>> >>
>> >>
>> >> On Thu, Jul 25, 2013 at 11:55 AM, Filip Maj <fil@adobe.com> wrote:
>> >>
>> >>> Guess I need to sign up for this review board thing ..
>> >>>
>> >>> On 7/25/13 8:52 AM, "Andrew Grieve" <agrieve@chromium.org> wrote:
>> >>>
>> >>> >In case anyone else wants to have a look, the review requests are:
>> >>> >https://reviews.apache.org/r/12929/
>> >>> >https://reviews.apache.org/r/12931/
>> >>> >
>> >>> >Anis - it'd be good to set the group on these reviews to "cordova",
>> and to
>> >>> >enter in the bug iD.
>> >>> >
>> >>> >
>> >>> >On Wed, Jul 24, 2013 at 9:28 PM, Andrew Grieve
>><agrieve@chromium.org>
>> >>> >wrote:
>> >>> >
>> >>> >> Neat! I like this one:
>> >>> >>
>> >>> >>
>> >>>
>>
>>>>"_from":"/Users/anis/Projects/cordova/cordova-coho/cordova-plugin-camer
>>>>a/
>> >>> >>"
>> >>> >>
>> >>> >>
>> >>> >> So long as we're generating the package.json, we can commandeer
>> fields
>> >>> >>that we know get stored for our own purposes :). E.g. stuff
the
>> engine
>> >>> >>info in the _from field.
>> >>> >>
>> >>> >>
>> >>> >>
>> >>> >> On Wed, Jul 24, 2013 at 7:23 PM, Anis KADRI
>><anis.kadri@gmail.com>
>> >>> >>wrote:
>> >>> >>
>> >>> >>> Just `curl
>> >>> >>>http://registry.cordova.io/camera`<
>> http://registry.cordova.io/camera
>> >>> >(or
>> >>> >>> any plugin name) or
>> >>> >>> enter it in your browser to see what's stored. Basically:
"not a
>> hell
>> >>> >>> of a lot" xD.
>> >>> >>>
>> >>> >>> I've created a review request for Andrew and Braden, both
>>plugman
>> and
>> >>> >>> cli and if you guys agree I can merge it this week. I was
hoping
>> today
>> >>> >>> but it can wait a few more days.
>> >>> >>>
>> >>> >>> On Wed, Jul 24, 2013 at 4:04 PM, Andrew Grieve <
>> agrieve@chromium.org>
>> >>> >>> wrote:
>> >>> >>> > On Wed, Jul 24, 2013 at 3:47 PM, Anis KADRI <
>> anis.kadri@gmail.com>
>> >>> >>> wrote:
>> >>> >>> >
>> >>> >>> >> Just catching up...
>> >>> >>> >>
>> >>> >>> >> To answer various questions:
>> >>> >>> >>
>> >>> >>> >> I am not sure about the Apache VM but irishcouch
[1] is
>> graciously
>> >>> >>> >> offering us a free CouchDB instance. What's cool
about couch
>>is
>> that
>> >>> >>> >> it's really easy to replicate data across multiple
>>instances. I
>> am
>> >>> >>> >> using a proxy on EC2 to handle URL rewrites and
requests but
>>it
>> >>> >>> >> doesn't do anything else. I believe CouchDB can
do that
>>itself
>> but I
>> >>> >>> >> haven't looked into it.
>> >>> >>> >>
>> >>> >>> > Awesome, yeah, I pinged Mike's bug, but don't think
it's worth
>> >>> >>>delaying
>> >>> >>> if
>> >>> >>> > it doesn't get resolved in time.
>> >>> >>> >
>> >>> >>> >
>> >>> >>> >>
>> >>> >>> >> As far as stats, I haven't looked into it yet
but it's
>> definitely an
>> >>> >>> >> important feature. So it is one of the first things
I will be
>> >>> >>>looking
>> >>> >>> >> into once I merge everything back into master.
>> >>> >>> >>
>> >>> >>> >> It stores the tarball (tgz) indeed and doesn't
store the
>> plugin.xml
>> >>> >>> >> separately unfortunately. We'd have to fork npm
in order to
>>do
>> that.
>> >>> >>> >>
>> >>> >>> > I suppose what I was getting at was - how does "npm
search"
>>work?
>> >>> >>>what
>> >>> >>> info
>> >>> >>> > is kept in the DB besides the tgz? I think we'd want
our
>><engine>
>> >>> >>>info
>> >>> >>> > before downloading the tgz.
>> >>> >>> >
>> >>> >>> >
>> >>> >>> >>
>> >>> >>> >> The README is indeed not up to date (I should
get on that!)
>>but
>> the
>> >>> >>> >> help doc is [2]
>> >>> >>> >>
>> >>> >>> >> As far as naming conventions. According to npm's
conventions
>> >>> >>> >> "The name ends up being part of a URL, an argument
on the
>> command
>> >>> >>> >> line, and a folder name. Any name with non-url-safe
>>characters
>> will
>> >>> >>>be
>> >>> >>> >> rejected. Also, it can't start with a dot or an
underscore."
>> >>> >>> >>
>> >>> >>> >> I am guessing that we could use org.apache.cordova.camera
or
>> >>> >>>whatever
>> >>> >>> >> but is that user friendly ? I don't know....It's
definitely
>> better
>> >>> >>>to
>> >>> >>> >> have that because it allows developers to create
plugins for
>>the
>> >>> >>>same
>> >>> >>> >> functionality without having to name it "camera4156"
for
>> example.
>> >>> >>>It's
>> >>> >>> >> up for debate.
>> >>> >>> >>
>> >>> >>> >> [1] http://www.iriscouch.com/
>> >>> >>> >> [2]
>> >>> >>> >>
>> >>> >>>
>> >>> >>>
>> >>>
>> https://git-wip-us.apache.org/repos/asf?p=cordova-plugman.git;a=blob;f=d
>> >>>
>>
>>>>>oc/help.txt;h=ae90cd7568f28d7816abad8cfa1b786cbee956ef;hb=plugman-regi
>>>>>st
>> >>> >>>ry
>> >>> >>> >>
>> >>> >>> >> On Wed, Jul 24, 2013 at 10:58 AM, Steven Gill
>> >>> >>><stevengill97@gmail.com>
>> >>> >>> >> wrote:
>> >>> >>> >> > I created this issue [1] for fixing the name
tag for the
>> plugins.
>> >>> >>>I
>> >>> >>> think
>> >>> >>> >> > this is what Anis meant. I am creating dev
branches for
>> plugins
>> >>> >>> right now
>> >>> >>> >> > and going to make this change + [2] and [3].
>> >>> >>> >> >
>> >>> >>> >> > [1] https://issues.apache.org/jira/browse/CB-4370
>> >>> >>> >> > [2] https://issues.apache.org/jira/browse/CB-4338
>> >>> >>> >> > [3] https://issues.apache.org/jira/browse/CB-4371
>> >>> >>> >> >
>> >>> >>> >> >
>> >>> >>> >> > On Wed, Jul 24, 2013 at 10:47 AM, Andrew
Grieve <
>> >>> >>> agrieve@chromium.org
>> >>> >>> >> >wrote:
>> >>> >>> >> >
>> >>> >>> >> >> Anis - want to put your changes up on
reviews.apache.org?
>> Would
>> >>> >>> make it
>> >>> >>> >> >> easier to see & comment on your changes.
>> >>> >>> >> >>
>> >>> >>> >> >>
>> >>> >>> >> >> On Wed, Jul 24, 2013 at 12:08 PM, Mike
Billau <
>> >>> >>> mike.billau@gmail.com>
>> >>> >>> >> >> wrote:
>> >>> >>> >> >>
>> >>> >>> >> >> > (Still no word from ASF on the Apache
VM:
>> >>> >>> >> >> > https://issues.apache.org/jira/browse/INFRA-6422)
>> >>> >>> >> >> >
>> >>> >>> >> >> > On Wed, Jul 24, 2013 at 11:50 AM,
Braden Shepherdson <
>> >>> >>> >> >> braden@chromium.org
>> >>> >>> >> >> > >wrote:
>> >>> >>> >> >> >
>> >>> >>> >> >> > > Looks good!
>> >>> >>> >> >> > >
>> >>> >>> >> >> > > I'm wondering about the naming
convention - does that
>> mean
>> >>> >>>we'll
>> >>> >>> >> have
>> >>> >>> >> >> to
>> >>> >>> >> >> > > change all our plugins from
>>java.style.reverse.domains to
>> >>> >>> >> singlewords?
>> >>> >>> >> >> > >
>> >>> >>> >> >> > > Braden
>> >>> >>> >> >> > >
>> >>> >>> >> >> > >
>> >>> >>> >> >> > > On Tue, Jul 23, 2013 at 6:04
PM, Andrew Grieve <
>> >>> >>> >> agrieve@chromium.org
>> >>> >>> >> >> > >wrote:
>> >>> >>> >> >> > >
>> >>> >>> >> >> > >> Will definitely try this
out tomorrow! Super awesome!
>> >>> >>> >> >> > >>
>> >>> >>> >> >> > >> Where is the database hosted?
I remember a while ago
>> Mike
>> >>> >>>was
>> >>> >>> >> going to
>> >>> >>> >> >> > >> look
>> >>> >>> >> >> > >> into getting an Apache
VM to host server things on.
>>Did
>> that
>> >>> >>> come
>> >>> >>> >> >> about?
>> >>> >>> >> >> > >>
>> >>> >>> >> >> > >> One of the things we'll
hugely benefit from this is
>> getting
>> >>> >>> stats
>> >>> >>> >> on
>> >>> >>> >> >> > what
>> >>> >>> >> >> > >> plugins people are installing.
Is that a feature yet?
>> >>> >>> >> >> > >>
>> >>> >>> >> >> > >> Wondering what gets hosted
on the server when you
>> "plugman
>> >>> >>> >> publish".
>> >>> >>> >> >> I'm
>> >>> >>> >> >> > >> guessing a .tgz of your
plugin... Is that right?
>>Does it
>> >>> >>>store
>> >>> >>> the
>> >>> >>> >> >> > >> plugin.xml separately from
the .tgz so that metadata
>> can be
>> >>> >>> queried
>> >>> >>> >> >> from
>> >>> >>> >> >> > >> it?
>> >>> >>> >> >> > >>
>> >>> >>> >> >> > >>
>> >>> >>> >> >> > >>
>> >>> >>> >> >> > >> On Tue, Jul 23, 2013 at
7:50 PM, Anis KADRI <
>> >>> >>> anis.kadri@gmail.com>
>> >>> >>> >> >> > wrote:
>> >>> >>> >> >> > >>
>> >>> >>> >> >> > >> > Sure,
>> >>> >>> >> >> > >> >
>> >>> >>> >> >> > >> > cordova-plugman is
here [1]
>> >>> >>> >> >> > >> >
>> >>> >>> >> >> > >> > cordova-cli is here
[2]
>> >>> >>> >> >> > >> >
>> >>> >>> >> >> > >> > the registry itself
is  here [3] and the website to
>> browse
>> >>> >>> >> plugins
>> >>> >>> >> >> is
>> >>> >>> >> >> > >> here
>> >>> >>> >> >> > >> > [4]
>> >>> >>> >> >> > >> >
>> >>> >>> >> >> > >> > [1] http://bit.ly/1bKTvsd
>> >>> >>> >> >> > >> > [2] http://bit.ly/1bKTHYu
>> >>> >>> >> >> > >> > [3] http://registry.cordova.io
>> >>> >>> >> >> > >> > [4] http://plugins.cordova.io
>> >>> >>> >> >> > >> >
>> >>> >>> >> >> > >> > It's easy to setup
your own registry and it's also
>> easy to
>> >>> >>> >> replicate
>> >>> >>> >> >> > >> from
>> >>> >>> >> >> > >> > one registry to the
other.
>> >>> >>> >> >> > >> >
>> >>> >>> >> >> > >> > -a
>> >>> >>> >> >> > >> >
>> >>> >>> >> >> > >> >
>> >>> >>> >> >> > >> > On Tue, Jul 23, 2013
at 4:41 PM, Steven Gill <
>> >>> >>> >> >> stevengill97@gmail.com>
>> >>> >>> >> >> > >> > wrote:
>> >>> >>> >> >> > >> >
>> >>> >>> >> >> > >> > > Hey Anis,
>> >>> >>> >> >> > >> > >
>> >>> >>> >> >> > >> > > Sounds awesome!
Can you share the repo links for
>>the
>> >>> >>> registry
>> >>> >>> >> >> itself
>> >>> >>> >> >> > >> and
>> >>> >>> >> >> > >> > > any other repos
that are related?
>> >>> >>> >> >> > >> > >
>> >>> >>> >> >> > >> > >
>> >>> >>> >> >> > >> > > On Tue, Jul 23,
2013 at 4:26 PM, Anis KADRI <
>> >>> >>> anis@apache.org>
>> >>> >>> >> >> > wrote:
>> >>> >>> >> >> > >> > >
>> >>> >>> >> >> > >> > > > Hello,
>> >>> >>> >> >> > >> > > >
>> >>> >>> >> >> > >> > > > I wanted
to update the list on my progress on
>>this
>> >>> >>>topic.
>> >>> >>> >> I've
>> >>> >>> >> >> > been
>> >>> >>> >> >> > >> > > working
>> >>> >>> >> >> > >> > > > on this
for the past month or so and I believe
>>it
>> will
>> >>> >>> solve
>> >>> >>> >> the
>> >>> >>> >> >> > >> issue
>> >>> >>> >> >> > >> > of
>> >>> >>> >> >> > >> > > > finding
plugins for our users (core and 3rd
>> party).
>> >>> >>> >> >> > >> > > >
>> >>> >>> >> >> > >> > > > I committed
everything to plugman and
>>cordova-cli
>> >>> >>>under
>> >>> >>> the
>> >>> >>> >> >> > >> > > > 'plugman-registry'
>> >>> >>> >> >> > >> > > >
>> >>> >>> >> >> > >> > > > Everything
should be well documented under the
>> >>> >>>project's
>> >>> >>> >> README
>> >>> >>> >> >> > but
>> >>> >>> >> >> > >> I
>> >>> >>> >> >> > >> > > > should probably
update the wiki and write a
>>blog
>> post
>> >>> >>>as
>> >>> >>> >> well.
>> >>> >>> >> >> > >> >
>> >>> >>> >> >> > >> I don't see the README
updated with any of the new
>> features
>> >>> >>> >> (looking
>> >>> >>> >> >> > here:
>> >>> >>> >> >> > >>
>> >>> >>> >> >> > >>
>> >>> >>> >> >> >
>> >>> >>> >> >>
>> >>> >>> >>
>> >>> >>>
>> >>> >>>
>> >>>
>> https://git-wip-us.apache.org/repos/asf?p=cordova-plugman.git;a=blob_pla
>> >>> >>>in;f=README.md;hb=plugman-registry
>> >>> >>> >> >> > >> )
>> >>> >>> >> >> > >>
>> >>> >>> >> >> > >>
>> >>> >>> >> >> > >>
>> >>> >>> >> >> > >> > > >
>> >>> >>> >> >> > >> > > > Now that
3.0 is released, I would like to merge
>> this
>> >>> >>> feature
>> >>> >>> >> >> back
>> >>> >>> >> >> > >> into
>> >>> >>> >> >> > >> > > > master on
both projects by the end of this
>>week if
>> >>> >>> possible.
>> >>> >>> >> It
>> >>> >>> >> >> > >> would
>> >>> >>> >> >> > >> > be
>> >>> >>> >> >> > >> > > > great if
folks try it out and give me some
>> feedback.
>> >>> >>> >> >> > >> > > >
>> >>> >>> >> >> > >> > > > The registry
is based on NPM and because of
>>that,
>> all
>> >>> >>>of
>> >>> >>> your
>> >>> >>> >> >> > >> plugins
>> >>> >>> >> >> > >> > > need
>> >>> >>> >> >> > >> > > > to be renamed
using NPM's conventions: all
>> lowercase
>> >>> >>>and
>> >>> >>> no
>> >>> >>> >> >> > spaces.
>> >>> >>> >> >> > >> > I've
>> >>> >>> >> >> > >> > > > already
published some to plugins.cordova.io.
>> >>> >>> >> >> > >> > > >
>> >>> >>> >> >> > >> > > > plugman
allows to install, adduser, publish,
>> unpublish
>> >>> >>> and
>> >>> >>> >> >> search.
>> >>> >>> >> >> > >> > > > cordova-cli
only allows installation and
>>search. I
>> >>> >>>think
>> >>> >>> that
>> >>> >>> >> >> > plugin
>> >>> >>> >> >> > >> > > > authoring
should be done with plugman as I am
>>not
>> sure
>> >>> >>> what
>> >>> >>> >> the
>> >>> >>> >> >> > flow
>> >>> >>> >> >> > >> > > would
>> >>> >>> >> >> > >> > > > look like
with cordova-cli. This can change.
>> >>> >>> >> >> > >> > > >
>> >>> >>> >> >> > >> > > > The only
difference between npm and plugman is
>>the
>> >>> >>> >> >> > (un)installation
>> >>> >>> >> >> > >> > part.
>> >>> >>> >> >> > >> > > > Everything
else is pretty much the same. Plugin
>> >>> >>>authors
>> >>> >>> are
>> >>> >>> >> not
>> >>> >>> >> >> > >> > required
>> >>> >>> >> >> > >> > > to
>> >>> >>> >> >> > >> > > > write a
package.json as this is automatically
>> >>> >>>generated
>> >>> >>> by
>> >>> >>> >> the
>> >>> >>> >> >> > tool
>> >>> >>> >> >> > >> > from
>> >>> >>> >> >> > >> > > > the plugin.xml.
I've added a few tags to the
>> >>> >>> specification to
>> >>> >>> >> >> > >> support
>> >>> >>> >> >> > >> > > this
>> >>> >>> >> >> > >> > > > new use
case.
>> >>> >>> >> >> > >> > > >
>> >>> >>> >> >> > >> > > > Let me know
if you have any questions or
>>requests.
>> >>> >>> >> >> > >> > > >
>> >>> >>> >> >> > >> > > > Anis
>> >>> >>> >> >> > >> > > >
>> >>> >>> >> >> > >> > >
>> >>> >>> >> >> > >> >
>> >>> >>> >> >> > >>
>> >>> >>> >> >> > >
>> >>> >>> >> >> > >
>> >>> >>> >> >> >
>> >>> >>> >> >>
>> >>> >>> >>
>> >>> >>>
>> >>> >>
>> >>> >>
>> >>>
>> >>>
>>


Mime
View raw message