Wendy Smoak wrote:
There were no objections to your proposed reorganization, so IMO the
thing to do is try it and see what happens.  Preferably in the test
repo. :)  We don't need history for this, so svn export/svn import
should do it.  (https://svn.apache.org/repos/test)  If not, maybe Sean
can help; he did the same thing for MyFaces before they switched to
Maven 2.

Ok, you asked for it... :) I put my first cut at https://svn.apache.org/repos/test/action1 I moved the subprojects under action1, action as core, and adjusted the builds so that they all shared the same version. I'm having trouble getting the nightly to run, as it seems the default built isn't installing the snapshot jars correctly. What am I missing here?

Minor issues:
 * I think Struts EL and Taglib should stay in separate jars because
you only need Struts EL in a Servlet 2.3 environment.  (Further, I
thought there was some technical reason they could not be combined.)

Well, our sys requirement is servlet 2.3 now. I haven't tried this yet, so there may be a technical reason it can't be done. Well, thinking about it, if both were part of action 1, and according to maven, each subproject can have only one artifact, I guess we should leave them.

 * Faces doesn't compile against the latest code.  It's complaining
about RequestUtils.selectModule(...).  Right now, it depends on Struts
1.2.2.  Also, its
Maven 1 build is not finished.  It's probably best to leave it separate for now.

No problem, I can fix this.  I've been going through the code as I might want 
to port something similar for Action 2.

I can't take on the reorganization project, but I have time to help
with the build if you need it.  I made some comments in the proposal
thread about removing the svn:externals, etc.

Ok, I'll look that up again.  Thanks for the help,

Don


And, thanks. :)

--
Wendy

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]




---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to