> -----Original Message----- > From: David Crossley [mailto:cross...@apache.org] > Sent: Sunday, 11 January 2009 11:49 PM > To: dev@forrest.apache.org > Subject: Re: Remove local edits for input tei plugin > > Gavin wrote: > > Just checking, the forrest.properties of the input.tei plugin has :- > > > > > forrest.plugins.descriptors=http://forrest.apache.org/plugins/plugins.xml, > ht > > tp://forrest.apache.org/plugins/whiteboard- > plugins.xml,file:///c:/projects/f > > orrestPlugins/plugins.xml > > > > > project.required.plugins.src=/home/rgardler/projects/forrestPlugins,${forr > es > > t.home}/plugins,${forrest.home}/whiteboard/plugins > > > project.required.plugins=org.apache.forrest.plugin.input.tei,org.apache.fo > rr > > est.plugin.input.projectInfo > > > > IS it safe for me to remove :- > > > > file:///c:/projects/forrestPlugins/plugins.xml > > > > > project.required.plugins.src=/home/rgardler/projects/forrestPlugins,${forr > es > > t.home}/plugins,${forrest.home}/whiteboard/plugins > > Yes. > > Both of those local paths should not be there in the > committed work. > > They look like config for Ross' own local extra development stuff. > > AFAIK the "project.required.plugins.src" is not required. > I presume that Ross was over-riding that, so as to also > point to his extra work stuff. > > Looks like just a mistake with the original commit.
Ok, they are removed now. > > Are there similar things with the other recently added > plugins? I will check. I was going to do a nice perl line to add the extra lines needed by windows users to all the locationmap.xml files in all plugins, however I see it as a good opportunity to go through each one and check for things like this, so I'll do it that way. Thanks Gav... > > -David > > > -- > No virus found in this incoming message. > Checked by AVG. > Version: 7.5.552 / Virus Database: 270.10.5/1885 - Release Date: 1/9/2009 > 7:59 PM