[ http://issues.apache.org/jira/browse/GERONIMO-1703?page=all ]

Paul McMahan updated GERONIMO-1703:
-----------------------------------

    Attachment: ServerInfoWebApp_g1.1.war

I'm attaching a new version of the WAR file that demonstrates the behavior, 
updated for the API and schema changes that have occurred in G1.1.

My reading of the current ServerInfo javadoc implies that the API in question 
*should* return null unless the setting is specified in config.xml.

    /**
     * A config.xml setting for the base directory.  This is normally
     * left null, which means the ServerInfo will use the Geronimo
     * install directory.
     */
    public String getBaseDirectory();

However, I have a patch ready if there's consensus that the API should return 
the current base directory instead of null.  This would make it redundant with 
the getCurrentBaseDirectory() API.  If you want the patch then just let me know.

> ServerInfo.getBaseDirectory() returns null
> ------------------------------------------
>
>          Key: GERONIMO-1703
>          URL: http://issues.apache.org/jira/browse/GERONIMO-1703
>      Project: Geronimo
>         Type: Bug
>     Security: public(Regular issues) 
>   Components: console
>  Environment: Sun JDK 1.4.2_08, Win XP, Geronimo-Tomcat
>     Reporter: Vamsavardhana Reddy
>     Assignee: Aaron Mulder
>     Priority: Critical
>      Fix For: 1.1
>  Attachments: ServerInfoWebApp.war, ServerInfoWebApp_g1.1.war
>
> Calling getBaseDirectory on org.apache.geronimo.system.serverinfo.ServerInfo 
> returned by KernelManagementHelper.getServerInfo() returns null instead of 
> Geronimo Base Directory.

-- 
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

Reply via email to