Done (in trunk). Thanks Alex. Rich Rich Feit wrote:
>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(); >>> >>> >>> >>> >>> >>> >> >> >> >> > > >
