--On Monday, November 28, 2005 12:34 AM -1000 "(Cedric) Qin ZHANG"
<[EMAIL PROTECTED]> wrote:
Given that we want people to be able to run all unit test cases after
downloading the distribution zip file, we need to either (1) continue our
current practice, or (2) move all non-server side class files to some
other location under hackyCore_Build/build.
Before any decision is made, I'll stick to status-quo by putting
telemetry control center class files in WEB-INF/classes.
Yes, it does seem strange to have both client-side and server-side classes
in web-inf/classes, but as you note, those classes would have to be
somewhere in the distribution in order to allow unit testing of the entire
system after download (which I think is a very good thing).
We could put them someplace else, but this would add a bit of complexity to
the build and distribution and I'm not sure it would add proportional
value. What we should do at least for starters is document in our build
system documentation that web-inf/classes contains all the code, and why we
do it that way.
If someone can think of a reason why our current approach creates bugs (as
opposed to just being idiosyncratic), please let us know, and that might
warrant the change (after 7.0).
Cheers,
Philip
