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.  

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


Index: 
netui/test/src/compilerTests/org/apache/beehive/netui/test/compiler/RunApt.java
===================================================================
--- 
netui/test/src/compilerTests/org/apache/beehive/netui/test/compiler/RunApt.java 
    (revision 290295)
+++ 
netui/test/src/compilerTests/org/apache/beehive/netui/test/compiler/RunApt.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();

Reply via email to