[jboss-jira] [JBoss JIRA] Commented: (JBAS-4736) -P circumvents the additional processing needed for some props

[Raja (JIRA)]

    [ http://jira.jboss.com/jira/browse/JBAS-4736?page=comments#action_12378484 ] 
            
Raja commented on JBAS-4736:
----------------------------

I think its because of this:
Properties props = System.getProperties();

You dont need this as the props variable is defined at the top of this class and recreating a props variable here creates it in a local scope and doesnt do anything with the global props variable. 

> -P circumvents the additional processing needed for some props
> --------------------------------------------------------------
>
>                 Key: JBAS-4736
>                 URL: http://jira.jboss.com/jira/browse/JBAS-4736
>             Project: JBoss Application Server
>          Issue Type: Bug
>      Security Level: Public(Everyone can see) 
>    Affects Versions: JBossAS-4.2.0.GA
>            Reporter: John Mazzitelli
>
> org.jboss.Main does some special handling for some specific system properties (see JBAS-4181 for example).  
> So, when -b is passed in for example, it does some special things for UDP/JGroups stuff.
> However, if I start JBossAS using the -P option (a properties file that sets things like jboss.bind.address and a bunch of other things), none of this special processing is done.  From Main.java:
>             case 'P':
>             {
>                // Set system properties from url/file
>                URL url = makeURL(getopt.getOptarg());
>                Properties props = System.getProperties();
>                props.load(url.openConnection().getInputStream());
>                break;
>             }
> I think that case statement needs to do the things that the case '-b' does and that case 'u' does - based on what properties were loaded from that file.

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