Actually, it's still broken.

The original patch was applied correctly, but
SampleContainerHandlerProvider.java was never reinstated during
application of shindig-638-fix.patch.

Thanks,
Dave

-----Original Message-----
From: Ian Boston [mailto:[EMAIL PROTECTED] On Behalf Of Ian
Boston
Sent: Thursday, October 02, 2008 4:29 PM
To: [email protected]
Subject: Re: svn commit: r701087 - in /incubator/shindig/trunk/java:
server/src/test/java/org/apache/shindig/server/endtoend/
social-api/src/main/java/org/apache/shindig/social/core/config/
social-api/src/main/java/org/apache/shindig/social/opensocial/serviceLin
k 

Looks like Adam got there before me.
Thanks Adam. (for fixing my mistake:))

Ian
On 2 Oct 2008, at 21:19, Citron, David wrote:

> Maybe I'm confused, but this patch fix seems to have broken my build.
>
> The patch refers to two files:
>
> 1)
> java/social-api/src/main/java/org/apache/shindig/social/core/config/ 
> Soci
> alApiGuiceModule.java
>
> 2)
> java/social-api/src/main/java/org/apache/shindig/social/sample/ 
> service/S
> ampleContainerHandlerProvider.java
>
> File "2" was deleted with a previous patch (701087) and as such  
> does not
> exist anymore, yet the patch fix changes file "1" to refer to class  
> "2",
> hence the build error:
>
> C:/dev/shindig/java/social-api/src/main/java/org/apache/shindig/ 
> social/c
> ore/config/SocialApiGuiceModule.java:[31,48] cannot find symbol
> symbol  : class SampleContainerHandlerProvider
> location: package org.apache.shindig.social.sample.service
>
> Is something messed up in my configuration?
>
> Thanks,
> Dave

Reply via email to