cordova-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Chuck Lantz <cla...@microsoft.com>
Subject RE: [Review] Plugins Package.json
Date Wed, 04 Feb 2015 20:10:58 GMT
+1

The main reason we left windows8 as an alias is we didn't want to break existing plugins because
of a name change given the code was compatible. So, the same goes for referencing windows8
in plugin.xml - that will apply to the cordova-windows platform for backwards compatibility
reasons. (The intent is to keep cordova-windows version agnostic unless there is a significant
architecture change forces a switch. Today cordova-windows supports 8.0, 8.1, and Phone 8.1
and we expect will ultimately support 10 as it comes out.)  

Really, the platform is just "Windows" now.

At the moment there are two supported repos:

cordova-wp8 - SilverLight based. Pre-dates "Windows Runtime APIs"
cordova-windows - Windows Runtime based.

wp7 is also deprecated.

-Chuck

-----Original Message-----
From: Jesse [mailto:purplecabbage@gmail.com] 
Sent: Wednesday, February 4, 2015 12:05 PM
To: dev@cordova.apache.org
Subject: Re: [Review] Plugins Package.json

Yes, you can skip all the 'windows8' stuff and just use 'windows'.
windows8 is legacy and slowly all plugins are being migrated to just use the 'windows' section
in plugin.xml

@purplecabbage
risingj.com

On Wed, Feb 4, 2015 at 11:36 AM, Steven Gill <stevengill97@gmail.com> wrote:

> I have made the changes you suggested. I like the idea of cordova 
> parent key.
>
> I added cordova-windows8 as a keyword but I would like someone from 
> microsoft to chime in about it. My understanding is that windows8 is 
> around for legacy and windows is preferred. Please correct me if I'm 
> wrong. I'd prefer to just list windows and cordova-windows.
>
> If anyone else has feedback on the package.json file, please share! 
> You can view it at
>
> https://github.com/stevengill/cordova-plugin-device/blob/npmpub/packag
> e.json
>
> I am working on createpackagejson command now
> https://issues.apache.org/jira/browse/CB-8416
>
> On Tue, Feb 3, 2015 at 5:29 PM, Andrew Grieve <agrieve@chromium.org>
> wrote:
>
> > From looking at it, only thought is whether we should put all of the 
> > non-standard fields into a "cordova" parent key. E.g.:
> >
> > {
> >   "name": "cordova-plugin-device",
> >   "cordova": {
> >      "id": "org.apache.cordova.device",
> >      "platforms": [...]
> >   }
> > }
> >
> > Sounds like we need to re-write the file when publishing to CPR 
> > anyways,
> so
> > transforming from this to what is currently expected can be done in 
> > that step.
> >
> > Other tidbits:
> > - windows8 is missing from keywords. Intentional? (not sure if 
> > windows is now an alias?)
> > - author: might just set this to "Apache"
> >
> >
> > On Tue, Feb 3, 2015 at 7:50 PM, Steven Gill <stevengill97@gmail.com>
> > wrote:
> >
> > > Please review at
> > >
> > >
> >
> https://github.com/stevengill/cordova-plugin-device/blob/npmpub/packag
> e.json
> > >
> > > You will notice that I added cordova-PLATFORM as a keyword.
> > >
> > > I have also kept the platforms tag for now. I can remove it but 
> > > maybe
> we
> > > can find some future use for it. We currently need it when 
> > > publising to
> > CPR
> > > but can add/rm it during plugman publish.
> > >
> > > I am going to create a createpackagejson command in plugman that 
> > > will
> > build
> > > something like this from plugin.xml. Other plugin devs will be 
> > > able to
> > use
> > > this command to quickly add package.json files to their plugins.
> > >
> > > I will then modify plugman publish to use the createpackagejson 
> > > command
> > if
> > > needed. Plugman publish will still have to add the contents of the
> readme
> > > (or doc/index.md) to the package.json as well as the platforms tag 
> > > if
> we
> > > decide to remove it. It will also have to change the package-name 
> > > field
> > to
> > > package-id when publishing to CPR. Once published, we can remove 
> > > these changes package.json.
> > >
> > > If the package.json file looks good, I'm going to start adding one 
> > > to
> all
> > > of our plugins.
> > >
> >
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
For additional commands, e-mail: dev-help@cordova.apache.org
Mime
View raw message