Hi there...sorry for the delay in activity, I was out of town for a week or
so and I am still catching up on email and trying to upgrade from Eclipse
3.1 to Eclipse 3.1 (w/WTP 0.7RC2) and in doing so I decided to dump my
current .classpath and .project info from my prior version and try to pick
up a new config via 'Create Project'.
I am noticing some strange things...
Build
-----
o In build/build.xml, the default target is 'dist-all' - was that
intentional?
o Should the clean or clean-all target remove the items retrieved via
download-dependencies? Or is there a different one for that? (like
'clean-lib' or 'clean-all-lib')
ASL 2.0
-------
o I noticed that not every .java file has the standard (and required)
license. Perhaps we should do an audit of the code to ensure that we are in
compliance.
Duplicate class
---------------
o We define o.a.m.e.s.SourceCodeServlet.java twice. Once in
examples/sandbox, and once in examples/simple.
Given the fact that this is a simple utility class and these are both exact
duplicates (line for line - including missing ASL 2.0) makes me inclined to
think that this class should probably be moved to a single location and
referenced from there. My thoughts are to make a "shared.jar" or
"examples-shared.jar" that is itself a subproject under examples (part of
the examples subproject), but used in both examples/sandbox and
examples/simple. Come to think of it, I'm not sure why it is not needed in
tiles and wml.
Your thoughts?
tomahawk/src/wml
----------------
o Eclipse always complains about missing wml sources which get generated via
xdoclet during the ant build, or so they used to....perhaps I'm missing
something, but did this commit remove the xdoclet wml source generation?
http://mail-archives.apache.org/mod_mbox/myfaces-commits/200506.mbox/[EMAIL
PROTECTED]
...thanks for your help.
--
James Mitchell
Software Engineer / Open Source Evangelist
Consulting / Mentoring / Freelance
EdgeTech, Inc.
http://www.edgetechservices.net/
678.910.8017
AIM: jmitchtx
MSN: [EMAIL PROTECTED]
Skype: jmitchtx
