Agreed. Everything is upstream on the branch. The package.json wasn't fully thought out, so please peak at it. Ping me if you need anything, I'll rebase cordova-lib and move to master when you are done.
Thanks Steve. On Fri, Mar 6, 2015 at 4:08 PM, Steven Gill <stevengil...@gmail.com> wrote: > Sure. I can make the changes to release process + coho for it. > > I imagine it won't be released often, so it might be best give > app-hello-world its own release process instead of tying it into tools, > platforms or plugins release. > On Mar 6, 2015 1:04 PM, "Michal Mocny" <mmo...@chromium.org> wrote: > >> (Sorry for all the messages) >> >> Fixed all the remaining breaks, and now all tests pass without >> modification and cordova-cli doesn't need changes. >> >> However, there is a new npm dependency on cordova-app-hello-word from >> cordova-lib. We will need to release that to npm before publishing the >> next tools release. >> >> Steve/Andrew, I will attempt to update coho etc, but I'm not sure of all >> the places to update release instructions for this new thing, nor am I sure >> of all the licensing requirements. Care to advise / help? >> >> Thanks! >> -Michal >> >> On Fri, Mar 6, 2015 at 11:30 AM, Michal Mocny <mmo...@chromium.org> >> wrote: >> >>> Updated as described. >>> >>> Now you no longer need to use the branch of cordova-cli, so I'm pretty >>> sure this is 100% compat. Huzzah. >>> >>> On Fri, Mar 6, 2015 at 11:06 AM, Michal Mocny <mmo...@chromium.org> >>> wrote: >>> >>>> I wanted to originally, but you can --copy-from a www/ only (no >>>> config.xml), and then we need a template. If I change cordova-lib to >>>> depend on app-hello-world then we can remove it. Another reason to do so. >>>> >>>> -Michal >>>> >>>> On Thu, Mar 5, 2015 at 10:17 PM, Andrew Grieve <agri...@chromium.org> >>>> wrote: >>>> >>>>> Can we move the config.xml template out of cordova-lib/templates and >>>>> into >>>>> app-hello-world as a part of this? >>>>> >>>>> On Thu, Mar 5, 2015 at 5:13 PM, Michal Mocny <mmo...@chromium.org> >>>>> wrote: >>>>> >>>>> > The current breaking change is that you cannot create a new project >>>>> using >>>>> > cordova-lib directly, without providing a project template to import >>>>> from. >>>>> > cordova-cli works as it always has, and will provide the >>>>> app-hello-world >>>>> > as a default if you do not use --copy-from / --link-to. >>>>> > >>>>> > However, writing that explanation out now, I realize I could just >>>>> change >>>>> > cordova-lib to be the one to depend on cordova-app-hello-world and >>>>> do the >>>>> > defaulting there, and not break compatibility. I think I originally >>>>> > envisioned the cordova-lib script to be more "primitive", but its >>>>> already >>>>> > doing a tremendous amount of input handling and backwards compat >>>>> > workarounds. Probably I can just make this adjustment. >>>>> > >>>>> > -Michal >>>>> > >>>>> > On Thu, Mar 5, 2015 at 4:57 PM, Nikhil Khandelwal < >>>>> nikhi...@microsoft.com> >>>>> > wrote: >>>>> > >>>>> > > Michal: I'm new to this area - can you please elaborate why this >>>>> change >>>>> > > could potentially break downstream distributions? IS there an API >>>>> change? >>>>> > > >>>>> > > -Nikhil >>>>> > > >>>>> > > -----Original Message----- >>>>> > > From: mmo...@google.com [mailto:mmo...@google.com] On Behalf Of >>>>> Michal >>>>> > > Mocny >>>>> > > Sent: Thursday, March 5, 2015 1:12 PM >>>>> > > To: dev >>>>> > > Subject: Re: [DISCUSS] publishing cordova-app-hello-world to npm >>>>> and >>>>> > > adding as dependency of cordova-lib >>>>> > > >>>>> > > Alight, remote branches (named copy-from-app-hello-world) are >>>>> created for >>>>> > > cordova-lib, cordova-cli, and cordova-app-hello-world. >>>>> > > >>>>> > > There are still 2 failing cordova-lib tests, but I'm working to >>>>> resolve >>>>> > > that. My actual end-to-end tests appear to be working with all use >>>>> > cases. >>>>> > > >>>>> > > Please, please: if you develop a downstream distribution that uses >>>>> > > cordova-lib directly (PhoneGap, BlackBerry, Ionic, etc), please >>>>> try this >>>>> > > out and see if it works for you. I will do so for cca. Please >>>>> forward >>>>> > > this to anyone on your team responsible for this work! >>>>> > > >>>>> > > -Michal >>>>> > > >>>>> > > On Wed, Mar 4, 2015 at 2:21 PM, Nikhil Khandelwal < >>>>> > nikhi...@microsoft.com> >>>>> > > wrote: >>>>> > > >>>>> > > > I like moving this to npm because Apache Git as a distribution >>>>> > > > mechanism for the hello world app has been less reliable than >>>>> desirable >>>>> > > in the past. >>>>> > > > >>>>> > > > Thanks, >>>>> > > > Nikhil >>>>> > > > >>>>> > > > >>>>> > > > -----Original Message----- >>>>> > > > From: mmo...@google.com [mailto:mmo...@google.com] On Behalf Of >>>>> Michal >>>>> > > > Mocny >>>>> > > > Sent: Tuesday, March 3, 2015 12:08 PM >>>>> > > > To: Michal Mocny >>>>> > > > Cc: dev >>>>> > > > Subject: Re: [DISCUSS] publishing cordova-app-hello-world to npm >>>>> and >>>>> > > > adding as dependency of cordova-lib >>>>> > > > >>>>> > > > Pushed first attempt to remote branches: >>>>> > > > - >>>>> https://github.com/Apache/cordova-cli/tree/copy-from-app-hello-world >>>>> > > > - >>>>> > > > >>>>> > > > >>>>> https://github.com/Apache/cordova-app-hello-world/tree/copy-from-app-h >>>>> > > > ello-world >>>>> > > > >>>>> > > > Cleaning up cordova-lib is strictly required, though I think we >>>>> should >>>>> > > > still take the opportunity to do so. >>>>> > > > >>>>> > > > On Tue, Mar 3, 2015 at 2:46 PM, Michal Mocny < >>>>> mmo...@chromium.org> >>>>> > > wrote: >>>>> > > > >>>>> > > > > Jesse, I think not. The platform scripts just copy the >>>>> bundled www/ >>>>> > > > > (its already included with the platform). >>>>> > > > > >>>>> > > > > This is only for the top-level project www/ of a cordova-cli >>>>> project. >>>>> > > > > >>>>> > > > > I guess, though, that we could consider changing the platform >>>>> create >>>>> > > > > scripts to import config.xml's, since they currently do not. >>>>> This >>>>> > > > > may be difficult given that the format is different even >>>>> though the >>>>> > > > > name is the same (platform vs project config). >>>>> > > > > >>>>> > > > > -Michal >>>>> > > > > >>>>> > > > > On Tue, Mar 3, 2015 at 2:40 PM, Jesse <purplecabb...@gmail.com >>>>> > >>>>> > wrote: >>>>> > > > > >>>>> > > > >> +1 >>>>> > > > >> Does this affect the ability to use an individual platform? >>>>> > > > >> ie. cordova-ios/bin/create ... >>>>> > > > >> Will platform create scripts need to change? >>>>> > > > >> >>>>> > > > >> >>>>> > > > >> >>>>> > > > >> @purplecabbage >>>>> > > > >> risingj.com >>>>> > > > >> >>>>> > > > >> On Tue, Mar 3, 2015 at 11:32 AM, Michal Mocny < >>>>> mmo...@google.com> >>>>> > > > wrote: >>>>> > > > >> >>>>> > > > >> > This came up in the context of whitelist discussion, but >>>>> wanted >>>>> > > > >> > to >>>>> > > > >> bring it >>>>> > > > >> > to everyones attention. >>>>> > > > >> > >>>>> > > > >> > JIRA: https://issues.apache.org/jira/browse/CB-8597 >>>>> > > > >> > >>>>> > > > >> > We would like the default app to support installing the >>>>> whitelist >>>>> > > > >> > plugin automatically using the new plugin auto-restore >>>>> feature >>>>> > > > >> > (If you >>>>> > > > >> disagree, >>>>> > > > >> > discuss on the whitelist thread, that is not the topic of >>>>> this >>>>> > > post). >>>>> > > > >> > >>>>> > > > >> > However, the cordova-lib create script cannot support this >>>>> > > > >> > feature right now anyway (that is the topic of this post). >>>>> I've >>>>> > > > >> > outlined in the JIRA >>>>> > > > >> how >>>>> > > > >> > we could fix that (open to feedback). >>>>> > > > >> > >>>>> > > > >> > TLDR; --copy-from already supports this, so lets just vastly >>>>> > > > >> > simplify cordova-lib create.js to just always do a >>>>> --copy-from >>>>> > > > >> > (or --link-to), >>>>> > > > >> and >>>>> > > > >> > change cordova-cli to provide the default >>>>> > > > >> > --copy-from=./node_modules/cordova-app-hello-world. >>>>> > > > >> > >>>>> > > > >> > This also implies we can remove our last use of lazy_load >>>>> from >>>>> > > > >> cordova-lib >>>>> > > > >> > (minus explicit git clones or platforms/plugins), and npm >>>>> all the >>>>> > > > >> things! >>>>> > > > >> > >>>>> > > > >> > It also means a semver MAJOR change to cordova-lib that >>>>> > > > >> > downstreams directly using cordova-lib's create will need to >>>>> > adjust >>>>> > > to. >>>>> > > > >> > >>>>> > > > >> > Thoughts? >>>>> > > > >> > >>>>> > > > >> > -Michal >>>>> > > > >> > >>>>> > > > >> >>>>> > > > > >>>>> > > > > >>>>> > > > >>>>> > > >>>>> > > >>>>> --------------------------------------------------------------------- >>>>> > > To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org >>>>> > > For additional commands, e-mail: dev-h...@cordova.apache.org >>>>> > > >>>>> > >>>>> >>>> >>>> >>> >>