You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cocoon.apache.org by Christian Haul <ha...@dvs1.informatik.tu-darmstadt.de> on 2003/06/04 14:44:10 UTC
Re: cvs commit: cocoon-2.1/src/java/org/apache/cocoon/bean/helpers - New directory
On 04.Jun.2003 -- 09:23 AM, upayavira@apache.org wrote:
> upayavira 2003/06/04 02:23:03
>
> cocoon-2.1/src/java/org/apache/cocoon/bean/helpers - New directory
Would CocoonBean.INDEX_URI be the same as Constants.INDEX_URI?
CocoonProcessorDelegate in scratchpad seems to miss this constant....
Chris.
--
C h r i s t i a n H a u l
haul@informatik.tu-darmstadt.de
fingerprint: 99B0 1D9D 7919 644A 4837 7D73 FEF9 6856 335A 9E08
Re: cvs commit: cocoon-2.1/src/java/org/apache/cocoon/bean/helpers
Posted by Upayavira <uv...@upaya.co.uk>.
...
> >So, would you move the contants into Main? Or the Bean and change all
> >references? Where should they go?
> >
>
> I would move those into CocoonBean as you did; but after 2.1 is
> released.
Why?
There only a few constants. Why shouldn't we move them now, and change the
references?
Upayavira
Re: cvs commit: cocoon-2.1/src/java/org/apache/cocoon/bean/helpers
Posted by Vadim Gritsenko <va...@verizon.net>.
Upayavira wrote:
>>>But what about the dependencies upon them that are outside the scope
>>>of the CLI?
>>>
>>>
>>If you are referring to ResolverImplTestCase.java, then it *is* in
>>scope of CLI.
>>
>>
>
>Haven't a clue what it does.
>
>
>
>>See source:
>>
>> public void initialize() throws Exception {
>> workDir = new File(System.getProperty("java.io.tmpdir"),
>> Constants.DEFAULT_WORK_DIR);
>> commandlineContextDir = new File(workDir.getAbsolutePath());
>>
>>Key word here is "commandlineContextDir" :)
>>
>>
>
>So, would you move the contants into Main? Or the Bean and change all references?
>Where should they go?
>
I would move those into CocoonBean as you did; but after 2.1 is released.
Vadim
Re: cvs commit: cocoon-2.1/src/java/org/apache/cocoon/bean/helpers
Posted by Upayavira <uv...@upaya.co.uk>.
> >But what about the dependencies upon them that are outside the scope
> >of the CLI?
>
> If you are referring to ResolverImplTestCase.java, then it *is* in
> scope of CLI.
Haven't a clue what it does.
> See source:
>
> public void initialize() throws Exception {
> workDir = new File(System.getProperty("java.io.tmpdir"),
> Constants.DEFAULT_WORK_DIR);
> commandlineContextDir = new File(workDir.getAbsolutePath());
>
> Key word here is "commandlineContextDir" :)
So, would you move the contants into Main? Or the Bean and change all references?
Where should they go?
Regards, Upayavira
Re: cvs commit: cocoon-2.1/src/java/org/apache/cocoon/bean/helpers
Posted by Vadim Gritsenko <va...@verizon.net>.
Upayavira wrote:
>>>>Found another one: Constants.DEFAULT_WORK_DIR is used by
>>>>src/test/org/apache/cocoon/components/resolver/test/ResolverImplTest
>>>>Ca se.java
>>>>
>>>>
>>>I've moved the lot back. Obviously the comments that I based my move
>>>upon were wrong!
>>>
>>>
>>>
>>Comments are not wrong... Those constants should be moved. Eventually.
>>After 2.1 release, may be?
>>
>>
>
>But what about the dependencies upon them that are outside the scope of the CLI?
>
If you are referring to ResolverImplTestCase.java, then it *is* in scope
of CLI. See source:
public void initialize() throws Exception {
workDir = new File(System.getProperty("java.io.tmpdir"),
Constants.DEFAULT_WORK_DIR);
commandlineContextDir = new File(workDir.getAbsolutePath());
Key word here is "commandlineContextDir" :)
Vadim
Re: cvs commit: cocoon-2.1/src/java/org/apache/cocoon/bean/helpers
Posted by Upayavira <uv...@upaya.co.uk>.
> >>Found another one: Constants.DEFAULT_WORK_DIR is used by
> >>src/test/org/apache/cocoon/components/resolver/test/ResolverImplTest
> >>Ca se.java
>
> >I've moved the lot back. Obviously the comments that I based my move
> >upon were wrong!
> >
>
> Comments are not wrong... Those constants should be moved. Eventually.
> After 2.1 release, may be?
But what about the dependencies upon them that are outside the scope of the CLI?
Upayavira
Re: cvs commit: cocoon-2.1/src/java/org/apache/cocoon/bean/helpers
- New directory
Posted by Vadim Gritsenko <va...@verizon.net>.
Upayavira wrote:
>>>Fair enough. I've moved it back.
>>>
>>>
>>Found another one: Constants.DEFAULT_WORK_DIR is used by
>>src/test/org/apache/cocoon/components/resolver/test/ResolverImplTestCa
>>se.java
>>
>>
>
>I've moved the lot back. Obviously the comments that I based my move upon were
>wrong!
>
Comments are not wrong... Those constants should be moved. Eventually.
After 2.1 release, may be?
Vadim
Re: cvs commit: cocoon-2.1/src/java/org/apache/cocoon/bean/helpers - New directory
Posted by Upayavira <uv...@upaya.co.uk>.
> > Fair enough. I've moved it back.
>
> Found another one: Constants.DEFAULT_WORK_DIR is used by
> src/test/org/apache/cocoon/components/resolver/test/ResolverImplTestCa
> se.java
I've moved the lot back. Obviously the comments that I based my move upon were
wrong!
Regards, Upayavira
Re: cvs commit: cocoon-2.1/src/java/org/apache/cocoon/bean/helpers - New directory
Posted by Christian Haul <ha...@dvs1.informatik.tu-darmstadt.de>.
On 04.Jun.2003 -- 02:44 PM, Upayavira wrote:
> Christian,
>
> > > cocoon-2.1/src/java/org/apache/cocoon/bean/helpers - New directory
> >
> > Would CocoonBean.INDEX_URI be the same as Constants.INDEX_URI?
> > CocoonProcessorDelegate in scratchpad seems to miss this constant....
>
> Fair enough. I've moved it back.
Found another one: Constants.DEFAULT_WORK_DIR is used by
src/test/org/apache/cocoon/components/resolver/test/ResolverImplTestCase.java
Chris.
--
C h r i s t i a n H a u l
haul@informatik.tu-darmstadt.de
fingerprint: 99B0 1D9D 7919 644A 4837 7D73 FEF9 6856 335A 9E08
Re: cvs commit: cocoon-2.1/src/java/org/apache/cocoon/bean/helpers - New directory
Posted by Upayavira <up...@fwbo.org>.
Christian,
> > cocoon-2.1/src/java/org/apache/cocoon/bean/helpers - New directory
>
> Would CocoonBean.INDEX_URI be the same as Constants.INDEX_URI?
> CocoonProcessorDelegate in scratchpad seems to miss this constant....
Fair enough. I've moved it back.
Upayavira
Re: cvs commit: cocoon-2.1/src/java/org/apache/cocoon/bean/helpers - New directory
Posted by Upayavira <uv...@upaya.co.uk>.
Christian,
> > cocoon-2.1/src/java/org/apache/cocoon/bean/helpers - New directory
>
> Would CocoonBean.INDEX_URI be the same as Constants.INDEX_URI?
> CocoonProcessorDelegate in scratchpad seems to miss this constant....
Fair enough. I've moved it back.
Upayavira