[Building Sakai] WHAT HAPPENED TO SERVER-CONFIGURATION-SERVICE ?!?

[Jim Eng]

Aaron, and all:

Yes, I am shouting!

It looks like two checkins were made to ServerConfigurationService recently, but I cannot tell what has changed because the entire contents of the interface was replaced though it looks like many of the methods did not change at all.  That means all history for that file is gone. Or maybe I should say useless.  We can no longer do a simple svn command to find out when a particular line of code changed and why. It looks like every single line of code except for the interface name changed in rev 98010.  

http://source.sakaiproject.org/viewsvn/kernel/trunk/component-manager/src/main/java/org/sakaiproject/component/api/ServerConfigurationService.java?r1=66281&r2=98010&pathrev=98098

Then a few more changes were added in rev rev 98098.  

http://source.sakaiproject.org/viewsvn/kernel/trunk/component-manager/src/main/java/org/sakaiproject/component/api/ServerConfigurationService.java?r1=98010&r2=98098&pathrev=98098

I looked this up because I am trying to update a mock in a contrib project, and someone has added inner interfaces to ServerConfigurationService, which I am having difficulty dealing with.  I thought we learned the problems with inner interfaces a long time ago, and I seem to remember Aaron Z complaining about the problems they created when implementing mocks. 

I think revs 98010 and 98098 should be backed out until they are cleaned up.  The code should NOT be reformatted in this way that loses all of the history and masks the actual changes that are being made.  The inner interfaces should be moved out so they are separate interfaces. etc.

Please.

Jim