But what should we store in the shed? On Aug 12, 2014 12:33 PM, "Brian LeRoux" <b...@brian.io> wrote:
> In another life I'd love to paint that shed. > > > > On Tue, Aug 12, 2014 at 12:25 PM, Jesse <purplecabb...@gmail.com> wrote: > > > Michal, exactly! > > Brian, hilarious, I was implying rewrite ALL the platform scripts in > nodejs > > ... as was recently done for cordova-windows. May be more work than it is > > worth though. > > > > @purplecabbage > > risingj.com > > > > > > On Tue, Aug 12, 2014 at 12:17 PM, Myles Borins <my...@famo.us> wrote: > > > > > Ahhhh, I forgot to shim my path with a sense of humor... Oops! > > > On Aug 12, 2014 12:16 PM, "Brian LeRoux" <b...@brian.io> wrote: > > > > > > > main benefit was humor though an argument could be made for other > ones > > ;) > > > > > > > > > > > > > > > > On Tue, Aug 12, 2014 at 12:06 PM, Myles Borins <my...@famo.us> > wrote: > > > > > > > > > Brian, would there be any benefit to using bash to call node rather > > > than > > > > a > > > > > script that started with #! /bin/env node? > > > > > On Aug 12, 2014 12:01 PM, "Brian LeRoux" <b...@brian.io> wrote: > > > > > > > > > > > OR BOTH > > > > > > > > > > > > #!/bin/sh > > > > > > node -e 'console.log(require("./package.json").version)' > > > > > > > > > > > > > > > > > > On Tue, Aug 12, 2014 at 11:52 AM, Jesse <purplecabb...@gmail.com > > > > > > wrote: > > > > > > > > > > > > > Yeah, if you are going to replace bash, replace it with nodejs! > > > > > > > > > > > > > > > > > > > > > @purplecabbage > > > > > > > risingj.com > > > > > > > > > > > > > > > > > > > > > On Tue, Aug 12, 2014 at 11:48 AM, Myles Borins <my...@famo.us> > > > > wrote: > > > > > > > > > > > > > > > Have you considered writing a small node script to pass the > > json? > > > > > This > > > > > > > > would make it as simple as requiring in the package json an > > > piping > > > > > the > > > > > > > > relevant info to stdout > > > > > > > > On Aug 12, 2014 11:47 AM, "Shazron" <shaz...@gmail.com> > wrote: > > > > > > > > > > > > > > > > > Yeah I value life and my sanity - I'll probably replace the > > > bash > > > > > > > > > script with python > > > > > > > > > > > > > > > > > > On Tue, Aug 12, 2014 at 11:40 AM, Lorin Beer < > > > > lorin.b...@gmail.com > > > > > > > > > > > > > > wrote: > > > > > > > > > > one source for version information is better > > > > > > > > > > > > > > > > > > > > although parsing json with bash scripts seems janky > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Tue, Aug 12, 2014 at 11:31 AM, Jesse < > > > > purplecabb...@gmail.com > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > >> I think it still needs to exist in an output project ... > > > which > > > > > is > > > > > > > not > > > > > > > > > >> (yet?) an npm project, and so does not have a > > package.json. > > > > > > > > > >> > > > > > > > > > >> The individual platform repos can get rid of it, they > will > > > > just > > > > > > need > > > > > > > > to > > > > > > > > > >> modify the way they `create` new projects to read the > > value > > > > from > > > > > > > > > >> package.json and output it to NewProject/VERSION > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > > >> @purplecabbage > > > > > > > > > >> risingj.com > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > > >> On Tue, Aug 12, 2014 at 11:25 AM, Shazron < > > > shaz...@gmail.com> > > > > > > > wrote: > > > > > > > > > >> > > > > > > > > > >> > For iOS, the only file I can see that depends on this > > is: > > > > > > > > > >> > > > > > > > > > > >> > 1. > > > > > > > > > >> > > > > > > > > > > >> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://github.com/apache/cordova-ios/blob/master/bin/templates/scripts/cordova/version > > > > > > > > > >> > > > > > > > > > > >> > Not sure of the alternative. > > > > > > > > > >> > > > > > > > > > > >> > This references it but can be removed: > > > > > > > > > >> > > > > > https://github.com/apache/cordova-ios/blob/master/bin/create > > > > > > > > > >> > > > > > > > > > > >> > > > > > > > > > > >> > On Tue, Aug 12, 2014 at 11:19 AM, Steven Gill < > > > > > > > > stevengil...@gmail.com > > > > > > > > > > > > > > > > > > > >> > wrote: > > > > > > > > > >> > > Most of our repos have a package.json. It keeps > track > > of > > > > > > > > versions. I > > > > > > > > > >> > think > > > > > > > > > >> > > we should work towards removing the VERSION files > from > > > the > > > > > > repos > > > > > > > > we > > > > > > > > > >> can. > > > > > > > > > >> > > > > > > > > > > > >> > > Thoughts? > > > > > > > > > >> > > > > > > > > > > > >> > > This would require some changes to coho so it > doesn't > > > try > > > > to > > > > > > > > update > > > > > > > > > the > > > > > > > > > >> > > version file when doing releases. > > > > > > > > > >> > > > > > > > > > > >> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >