[+luke]

Luke, do you think it'd be possible to allow overriding (maybe hidden
behind a flag and also printing out a warning to the console)?  Maybe just
if the return types are exactly the same?

sam

On Mon, Sep 29, 2014 at 1:47 PM, Roger Kapsi <[email protected]> wrote:

> Hey,
>
> one of our projects is fiddling with Dropwizard and Jersey 1.18.1. One of
> the things we need to do to make things work is overwrite the
> JerseyServletModule's webApp() provider method. It appears Guice 4.0 beta5
> is introducing a new behavior that breaks the ability to overwrite provider
> methods.
>
> 1) Overriding @Provides methods is not allowed.
> @Provides method: com.sun.jersey.guice.JerseyServletModule.webApp()
> overridden by: com.squarespace.dropwizard.guice.GuiceBundleModule.webApp()
> at
> com.google.inject.internal.ProviderMethodsModule.getProviderMethods(ProviderMethodsModule.java:128)
>
>
> On Thursday, September 25, 2014 10:51:04 AM UTC-4, Sam Berlin wrote:
>>
>> Yesterday we pushed out a 4.0 beta5
>> <https://github.com/google/guice/wiki/Guice40> release for Guice.  It's
>> available on maven central as usual.
>>
>> Some notable changes since the last beta
>> <https://github.com/google/guice/compare/4.0-beta4...master> include:
>>   * Better java8 compatibility.
>>   * Various optimizations to speed things up & reduce allocations.
>>   * Added OptionalBinder
>> <http://google.github.io/guice/api-docs/latest/javadoc/com/google/inject/multibindings/OptionalBinder.html>
>> for better optional bindings or bindings with default values.
>>   * Allow Keys from child injectors to be GC'd when the child injector
>> gets GC'd.
>>   * Added a "testlib" extension with a new 'BoundFieldModule
>> <http://google.github.io/guice/api-docs/latest/javadoc/com/google/inject/testing/fieldbinder/BoundFieldModule.html>'
>> to support easier binding in tests.
>>
>> For changes since the last 3.0 release, see the commits
>> <https://github.com/google/guice/compare/3.0...master> & API diffs
>> <http://google.github.io/guice/api-docs/latest/api-diffs/changes.html>.
>>
>> Special thanks to Colin Decker for building/staging the release!
>>
>> sam
>>
>  --
> You received this message because you are subscribed to the Google Groups
> "google-guice" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to [email protected].
> To post to this group, send email to [email protected].
> Visit this group at http://groups.google.com/group/google-guice.
> To view this discussion on the web visit
> https://groups.google.com/d/msgid/google-guice/f3e636d4-286e-4ddc-bcdb-1c9ba7b37827%40googlegroups.com
> <https://groups.google.com/d/msgid/google-guice/f3e636d4-286e-4ddc-bcdb-1c9ba7b37827%40googlegroups.com?utm_medium=email&utm_source=footer>
> .
> For more options, visit https://groups.google.com/d/optout.
>

-- 
You received this message because you are subscribed to the Google Groups 
"google-guice" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/google-guice.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/google-guice/CAJEBNUfn%3DNbJ6uc%3D%3Du5A5Aa0Uf_59w0DtLqP79uB3TfrmmuMuA%40mail.gmail.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to