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 agrieve@google.com designates
 209.85.219.45 as permitted sender)
MIME-Version: 1.0
Sender: agrieve@google.com
In-Reply-To: 
 <0768740a4c274b4c96053445252a0e44@BL2PR03MB417.namprd03.prod.outlook.com>
References: 
 <bc419e3fcf144715a9284b3d5b47305b@BL2PR03MB417.namprd03.prod.outlook.com>
 <CAKP6k2u+6R6VvWG0U=1_NXJ-tf6qtx6FAHw68V18y8mVLHFwsg@mail.gmail.com>
 <CABiQX1WnoVvjj9FObL3tgTKiD6QFL6qhdex+CdHuaKMs94Q46A@mail.gmail.com>
 <967c98a06b7c4227b93cc932be17f986@BLUPR03MB359.namprd03.prod.outlook.com>
 <E0F2DA99-1D6A-4976-9A63-1C59861B4A3A@gmail.com>
 <f50af87165204baebfb695736ac75bfc@BN1PR07MB135.namprd07.prod.outlook.com>
 <CABiQX1Xdrk4ki1Mt6EHBytEQNE-4uAJvu57we8sRqTzvj2usOw@mail.gmail.com>
 <CAKP6k2tvhV3JLO472Efd5wvz_3qnzHU24rq1x7xzRdYUvyhLMg@mail.gmail.com>
 <e9e80d1773d14a3b83be5d6be96724ad@BN1PR07MB135.namprd07.prod.outlook.com>
 <CAKP6k2uSp_ot6ACX97+sX81OHLvoOP-_P+TrEQmWvJ2vH05++A@mail.gmail.com>
 <CAEeF2TeJY_tLLBtsX833adO4v+jpVxR-ud1tCcGCh-ueC3JZBQ@mail.gmail.com>
 <382e88be07544c9a8d3988310f89cb39@BN1PR07MB135.namprd07.prod.outlook.com>
 <CAEeF2TfmSXea1FLc4X2AMcHBtNLstyqKvMWPFWrRSajnJU=dxg@mail.gmail.com>
 <336209849c2043928a22188eef9c3447@BN1PR07MB135.namprd07.prod.outlook.com>
 <2806162a1ff24783b5d588a8f4800131@BL2PR03MB417.namprd03.prod.outlook.com>
 <d78cbfd136174dc1ae3dbec7623951da@BLUPR07MB129.namprd07.prod.outlook.com>
 <4a00e18c98af4daaa6788f695932327d@BL2PR03MB417.namprd03.prod.outlook.com>
 <8a9fa4dbe24c4ebe9f8b4e2302d43e2b@DM2PR03MB366.namprd03.prod.outlook.com>
 <CAE6O_-a2Les_JM_skTWOKgQhtcP4GMx8bfa+sKh9JbWSoxTKPw@mail.gmail.com>
 <CADVgkyMA5JhCcjbW6wqEr5g-7pXAn2QNqAQg-+dVCmOknLtdsg@mail.gmail.com>
 <a761948cab184346ba541df58c31b0dc@BL2PR03MB417.namprd03.prod.outlook.com>
 <CADVgkyN-ArZCocD8Er47n8j5psjhVESR9e7Xm3p+JwW_BOo3NA@mail.gmail.com>
 <CAE6O_-aifuyAsJi-Yg2VKWADE2rPzJ3e1Zn6in8TyGd2HO0Mvg@mail.gmail.com>
 <CAE6O_-Y16krj26sqXTDeHvArPerCkYWmmezu+5c4S1farjWScQ@mail.gmail.com>
 <157754a15285493cac3efe11ee9b52ae@DM2PR03MB366.namprd03.prod.outlook.com>
 <6fb754918e77424c88118529f2ad940e@BL2PR03MB417.namprd03.prod.outlook.com>
 <e32fc07c26494a74aab70a0d51ebba50@BN1PR07MB135.namprd07.prod.outlook.com>
 <0768740a4c274b4c96053445252a0e44@BL2PR03MB417.namprd03.prod.outlook.com>
From: Andrew Grieve <agrieve@chromium.org>
Date: Tue, 22 Apr 2014 12:04:57 -0400
Message-ID: 
 <CABiQX1VRDuQps2WyHBF6fGxEydA+YWoCWRpZEU+bMqPeqWadsg@mail.gmail.com>
Subject: Re: Proposal: hooks support for plugins
To: dev <dev@cordova.apache.org>
Content-Type: multipart/alternative; boundary=089e011764e935234504f7a3c8d7

--089e011764e935234504f7a3c8d7
Content-Type: text/plain; charset=UTF-8

There are some *disadvantages* to not sub-shelling out for hooks:
- Harder to capture their stdio (certainly do-able though by swapping out
system.std* for the duration of the hook)
- Harder to handle script failures (e.g. if they throw an uncaught
exception, we would like to be able to say "This hook script failed:
foo.js")
  - Maybe this is doable, by storing a global exception-was-thrown callback?
- Gives hooks the ability to mess up cordova's environment (although maybe
the vm thing addresses this?)

Would like to see tests for these things added before we launch this
feature.


On Tue, Apr 22, 2014 at 10:40 AM, Sergey Grebnov (Akvelon) <
v-segreb@microsoft.com> wrote:

> +1, I will name it as 'context'
>
> Thx!
> Sergey
> -----Original Message-----
> From: Jonathan Bond-Caron [mailto:jbondc@gdesolutions.com]
> Sent: Tuesday, April 22, 2014 7:34 AM
> To: dev@cordova.apache.org
> Subject: RE: Proposal: hooks support for plugins
>
> On Mon Apr 21 03:39 PM, Sergey Grebnov (Akvelon) wrote:
> > module.exports = function(platform, projectDir, pluginDir, cmdLine) {
> >     console.log('hook.js: ' + platform);
> >     console.log('hook.js: ' + projectDir);
> >     console.log('hook.js: ' + pluginDir);
> >     console.log('hook.js: ' + cmdLine);
> >
>
> Personnaly prefer:
>
> > module.exports = function(hookApi) {
> >     console.log('hook.js: ' + hookApi.platform);
> >     console.log('hook.js: ' + hookApi.projectDir);
> >     console.log('hook.js: ' + hookApi.pluginDir);
> >     console.log('hook.js: ' + hookApi.cmdLine);}
>
> Make it easier to pass other stuff in the future & using a sandoxed
> hookApi object.
>
>
>

--089e011764e935234504f7a3c8d7--