On 28 Nov 2011, at 14:34, Alexander Urmuzov wrote:

> Thank you!
> 
> I tried, but have some problems:
> 1) Compilation of multibindings and grapher module fails

There's a failing test (MultibinderTest.testSetAndMapValueConflict) for a bug 
reported on the mailing list:

   
http://groups.google.com/group/google-guice/browse_thread/thread/934e30b7ed0da98f/b90383ad4cd1228c

I couldn't see an issue for this so I created one just now: 
http://code.google.com/p/google-guice/issues/detail?id=670

The AbstractInjectorGrapherTest class has an annotation 
(@com.google.testing.testsize.MediumTest) that's not available from any 
dependency - this annotation should probably be removed.

> 2) Some tests fails. In guice-servlet extension too. I turned tests off.

These failures appear to be because the servlet tests now use the Throwables 
class - Guice core doesn't use Throwables, so it gets removed by jarjar when it 
embeds Guava inside Guice

Either we need to include Throwables in core (by altering the jarjar 
instructions) or we need to fix the tests so they don't depend on Throwables

( note that in the experimental sisu-guice branch we don't embed Guava anymore, 
which is another solution 
http://groups.google.com/group/google-guice/browse_thread/thread/d146d38116d00333
 )

> 3) When I attach compiled atrifacts i've got this on startup:
> java.lang.NoClassDefFoundError: com/google/inject/internal/util/$Maps
>         at 
> com.google.inject.assistedinject.BindingCollector.<init>(BindingCollector.java:34)
>         at 
> com.google.inject.assistedinject.FactoryModuleBuilder.<init>(FactoryModuleBuilder.java:206)

It sounds like you might be mixing up core + extensions built from trunk with 
older extensions - due to a change to the jarjar instructions in trunk, moving 
embedded guava classes from internal/... to internal/guava/..., you need to 
make sure all extensions are from trunk (ie. all at the same version).

FYI, you can use "mvn clean install -Dmaven.test.skip" to build the complete 
tree without hitting the test failures

> Is there any snapshot repository where I can get most recent artifacts?

We host a CI build at Sonatype that deploys snapshots to:

   https://repository.sonatype.org/content/groups/forge/com/google/inject/

but it's a bit stale because of the failing tests - I'll do a re-spin with them 
disabled to refresh the snapshots

> --
> С уважением, Урмузов Александр Алесандрович
> 
> 
> 
> 2011/11/28 Sam Berlin <[email protected]>:
> > The latest Git head has some fixes from Isaac that should allow subsequent
> > filters in the chain to be injected with the supplied request/response.
> >
> > sam
> >
> > On Nov 27, 2011 9:31 PM, "Alexander Urmuzov" <[email protected]> wrote:
> >>
> >> I've got a filter that wraps HttpServletResponse like this:
> >>         HttpServletRequest httpRequest = (HttpServletRequest)
> >> servletRequest;
> >>         HttpServletResponse httpResponse = (HttpServletResponse)
> >> servletResponse;
> >>         ...
> >>         OutputStream out = httpResponse.getOutputStream();
> >>         GenericResponseWrapper wrapper = new
> >> GenericResponseWrapper(httpResponse);
> >>         filterChain.doFilter(servletRequest, wrapper);
> >>         
> >>         String content = new String(wrapper.getData(),
> >> httpResponse.getCharacterEncoding());
> >>         String designedContent = design.process(content, features);
> >>         out.write(designedContent.getBytes());
> >>         out.close();
> >> The problem comes when I try to inject Provider<HttpServletResponse>.
> >> Injected HttpServletResponse is not an instance of GenericResponseWrapper,
> >> it is a original HttpServletResponse. So it breaks my application.
> >> Maybe it is a correct behavior, but i need to override HttpServletResponse
> >> binding. How can I achieve it?
> >>
> >> --
> >> You received this message because you are subscribed to the Google Groups
> >> "google-guice" group.
> >> To view this discussion on the web visit
> >> https://groups.google.com/d/msg/google-guice/-/rmiLj9x_ws0J.
> >> To post to this group, send email to [email protected].
> >> To unsubscribe from this group, send email to
> >> [email protected].
> >> For more options, visit this group at
> >> http://groups.google.com/group/google-guice?hl=en.
> >
> > --
> > You received this message because you are subscribed to the Google Groups
> > "google-guice" group.
> > To post to this group, send email to [email protected].
> > To unsubscribe from this group, send email to
> > [email protected].
> > For more options, visit this group at
> > http://groups.google.com/group/google-guice?hl=en.
> >
> 
> 
> -- 
> You received this message because you are subscribed to the Google Groups 
> "google-guice" group.
> To post to this group, send email to [email protected].
> To unsubscribe from this group, send email to 
> [email protected].
> For more options, visit this group at 
> http://groups.google.com/group/google-guice?hl=en.

-- 
You received this message because you are subscribed to the Google Groups 
"google-guice" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/google-guice?hl=en.

Reply via email to