On Thu, 2 Dec 2004 14:03:45 -0600, Joe Germuska <[EMAIL PROTECTED]> wrote: > At 2:32 PM -0500 12/2/04, Deadman, Hal wrote: > >Are you using Eclipse? If so you can set Eclipse up to ignore a certain > >pattern (that package) in a particular source directory. > > > >It would be nice if the dependent jars and .project and .classpath files > >were checked into SVN the way spring does it. Then someone could import > >team project set and have all the various sub-projects automatically > >checked out and setup as projects in Eclipse. > > I feel strongly that JARs don't belong in a VCS system. As long as > we maintain a Maven project.xml file, anyone can create their > .classpath file with "maven eclipse". That's not that much work, it > avoids cluttering the repository with potentially large binary files, > and it keeps you from running into any licensing issues if you happen > to depend on a JAR you can't keep in CVS.
Other issues with keeping Eclipse files in our repo: 1) The expectation would be that they would be kept up to date. If a particular committer doesn't use Eclipse, I don't think it's fair to expect them to keep Eclipse config files up to date when they make changes elsewhere. 2) This would open the door for just about any other tool's config files to be kept in our repo. If it's OK for Eclipse, why not IDEA, netBeans, JBuilder, JDeveloper, emacs, ...? How would we keep that mess up to date? -- Martin Cooper > Joe > > > > > > > -----Original Message----- > >> From: James Mitchell [mailto:[EMAIL PROTECTED] > >> Sent: Thursday, December 02, 2004 2:25 PM > >> To: Struts Developers List > >> Subject: Re: Package removal with new Digester > >> > >> That change was made at the end of October. > >> > >> So now what? Instead of ignoring those classes, shouldn't we fix the > >> problem? Like include the examples from digester or remove our files > >that > >> refer to the no longer existing package and classes? > >> > >> I am *not* looking for a vi vs. ide debate, but this is simply > >> aggravating. > >> Frankly, I'm tired of cutting off the end of the ham ;) > >> > >> > >> -- > >> James Mitchell > >> Software Engineer / Open Source Evangelist > >> EdgeTech, Inc. > >> 678.910.8017 > >> AIM: jmitchtx > >> > >> ----- Original Message ----- > >> From: "James Mitchell" <[EMAIL PROTECTED]> > >> To: "Struts Developers List" <[EMAIL PROTECTED]>; "Martin Cooper" > >> <[EMAIL PROTECTED]> > >> Sent: Thursday, December 02, 2004 1:52 PM > >> Subject: Re: Package removal with new Digester > >> > >> > >> > Wow, I was missing something. > >> > > >> > I just noticed this in build-webapp.xml: > >> > > >> > > >> > ... > >> > ... > >> > <!-- Excludes for tiles-documentation to reflect the fact that > >> > Commons Digester 1.6 no longer ships the RSS demo classes > >> > --> > >> > <exclude name="org/apache/struts/webapp/tiles/rssChannel/*.java"/> > >> > ... > >> > ... > >> > > >> > > >> > > >> > -- > >> > James Mitchell > >> > Software Engineer / Open Source Evangelist > >> > EdgeTech, Inc. > >> > 678.910.8017 > >> > AIM: jmitchtx > >> > > >> > ----- Original Message ----- > >> > From: "Martin Cooper" <[EMAIL PROTECTED]> > >> > To: "Struts Developers List" <[EMAIL PROTECTED]> > >> > Sent: Thursday, December 02, 2004 1:14 PM > >> > Subject: Re: Package removal with new Digester > >> > > >> > > >> >> Hmm, that's odd. How are you trying to build? I can run 'ant clean > >> >> dist' just fine with Digester 1.6, and that builds > >> >> tiles-documentation.war. > >> >> > >> >> -- > >> >> Martin Cooper > >> >> > >> >> > >> >> On Thu, 2 Dec 2004 09:54:36 -0500, James Mitchell > >> <[EMAIL PROTECTED]> > >> >> wrote: > >> >>> I can't compile the tiles-documentation sources with Digester 1.6, > >it > >> >>> requires the rss package that was moved to "src/examples/api/rss" > >> >>> according > >> >>> to the release notes. > >> >>> > >> >>> Did I miss something? How are we dealing with this? > >> >>> > >> >>> -- > >> >>> James Mitchell > >> >>> Software Engineer / Open Source Evangelist > >> >>> EdgeTech, Inc. > >> >>> 678.910.8017 > >> >>> AIM: jmitchtx > >> >>> > >> >>> > >--------------------------------------------------------------------- > >> >>> 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] > >> >> > >> >> > >> > > >> > > >> > > >> > > >--------------------------------------------------------------------- > >> > 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] > >> > > > > > >--------------------------------------------------------------------- > >To unsubscribe, e-mail: [EMAIL PROTECTED] > >For additional commands, e-mail: [EMAIL PROTECTED] > > > -- > Joe Germuska > [EMAIL PROTECTED] > http://blog.germuska.com > "Narrow minds are weapons made for mass destruction" -The Ex > > > > --------------------------------------------------------------------- > 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]
