I see -- the patch doesn't add -sourcepath because it was already there in the file. :)
Alex Ramirez wrote: >Actually, -sourcepath is already there. Look at: > > >-----Original Message----- >From: Rich Feit [mailto:[EMAIL PROTECTED] >Sent: Monday, September 19, 2005 5:39 PM >To: Beehive Developers >Subject: Re: PATCH [jira] Resolved: (BEEHIVE-881) -Aweb.source.roots >option to Page Flow annotation processors is unnecessary > >Hey Alex, > >It looks like you deleted -Aweb.source.roots, but didn't add >-sourcepath. I'd expect that to blow up. It didn't cause any error >messages? > >Rich > >Alex Ramirez wrote: > > > >>Rich, >>For some reason I'm unable to attach the patch to the jira issue so >> >> >here > > >>it is. >> >>Alex >> >>-----Original Message----- >>From: Rich Feit (JIRA) [mailto:[EMAIL PROTECTED] >>Sent: Friday, September 16, 2005 3:36 PM >>To: [email protected] >>Subject: [jira] Resolved: (BEEHIVE-881) -Aweb.source.roots option to >>Page Flow annotation processors is unnecessary >> >> [ http://issues.apache.org/jira/browse/BEEHIVE-881?page=all ] >> >>Rich Feit resolved BEEHIVE-881: >>------------------------------- >> >> Resolution: Fixed >> Assign To: Alejandro Ramirez (was: Rich Feit) >> >>This was fixed with revision 279881. If you're running the annotation >>processor manually (through an apt command line), you can now just pass >>'-sourcepath <some path>' instead of '-Aweb.source.roots=<some path>'. >> >> >> >> >> >>>-Aweb.source.roots option to Page Flow annotation processors is >>> >>> >>> >>> >>unnecessary >> >> >>----------------------------------------------------------------------- >> >> >- > > >>--- >> >> >> >> >>> Key: BEEHIVE-881 >>> URL: http://issues.apache.org/jira/browse/BEEHIVE-881 >>> Project: Beehive >>> Type: Bug >>> Components: NetUI >>> Versions: v1m1 >>> Reporter: Rich Feit >>> Assignee: Alejandro Ramirez >>> Fix For: TBD >>> >>> >>> >>> >> >> >> >> >>>Currently, the Page Flow annotation processors discern the apt >>> >>> >>> >>> >>sourcepath by reading the value of -Aweb.source.roots. They can just >> >> >as > > >>well read the value of -sourcepath directly. >> >> >> >>----------------------------------------------------------------------- >> >> >- > > >>Index: >> >> >netui/test/src/compilerTests/org/apache/beehive/netui/test/compiler/RunA >pt.java > > >>=================================================================== >>--- >> >> >netui/test/src/compilerTests/org/apache/beehive/netui/test/compiler/RunA >pt.java (revision 290295) > > >>+++ >> >> >netui/test/src/compilerTests/org/apache/beehive/netui/test/compiler/RunA >pt.java (working copy) > > >>@@ -134,8 +134,6 @@ >> >> aptArgs.add("-Aweb.content.root=" + webappRoot); >> aptArgs.add("-Aweb.output.root=" + webappRoot); >>- aptArgs.add("-Aweb.source.roots=" + webappRoot + >> >> >File.pathSeparatorChar + webappRoot + File.separatorChar > > >>- + TestPropertyMgr.DIRNAME_WEB_INF + >> >> >File.separatorChar + TestPropertyMgr.SRC_DIR_NAME); > > >> // Find all the source files by the specified extentions >> Set srcExtentions = TestPropertyMgr.getSrcExts(); >> >> >> >> > > > >
