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

