[forge-dev] Fwd: Re: [core] FORGE-874: Proxy settings not used for maven web project (#314)
George Gastaldi
ggastald at redhat.com
Tue Apr 30 08:42:09 EDT 2013
+1, there's a package named "observers" in the shell project (https://github.com/forge/core/tree/master/shell/src/main/java/org/jboss/forge/shell/observers)
Just add this observer there.
Em 30/04/2013, às 09:32, "Lincoln Baxter, III" <lincolnbaxter at gmail.com> escreveu:
> Thoughts?
>
> ---
> Lincoln Baxter's Droid
> http://ocpsoft.org
> "Keep it Simple"
>
> ---------- Forwarded message ----------
> From: "Lincoln Baxter, III" <lincolnbaxter at gmail.com>
> Date: Apr 30, 2013 8:31 AM
> Subject: Re: [core] FORGE-874: Proxy settings not used for maven web project (#314)
> To: "forge/core" <reply+i-13796837-9f0837d6c9164cfde977b60d4c0f19df2d4d9f6b-362329 at reply.github.com>
> Cc: "forge/core" <core at noreply.github.com>
>
> Now that I see what you did to fix this, I think this makes sense to do earlier in the forge life cycle. I don't see why it should be tied either to projects or to plugins, do you?
>
> This should probably go into an @Observes PostStartup event method. Somewhere in the core.
>
> Wdyt?
>
> ---
> Lincoln Baxter's Droid
> http://ocpsoft.org
> "Keep it Simple"
>
> On Apr 30, 2013 4:33 AM, "Yann Albou" <notifications at github.com> wrote:
>>
>> You can merge this Pull Request by running
>>
>> git pull https://github.com/myannou/forge-core FORGE-874
>> Or view, comment on, or merge it at:
>>
>> https://github.com/forge/core/pull/314
>>
>> Commit Summary
>>
>> FORGE-874: Proxy settings not used for maven web project
>> File Changes
>>
>> M shell/src/main/java/org/jboss/forge/shell/plugins/builtin/ForgePlugin.java (21)
>> M shell/src/main/java/org/jboss/forge/shell/project/ProjectInitializer.java (26)
>> Patch Links:
>>
>> https://github.com/forge/core/pull/314.patch
>> https://github.com/forge/core/pull/314.diff
> _______________________________________________
> forge-dev mailing list
> forge-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/forge-dev
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/forge-dev/attachments/20130430/ef7a8436/attachment.html
More information about the forge-dev
mailing list