RE: [JBoss-dev] jboss.jcml reading properties problem/bug.

2001-07-19 Thread Schaefer, Andreas

Sorry

Ignore it because it is fixed with the latest CVS updates from
Scott.

Andy

> -Original Message-
> From: Schaefer, Andreas [mailto:[EMAIL PROTECTED]]
> Sent: Thursday, July 19, 2001 1:02 PM
> To: '[EMAIL PROTECTED]'
> Subject: RE: [JBoss-dev] jboss.jcml reading properties problem/bug.
> 
> 
> Hi Geeks
> 
> I just took the latest code from CVS and when I start JBoss it tells
> me that the Document Root element is missing:
> java.io.IOException: Document root element is missing
> which comes from
> org.jboss.configuration.ConfigurationService.loadConfiguration
> line 433.
> 
> Andy
> 
> > -Original Message-
> > From: Scott M Stark [mailto:[EMAIL PROTECTED]]
> > Sent: Thursday, July 19, 2001 10:16 AM
> > To: [EMAIL PROTECTED]
> > Subject: Re: [JBoss-dev] jboss.jcml reading properties problem/bug.
> > 
> > 
> > Yes.
> > 
> > - Original Message - 
> > From: "Jay Walters" <[EMAIL PROTECTED]>
> > To: <[EMAIL PROTECTED]>
> > Sent: Thursday, July 19, 2001 5:30 AM
> > Subject: RE: [JBoss-dev] jboss.jcml reading properties problem/bug.
> > 
> > 
> > > Did you make this change to the mainline as well?
> > > 
> > > -Original Message-
> > > From: Scott M Stark [mailto:[EMAIL PROTECTED]]
> > > Sent: Wednesday, July 18, 2001 7:17 PM
> > > To: [EMAIL PROTECTED]
> > > Subject: Re: [JBoss-dev] jboss.jcml reading properties 
> problem/bug.
> > > 
> > > 
> > > Throwing away the new lines is not correct. The jboss.jcml 
> > file is an xml
> > > file and it
> > > does not have line oriented semantics. The change you 
> > propose to add in the
> > > missing new line makes sense. I'll add it to the next beta 
> > release which
> > > will come out tomorrow.
> > > 
> > > - Original Message - 
> > > From: "John Landers" <[EMAIL PROTECTED]>
> > > To: <[EMAIL PROTECTED]>
> > > Sent: Wednesday, July 18, 2001 3:26 PM
> > > Subject: [JBoss-dev] jboss.jcml reading properties problem/bug.
> > > 
> > > 
> > > > I was working with jboss2.4 beta and now the code
> > > > that reads this file strips out the cr/lf since it is using
> > > > BufferedReader.readLine(). This works fine for most xml
> > > > but for MBeans that use and attribute that contains 
> > multiple properties
> > > > like this:
> > > >> > >  name="JCA:service=ConnectionFactoryLoader,name=JmsXA">
> > > > 
> > > >   MinSize=0
> > > >   MaxSize=10
> > > >   Blocking=true
> > > >   GCEnabled=false
> > > > 
> > > > It becomes a string with a single line of all the 
> > properties and the
> > > > java.util.Properties::load only gets
> > > > a single property.
> > > > 
> > > > The problem is in
> > > > org/jboss/configuration/ConfigurationService::loadConfiguration
> > > > public void loadConfiguration() throws Exception {
> > > > // The class loader used to kocal the configuration file
> > > > ClassLoader loader =
> > > Thread.currentThread().getContextClassLoader();
> > > > 
> > > > // Load user config from XML, and create the MBeans
> > > > InputStream input =
> > > loader.getResourceAsStream(CONFIGURATION_FILE);
> > > > 
> > > > StringBuffer sbufData = new StringBuffer();
> > > > BufferedReader br = new BufferedReader(new
> > > > InputStreamReader(input));
> > > > 
> > > > String sTmp;
> > > > 
> > > > try {
> > > > while((sTmp = br.readLine())!=null){
> > > >sbufData.append(sTmp);
> > > > // need to append in the eol
> > > > sbufData.append(System.getProperty("line.separator"));
> > > > }
> > > > } finally {
> > > > input.close();
> > > > }
> > > > 
> > > > Does this sound correct or are properties used in this 
> fashion not
> > > supported
> > > > anymore?
> > > > Maybe it is already fixed.
> > > > 
> > > > jcl.
> > > > 
> > > > ___
> > > > Jboss-development mailing list
> > > > [EMAIL PROTECTED]
> > > > http://lists.sourceforge.net/lists/listinfo/jboss-development
> > > > 
> > > 
> > > 
> > > ___
> > > Jboss-development mailing list
> > > [EMAIL PROTECTED]
> > > http://lists.sourceforge.net/lists/listinfo/jboss-development
> > > 
> > > ___
> > > Jboss-development mailing list
> > > [EMAIL PROTECTED]
> > > http://lists.sourceforge.net/lists/listinfo/jboss-development
> > > 
> > 
> > 
> > ___
> > Jboss-development mailing list
> > [EMAIL PROTECTED]
> > http://lists.sourceforge.net/lists/listinfo/jboss-development
> > 
> 
> 
> ___
> Jboss-development mailing list
> [EMAIL PROTECTED]
> http://lists.sourceforge.net/lists/listinfo/jboss-development
> 


___
Jboss-development mailing list
[EMAIL PROTECTED]
http://lists.sourceforge.net/lists/listinfo/jboss-development



Re: [JBoss-dev] jboss.jcml reading properties problem/bug.

2001-07-19 Thread Scott M Stark

Fixed.

- Original Message - 
From: "Schaefer, Andreas" <[EMAIL PROTECTED]>
To: <[EMAIL PROTECTED]>
Sent: Thursday, July 19, 2001 1:01 PM
Subject: RE: [JBoss-dev] jboss.jcml reading properties problem/bug.


> Hi Geeks
> 
> I just took the latest code from CVS and when I start JBoss it tells
> me that the Document Root element is missing:
> java.io.IOException: Document root element is missing
> which comes from
> org.jboss.configuration.ConfigurationService.loadConfiguration
> line 433.
> 
> Andy
> 


___
Jboss-development mailing list
[EMAIL PROTECTED]
http://lists.sourceforge.net/lists/listinfo/jboss-development



RE: [JBoss-dev] jboss.jcml reading properties problem/bug.

2001-07-19 Thread Schaefer, Andreas

Hi Geeks

I just took the latest code from CVS and when I start JBoss it tells
me that the Document Root element is missing:
java.io.IOException: Document root element is missing
which comes from
org.jboss.configuration.ConfigurationService.loadConfiguration
line 433.

Andy

> -Original Message-
> From: Scott M Stark [mailto:[EMAIL PROTECTED]]
> Sent: Thursday, July 19, 2001 10:16 AM
> To: [EMAIL PROTECTED]
> Subject: Re: [JBoss-dev] jboss.jcml reading properties problem/bug.
> 
> 
> Yes.
> 
> - Original Message - 
> From: "Jay Walters" <[EMAIL PROTECTED]>
> To: <[EMAIL PROTECTED]>
> Sent: Thursday, July 19, 2001 5:30 AM
> Subject: RE: [JBoss-dev] jboss.jcml reading properties problem/bug.
> 
> 
> > Did you make this change to the mainline as well?
> > 
> > -Original Message-
> > From: Scott M Stark [mailto:[EMAIL PROTECTED]]
> > Sent: Wednesday, July 18, 2001 7:17 PM
> > To: [EMAIL PROTECTED]
> > Subject: Re: [JBoss-dev] jboss.jcml reading properties problem/bug.
> > 
> > 
> > Throwing away the new lines is not correct. The jboss.jcml 
> file is an xml
> > file and it
> > does not have line oriented semantics. The change you 
> propose to add in the
> > missing new line makes sense. I'll add it to the next beta 
> release which
> > will come out tomorrow.
> > 
> > - Original Message - 
> > From: "John Landers" <[EMAIL PROTECTED]>
> > To: <[EMAIL PROTECTED]>
> > Sent: Wednesday, July 18, 2001 3:26 PM
> > Subject: [JBoss-dev] jboss.jcml reading properties problem/bug.
> > 
> > 
> > > I was working with jboss2.4 beta and now the code
> > > that reads this file strips out the cr/lf since it is using
> > > BufferedReader.readLine(). This works fine for most xml
> > > but for MBeans that use and attribute that contains 
> multiple properties
> > > like this:
> > >> >  name="JCA:service=ConnectionFactoryLoader,name=JmsXA">
> > > 
> > >   MinSize=0
> > >   MaxSize=10
> > >   Blocking=true
> > >   GCEnabled=false
> > > 
> > > It becomes a string with a single line of all the 
> properties and the
> > > java.util.Properties::load only gets
> > > a single property.
> > > 
> > > The problem is in
> > > org/jboss/configuration/ConfigurationService::loadConfiguration
> > > public void loadConfiguration() throws Exception {
> > > // The class loader used to kocal the configuration file
> > > ClassLoader loader =
> > Thread.currentThread().getContextClassLoader();
> > > 
> > > // Load user config from XML, and create the MBeans
> > > InputStream input =
> > loader.getResourceAsStream(CONFIGURATION_FILE);
> > > 
> > > StringBuffer sbufData = new StringBuffer();
> > > BufferedReader br = new BufferedReader(new
> > > InputStreamReader(input));
> > > 
> > > String sTmp;
> > > 
> > > try {
> > > while((sTmp = br.readLine())!=null){
> > >sbufData.append(sTmp);
> > > // need to append in the eol
> > > sbufData.append(System.getProperty("line.separator"));
> > > }
> > > } finally {
> > > input.close();
> > > }
> > > 
> > > Does this sound correct or are properties used in this fashion not
> > supported
> > > anymore?
> > > Maybe it is already fixed.
> > > 
> > > jcl.
> > > 
> > > ___
> > > Jboss-development mailing list
> > > [EMAIL PROTECTED]
> > > http://lists.sourceforge.net/lists/listinfo/jboss-development
> > > 
> > 
> > 
> > ___
> > Jboss-development mailing list
> > [EMAIL PROTECTED]
> > http://lists.sourceforge.net/lists/listinfo/jboss-development
> > 
> > ___
> > Jboss-development mailing list
> > [EMAIL PROTECTED]
> > http://lists.sourceforge.net/lists/listinfo/jboss-development
> > 
> 
> 
> ___
> Jboss-development mailing list
> [EMAIL PROTECTED]
> http://lists.sourceforge.net/lists/listinfo/jboss-development
> 


___
Jboss-development mailing list
[EMAIL PROTECTED]
http://lists.sourceforge.net/lists/listinfo/jboss-development



Re: [JBoss-dev] jboss.jcml reading properties problem/bug.

2001-07-19 Thread Scott M Stark

Yes.

- Original Message - 
From: "Jay Walters" <[EMAIL PROTECTED]>
To: <[EMAIL PROTECTED]>
Sent: Thursday, July 19, 2001 5:30 AM
Subject: RE: [JBoss-dev] jboss.jcml reading properties problem/bug.


> Did you make this change to the mainline as well?
> 
> -Original Message-
> From: Scott M Stark [mailto:[EMAIL PROTECTED]]
> Sent: Wednesday, July 18, 2001 7:17 PM
> To: [EMAIL PROTECTED]
> Subject: Re: [JBoss-dev] jboss.jcml reading properties problem/bug.
> 
> 
> Throwing away the new lines is not correct. The jboss.jcml file is an xml
> file and it
> does not have line oriented semantics. The change you propose to add in the
> missing new line makes sense. I'll add it to the next beta release which
> will come out tomorrow.
> 
> - Original Message - 
> From: "John Landers" <[EMAIL PROTECTED]>
> To: <[EMAIL PROTECTED]>
> Sent: Wednesday, July 18, 2001 3:26 PM
> Subject: [JBoss-dev] jboss.jcml reading properties problem/bug.
> 
> 
> > I was working with jboss2.4 beta and now the code
> > that reads this file strips out the cr/lf since it is using
> > BufferedReader.readLine(). This works fine for most xml
> > but for MBeans that use and attribute that contains multiple properties
> > like this:
> >>  name="JCA:service=ConnectionFactoryLoader,name=JmsXA">
> > 
> >   MinSize=0
> >   MaxSize=10
> >   Blocking=true
> >   GCEnabled=false
> > 
> > It becomes a string with a single line of all the properties and the
> > java.util.Properties::load only gets
> > a single property.
> > 
> > The problem is in
> > org/jboss/configuration/ConfigurationService::loadConfiguration
> > public void loadConfiguration() throws Exception {
> > // The class loader used to kocal the configuration file
> > ClassLoader loader =
> Thread.currentThread().getContextClassLoader();
> > 
> > // Load user config from XML, and create the MBeans
> > InputStream input =
> loader.getResourceAsStream(CONFIGURATION_FILE);
> > 
> > StringBuffer sbufData = new StringBuffer();
> > BufferedReader br = new BufferedReader(new
> > InputStreamReader(input));
> > 
> > String sTmp;
> > 
> > try {
> > while((sTmp = br.readLine())!=null){
> >sbufData.append(sTmp);
> > // need to append in the eol
> > sbufData.append(System.getProperty("line.separator"));
> > }
> > } finally {
> > input.close();
> > }
> > 
> > Does this sound correct or are properties used in this fashion not
> supported
> > anymore?
> > Maybe it is already fixed.
> > 
> > jcl.
> > 
> > ___
> > Jboss-development mailing list
> > [EMAIL PROTECTED]
> > http://lists.sourceforge.net/lists/listinfo/jboss-development
> > 
> 
> 
> ___
> Jboss-development mailing list
> [EMAIL PROTECTED]
> http://lists.sourceforge.net/lists/listinfo/jboss-development
> 
> ___
> Jboss-development mailing list
> [EMAIL PROTECTED]
> http://lists.sourceforge.net/lists/listinfo/jboss-development
> 


___
Jboss-development mailing list
[EMAIL PROTECTED]
http://lists.sourceforge.net/lists/listinfo/jboss-development



RE: [JBoss-dev] jboss.jcml reading properties problem/bug.

2001-07-19 Thread Jay Walters

Did you make this change to the mainline as well?

-Original Message-
From: Scott M Stark [mailto:[EMAIL PROTECTED]]
Sent: Wednesday, July 18, 2001 7:17 PM
To: [EMAIL PROTECTED]
Subject: Re: [JBoss-dev] jboss.jcml reading properties problem/bug.


Throwing away the new lines is not correct. The jboss.jcml file is an xml
file and it
does not have line oriented semantics. The change you propose to add in the
missing new line makes sense. I'll add it to the next beta release which
will come out tomorrow.

- Original Message - 
From: "John Landers" <[EMAIL PROTECTED]>
To: <[EMAIL PROTECTED]>
Sent: Wednesday, July 18, 2001 3:26 PM
Subject: [JBoss-dev] jboss.jcml reading properties problem/bug.


> I was working with jboss2.4 beta and now the code
> that reads this file strips out the cr/lf since it is using
> BufferedReader.readLine(). This works fine for most xml
> but for MBeans that use and attribute that contains multiple properties
> like this:
> name="JCA:service=ConnectionFactoryLoader,name=JmsXA">
> 
>   MinSize=0
>   MaxSize=10
>   Blocking=true
>   GCEnabled=false
> 
> It becomes a string with a single line of all the properties and the
> java.util.Properties::load only gets
> a single property.
> 
> The problem is in
> org/jboss/configuration/ConfigurationService::loadConfiguration
> public void loadConfiguration() throws Exception {
> // The class loader used to kocal the configuration file
> ClassLoader loader =
Thread.currentThread().getContextClassLoader();
> 
> // Load user config from XML, and create the MBeans
> InputStream input =
loader.getResourceAsStream(CONFIGURATION_FILE);
> 
> StringBuffer sbufData = new StringBuffer();
> BufferedReader br = new BufferedReader(new
> InputStreamReader(input));
> 
> String sTmp;
> 
> try {
> while((sTmp = br.readLine())!=null){
>sbufData.append(sTmp);
> // need to append in the eol
> sbufData.append(System.getProperty("line.separator"));
> }
> } finally {
> input.close();
> }
> 
> Does this sound correct or are properties used in this fashion not
supported
> anymore?
> Maybe it is already fixed.
> 
> jcl.
> 
> ___
> Jboss-development mailing list
> [EMAIL PROTECTED]
> http://lists.sourceforge.net/lists/listinfo/jboss-development
> 


___
Jboss-development mailing list
[EMAIL PROTECTED]
http://lists.sourceforge.net/lists/listinfo/jboss-development

___
Jboss-development mailing list
[EMAIL PROTECTED]
http://lists.sourceforge.net/lists/listinfo/jboss-development



Re: [JBoss-dev] jboss.jcml reading properties problem/bug.

2001-07-18 Thread Scott M Stark

Throwing away the new lines is not correct. The jboss.jcml file is an xml file and it
does not have line oriented semantics. The change you propose to add in the
missing new line makes sense. I'll add it to the next beta release which
will come out tomorrow.

- Original Message - 
From: "John Landers" <[EMAIL PROTECTED]>
To: <[EMAIL PROTECTED]>
Sent: Wednesday, July 18, 2001 3:26 PM
Subject: [JBoss-dev] jboss.jcml reading properties problem/bug.


> I was working with jboss2.4 beta and now the code
> that reads this file strips out the cr/lf since it is using
> BufferedReader.readLine(). This works fine for most xml
> but for MBeans that use and attribute that contains multiple properties
> like this:
> name="JCA:service=ConnectionFactoryLoader,name=JmsXA">
> 
>   MinSize=0
>   MaxSize=10
>   Blocking=true
>   GCEnabled=false
> 
> It becomes a string with a single line of all the properties and the
> java.util.Properties::load only gets
> a single property.
> 
> The problem is in
> org/jboss/configuration/ConfigurationService::loadConfiguration
> public void loadConfiguration() throws Exception {
> // The class loader used to kocal the configuration file
> ClassLoader loader = Thread.currentThread().getContextClassLoader();
> 
> // Load user config from XML, and create the MBeans
> InputStream input = loader.getResourceAsStream(CONFIGURATION_FILE);
> 
> StringBuffer sbufData = new StringBuffer();
> BufferedReader br = new BufferedReader(new
> InputStreamReader(input));
> 
> String sTmp;
> 
> try {
> while((sTmp = br.readLine())!=null){
>sbufData.append(sTmp);
> // need to append in the eol
> sbufData.append(System.getProperty("line.separator"));
> }
> } finally {
> input.close();
> }
> 
> Does this sound correct or are properties used in this fashion not supported
> anymore?
> Maybe it is already fixed.
> 
> jcl.
> 
> ___
> Jboss-development mailing list
> [EMAIL PROTECTED]
> http://lists.sourceforge.net/lists/listinfo/jboss-development
> 


___
Jboss-development mailing list
[EMAIL PROTECTED]
http://lists.sourceforge.net/lists/listinfo/jboss-development