Martin, Point taken on the module naming: "integration" is better.
Can you your focus on the "baggage"? With maven 2, dependencies can be marked as optional. They all have to be there when compiling the source, but not when depending on the library. Paul On 4/21/07, Martin Cooper <[EMAIL PROTECTED]> wrote:
First off, calling this "modules" would be very confusing, since Struts 1 already has a concept of modules that is not at all related to what you are looking to create. Since the purpose is integration with other libraries, I would suggest "integration". Second, I would strongly recommend against creating one all-encompassing jar file. The problem with that approach is that almost every user will be carrying around baggage that they don't want or need. If we provide integration with a half dozen other libraries, the chances are that any given developer will need at most one of them. That leaves 5/6 of the jar as baggage. I like Wendy's suggestion - start with one struts-spring jar and see how that works out. -- Martin Cooper On 4/21/07, Paul Benedict <[EMAIL PROTECTED]> wrote: > > Wendy, I was thinking of the first. > > On 4/21/07, Wendy Smoak <[EMAIL PROTECTED]> wrote: > > > > On 4/21/07, Paul Benedict <[EMAIL PROTECTED]> wrote: > > > > > I am proposing a new module for Struts 1.x called "modules" which > would > > > contain integration classes into popular open source libraries. For > > > starters, it would contain a new command to let Spring wire up the CRP > > > using a command. > > > > I would suggest adding one new module called struts-spring, and see > > how things go from there. > > > > (It's not clear to me whether you want to add a jar called > > "struts-modules" which would have integration code with Spring and > > other third-party libraries, or whether "modules" is intended to be a > > directory structure like "plugins" in the s2 build.) > > > > -- > > Wendy > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > >