Mailing-List: contact dev-help@cordova.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cordova.apache.org
Received-SPF: pass (athena.apache.org: domain of benn.mapes@gmail.com
 designates 209.85.192.177 as permitted sender)
MIME-Version: 1.0
In-Reply-To: 
 <CAP7NMPqqg1Y=K8ZLH6rzxQvwD5YfWM=i3b-_ma+Cuc4q187J-w@mail.gmail.com>
References: 
 <CAJOjTChyhTjm17Bh4ErZkBZxSB=RVS6qX-tVVXDkK7bq9gMhww@mail.gmail.com>
	<CDD285F1.1FEF0%fil@adobe.com>
	<CAMk5LLnCqzA=LgqOdNSTMjT+G9Wh1s+TzBPnCeXqTp-s3_0AJw@mail.gmail.com>
	<CAP7NMPqqg1Y=K8ZLH6rzxQvwD5YfWM=i3b-_ma+Cuc4q187J-w@mail.gmail.com>
Date: Tue, 4 Jun 2013 12:04:53 -0700
Message-ID: 
 <CAJOjTCiXg6K2VsC==aZEACb_w7NYeWGGU9mWJ-rKpb1_4ZBrTw@mail.gmail.com>
Subject: Re: [cli] Consolidating platform references
From: Benn Mapes <benn.mapes@gmail.com>
To: dev@cordova.apache.org
Content-Type: multipart/alternative; boundary=e89a8ffbab6b9ab11204de58c104

--e89a8ffbab6b9ab11204de58c104
Content-Type: text/plain; charset=ISO-8859-1

Allright,  I pushed the changes up to master2
https://git-wip-us.apache.org/repos/asf?p=cordova-cli.git;a=commit;h=796ad93a062d5e963d947f4c740a7c330184b49e


On Tue, Jun 4, 2013 at 9:32 AM, Michael Brooks <michael@michaelbrooks.ca>wrote:

> Great work Benn. Braden echoed exactly my thoughts.
>
>
> On Tue, Jun 4, 2013 at 7:47 AM, Braden Shepherdson <braden@chromium.org
> >wrote:
>
> > I've wanted to do this since I first opened the files in CLI. It's much
> > better these days than it was, at least. Glad to see it's still moving.
> >
> > Braden
> >
> >
> > On Mon, Jun 3, 2013 at 8:42 PM, Filip Maj <fil@adobe.com> wrote:
> >
> > > Works well for me on my machine, +1
> > >
> > > On 6/3/13 5:40 PM, "Benn Mapes" <benn.mapes@gmail.com> wrote:
> > >
> > > >Working with the cordova-cli code I have noticed that there are lots
> of
> > > >repetitive platform-specific references in many of the files. I have
> > tried
> > > >to consolidate all of these into the platforms.js file at the root of
> > the
> > > >project. I push up this branch to apache:
> > > >
> > >
> >
> https://git-wip-us.apache.org/repos/asf?p=cordova-cli.git;a=shortlog;h=ref
> > > >s/heads/platform-reorg
> > > >
> > > >As far as I know, all the test pass on windows and mac, I haven't
> > > >physically tested every part yet to make sure this didn't break
> anything
> > > >but I believe not.
> > > >
> > > >This change is working towards consolidating all platforms so that
> it's
> > > >very easy to add and remove them if needed. The ultimate goal for
> > adding a
> > > >new platform should be to just add it to platforms.js and create a
> > parser
> > > >+
> > > >tests for it.
> > > >
> > > >Please let me know what you think, if everyone is in agreeance hen I
> > will
> > > >add this to the master2 branch.
> > > >
> > > >~Benn
> > >
> > >
> >
>

--e89a8ffbab6b9ab11204de58c104--