To clear up my intent, I'm proposing 1. Keep the Plugman and Cordova/CLI as separate repos that we publish as discreet modules (and use npm / package.json to manage deps) 2. Create a new placeholder repo for staging common module extraction called cordova-lib 3. Publish many modules from this one git repo called cordova-lib and prefix any module from it with `cordova-lib` (for example cordova-lib-app-create would be a great module for sharing) 4. Evaluate if any modules can graduate from cordova-lib to more generally useful status and get their own git repos
Thoughts? On Wed, Apr 16, 2014 at 12:28 PM, Carlos Santana <csantan...@gmail.com>wrote: > Brian > yep I agree with directory "cordova-lib", "node_modules", "common". > "common-lib" > I think we are on the same page. > > What do you mean by "published"? in "-package.json (published as > cordova-lib-plugin-install)" > > no actually publishing to npm registry, but just having a convention for > the naming of the modules all starting with "cordova-lib-*" and matching > location within repo? > > { > "version": "0.0.1", > "name": "cordova-lib-plugin-install", > .. > } > > { > "version": "0.0.1", > "name": "cordova-lib-util-a", > .. > } > > > > > > On Wed, Apr 16, 2014 at 3:11 PM, Brian LeRoux <b...@brian.io> wrote: > > > I'm thinking a clean path might look something like this: > > > > plugman > > '-package.json -> cordova-lib-plugin-install > > > > cordova-cli > > '-package.json -> cordova-lib-plugin-install > > > > cordova-lib > > |-plugin-install > > | '-package.json (published as cordova-lib-plugin-install) > > etc > > > > Wherein all the 'meat' ends up in cordova-lib and plugman/cordova-cli > > become light CLI wrappers. I don't see any reason we change/remove the > > already extracted repos for the CLI and Plugman. > > > > > > > > > > On Wed, Apr 16, 2014 at 11:58 AM, Carlos Santana <csantan...@gmail.com > > >wrote: > > > > > I was going to suggest node_modules but I think it doesn't work for us > > > since we have two top level npm pacakges. If one top level npm pacakge > in > > > the repo then its fine. > > > > > > |cli > > > | '-package.json > > > | '-node_modules/util_a > > > |plugman > > > | '-package.json > > > | '-node_modules/util_a > > > > > > means "util_a" will be duplicated in repo > > > plugman/node_modules/util_a > > > cli/node_modules/util_a > > > > > > or > > > if you have > > > node_module/util_a at the root, npm link ../node_modules/util_a still > > needs > > > to be done for cli and plugman node modules. > > > > > > that's why I suggested to do the node_modules at dev/publish time to > > > populate the both node_modules one for cli and one for plugman > > > > > > Or maybe I missed something. > > > > > > The tag for smaller modules, might be tricky but at the same time not > > > necessary if they are consider bundle/private and living in same repo > > > > > > Thanks Brian for putting the question out there on twitter interesting > > > feedback. > > > > > > --Carlos > > > > > > > > > On Wed, Apr 16, 2014 at 2:10 PM, Brian LeRoux <b...@brian.io> wrote: > > > > > > > I thought the node_modules comment might have been cheeky. > (Suggesting > > we > > > > use npm to manage deps.) > > > > > > > > Crap. Totally forgot about Good Friday. I have a one hour window open > > on > > > > Thu. =( > > > > > > > > > > > > On Wed, Apr 16, 2014 at 10:51 AM, Mark Koudritsky <kam...@google.com > > > > > > wrote: > > > > > > > > > The tip about placing the deps under node_modules right away sounds > > > very > > > > > useful. This way the dev environment will be ready right after git > > > clone; > > > > > npm install with no extra magic. > > > > > > > > > > This Friday is a holiday in Canada (Good Friday). > > > > > > > > > > > > > > > On Wed, Apr 16, 2014 at 1:45 PM, Steven Gill < > stevengil...@gmail.com > > > > > > > > wrote: > > > > > > > > > > > Git tags are not something we have talked about yet. Thanks for > > > sharing > > > > > > Brian! > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Wed, Apr 16, 2014 at 10:39 AM, Brian LeRoux <b...@brian.io> > wrote: > > > > > > > > > > > > > hey guys could we add Fri to that doodle? > > > > > > > > > > > > > > > > > > > > > I asked around for opinions and got some interesting responses > to > > > add > > > > > to > > > > > > > the discussion: > > > > > > > > > > > > > > https://twitter.com/brianleroux/status/456242928298381312 > > > > > > > > > > > > > > (Nothing really negative or positive.) > > > > > > > > > > > > > > > > > > > > > On Apr 15, 2014 6:34 PM, "Andrew Grieve" <agri...@chromium.org > > > > > > wrote: > > > > > > > > > > > > > > > We didn't have time to cover this today, but I think it's > > better > > > > off > > > > > > > > as its own hangout anyways. > > > > > > > > > > > > > > > > Created a doodle for Wed/Thurs. Add yourself if you'd like to > > > > > > > > participate in the discussion. > > > > > > > > > > > > > > > > http://doodle.com/uvyr9454pvepz3a3 > > > > > > > > > > > > > > > > On Tue, Apr 15, 2014 at 11:19 AM, Mark Koudritsky < > > > > kam...@google.com > > > > > > > > > > > > > > wrote: > > > > > > > > > A short proposal< > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://docs.google.com/document/d/1GVtG6BD266dqRURKaS-GEDefb0tBYt56acxrJEKAfmE/edit > > > > > > > > > > > > > > > > > > for > > > > > > > > > how to rearrange cli and plugman code. Let's discuss it > > during > > > > the > > > > > > > > hangout > > > > > > > > > today. Feel free to comment. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > Carlos Santana > > > <csantan...@gmail.com> > > > > > > > > > -- > Carlos Santana > <csantan...@gmail.com> >