cordova-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Brian LeRoux...@brian.io>
Subject Re: cordova-lib split out
Date Fri, 02 May 2014 21:11:50 GMT
1. node_modules is a bad idea and yep it was a joke
2. are we looking to break out into something wherein the cordova-lib is a
dumb directory w/ modules inside it still? (unclear in current structure)


On Fri, May 2, 2014 at 12:13 PM, Michal Mocny <mmocny@chromium.org> wrote:

> On Fri, May 2, 2014 at 2:43 PM, Mark Koudritsky <kamrik@google.com> wrote:
>
> > Instructions for checking history in cordova-lib:
> >
> > git log <file> will show no history from before this commit for files
> that
> > moved. To see full history use either
> > git log --follow <file>
> > or
> > git log cli_pre_lib <old file name>
> > git log plugman_pre_lib <old file name>
> >
> > 'cli_pre_lib' and 'plugman_pre_lib' are tags for the last commit in
> > cordova-cli and plugman repos made before cordova-lib was split out.
> >
> > If you just do "git log <file>" on a file, the earliest visible commit
> will
> > contain the same suggestions as above, but it will also tell you which
> repo
> > this file lived in.
> >
>
> Sweet, that was a good idea.
>
>
> >
> >
> >
> > On Fri, May 2, 2014 at 2:34 PM, Mark Koudritsky <kamrik@google.com>
> wrote:
> >
> > > Hi All,
> > >
> > > Cordova-lib is here. And since there is no way to do it gradually
> without
> > > eternally chasing changes between repos, I'm landing it on master in a
> > big
> > > heavy chunk (as well as on branch named cordova-lib).
> > >
> > > Cordova lib is here:
> > > https://git-wip-us.apache.org/repos/asf/cordova-lib.git
> > >
> > > when cloning cordova lib please make sure to run
> > >    npm link in cordova-lib
> > >    and
> > >    npm link cordova-lib in cordova-cli and cordova-plugman repos
> > >
> > > The new code structure is based on what was discussed during the
> hangout
> > > with minor changes and is as follows:
> > >
> > > cordova-cli.git/
> > >
> > >    -
> > >
> > >    package.json
> > >    -
> > >
> > >    bin/cordova[.cmd]
> > >    -
> > >
> > >    src/cli.js
> > >    -
> > >
> > >    doc/help.txt
> > >    -
> > >
> > >    spec/cli.spec.js
> > >    -
> > >
> > >    cordova.js - a proxy for cordova-lib/src/cordova/cordova.js below,
> for
> > >    compatibility, will go away later, when downstream tools start using
> > >    crdova-lib.
> > >
> > >
> > >
> > > cordova-plugman repo has a structure very similar to cordova-cli.
> > >
> > > cordova-lib.git/
> > >
> > >    -
> > >
> > >    cordova-lib/
> > >    -
> > >
> > >       cordova-lib.js - The future Cordova API surface. For now only
> > >       contains proxy shims to cordova.js and plugman.js
> > >       -
> > >
> > >       src/cordova/<almost all of cordova-cli code>
> > >       -
> > >
> > >       src/plugman/<almost all of plugman code>
> > >       -
> > >
> > >       src/<common code> (for now only CordovaError)
> > >       -
> > >
> > >       templates/*
> > >       -
> > >
> > >       spec-plugman/
> > >       -
> > >
> > >       spec-cordva/
> > >       -
> > >
> > >          Specs are not in the same dir because they interact with each
> > >          other. This will be solved later, then they can be combined
> > into a single
> > >          spec dir.
> > >          -
> > >
> > >    cordova-future-moduleX/ - will live as sibling of cordova-lib/. We
> can
> > >    later move them to live under cordova-lib/node_modules/ if npm
> > linkwork becomes a burden for dev workflow.
> > >
> > >
> > >
> > > Some points:
> > >
> > >  - code that does require('cordova/something') will break.
> > >
> > >  - README files will need some attention.
> > >
> > >  - npm-shrinkwrap.json files were deleted and will need to be
> re-created
> > > again.
> > >
> >
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message