Okay can you make a note of that in the review.txt file, Remember right now we am only trying to establish what is needed, coming up with a plan will occur after we know what needs doing - and module maintainers will be on the ground figuring out what is needed where.... Jody > We need volunter for the work described below. > > Jody Garnett a écrit : >> The jjtree-javacc is (C) The Codehaus and is under an Apache license.... >>> /* >>> * Copyright 2001-2005 The Codehaus. >>> * >>> * Licensed under the Apache License, Version 2.0 (the "License"); >>> * you may not use this file except in compliance with the License. >>> * You may obtain a copy of the License at >>> * >>> * http://www.apache.org/licenses/LICENSE-2.0 >>> * >>> * Unless required by applicable law or agreed to in writing, software >>> * distributed under the License is distributed on an "AS IS" BASIS, >>> * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or >>> implied. >>> * See the License for the specific language governing permissions and >>> * limitations under the License. >>> */ >> Can you resolve this? (Or I guess kick it out of our codebase...) > > > It would be very nice to kick this module out of Geotools code base. > However, it would requires volunter work. I took this script form the > Mojo project and modified it at a time when the Mojo's Jacacc plugin > wasn't working. Now the real Mojo javacc plugin is released: > > http://mojo.codehaus.org/javacc-maven-plugin/ > > We should delete our custom "maven/gt2-jjtree-javacc" plugin and use > "javacc-maven-plugin" instead. The two goals "javacc:jjtree" and > "javacc:javacc" will be required (we need to perform "jjtree" on the > "module/main/src/org/geotools/filter/parser/expr.jjt" file first, and > then "javacc" on its output, taking in account the > "module/main/modified-src" directory). Do we have a volunter for this > work? Work is required because the "javacc-maven-plugin" is not > working exactly in the same way than the custom > "maven/gt2-jjtree-javacc" plugin I did 10 months ago. > > Note: "javacc" is used for generating a Filter parser. This code was > setup by James and Ian Schneider (if my memory serve me right) two > years ago. I don't know if it still used (which code people uses for > filter parsing today?). The generated sources are there: > > > http://javadoc.geotools.fr/snapshot/org/geotools/filter/parser/package-summary.html > > > > If we don't have a volunter work for doing this job now, I may be able > to do it in october. This job may be easier to do after we fixed > http://jira.codehaus.org/browse/GEOT-887 since it would make the > "javacc-maven-plugin" easier to configure. In the main time, since the > "maven/gt2-jjtree-javacc" plugin is only used for the build and is not > part of the distribution, I guess that Geotools users don't need to > care about this Apache license. > > > Martin.
------------------------------------------------------------------------- Take Surveys. Earn Cash. Influence the Future of IT Join SourceForge.net's Techsay panel and you'll get the chance to share your opinions on IT & business topics through brief surveys -- and earn cash http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV _______________________________________________ Geotools-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geotools-devel
