You are viewing a plain text version of this content. The canonical link for it is here.
Posted to server-dev@james.apache.org by jw...@inovem.com on 2004/07/17 11:57:23 UTC

Cornerstone confusion

I'm confused about the Cornerstone setup in 3.0. 
To get 3.0a1 to compile (mostly) I need to add the following to the 
include.properties file (and build.xml):

cornerstone-connection-impl.jar=${candidates.dir}/cornerstone-connection-impl-
1.0.jar
cornerstone-store-impl.jar=${candidates.dir}/cornerstone-store-impl-1.0.jar

This doesn't seem right. James should need no knowledge of the implementation 
at all.

What confuses me is that Gump now builds James OK (I assume at version 3.0a1)

And after these steps 3.0 still won't compile 

C:\java\james-
server\src\java\org\apache\james\util\connection\SimpleConnectionManager.java:4
0: org.apache.james.util.connection.SimpleConnectionManager is not abstract 
and does not override abstract method connect(java.lang.String,java.net
.ServerSocket,org.apache.avalon.cornerstone.services.connection.ConnectionHandl
erFactory,org.apache.avalon.excalibur.thread.ThreadPool) in 
org.apache.avalon.cornerstone.services.connection.ConnectionManager
public class SimpleConnectionManager

Are there more James changes that need to be made with regards to Cornerstone?

Help!
-- Jason

---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org


Re: Cornerstone confusion

Posted by Stephen McConnell <mc...@apache.org>.
Stephen McConnell wrote:
> jw@inovem.com wrote:
> 
>> I'm confused about the Cornerstone setup in 3.0. To get 3.0a1 to 
>> compile (mostly) I need to add the following to the include.properties 
>> file (and build.xml):
>>
>> cornerstone-connection-impl.jar=${candidates.dir}/cornerstone-connection-impl- 
>>
>> 1.0.jar
>> cornerstone-store-impl.jar=${candidates.dir}/cornerstone-store-impl-1.0.jar 
>>
>>
>> This doesn't seem right. James should need no knowledge of the 
>> implementation at all.
> 
> 
> I haven't looked into this myself - but your right - in principal james 
> should not need to reference cornerstone-store-impl for compilation.

I've just fixed this - the issues was that I moved references from the 
old excalibur io stuff to equivalent classes in the store imp package. 
I should have a build.xml equivalent to the gump build in place shortly 
so any changes against MAIN's HEAD can be quickly validated.

Steve.

-- 

|---------------------------------------|
| Magic by Merlin                       |
| Production by Avalon                  |
|                                       |
| http://avalon.apache.org              |
|---------------------------------------|

---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org


RE: Cornerstone confusion

Posted by Jason Webb <jw...@inovem.com>.

> -----Original Message-----
> From: Stephen McConnell [mailto:mcconnell@apache.org]
> Sent: 20 July 2004 04:00
> To: James Developers List
> Subject: Re: Cornerstone confusion
> 
> Jason Webb wrote:
> 
> > <snip'd>
> >
> >>Don't think so.  What are the actual compile errors that come
> >>up when you exclude cornerstone store impl?
> >>
> >
> >
> > Of all these errors (ignoring the warnings) the last error in the list
> > (SimpleConnectionManager) seems to be the strangest as there appears to
> be a
> > conflict between package names for the ThreadPool class
> 
> Thats probably because your using cornerstone products at 1.0.0 and not
> 2.0.0 - I have a build file locally that is working nicely but I have a
> couple of things to take care of before committing (mainly the setup of
> a antlib that handles dependency management properly).
> 
> Current process I'm following is :
> 
>    1. get a build working that is base on latest releases of the
>       respective projects (done locally - commits in a few days)
>    2. extend the build and gump to deal with container concerns
>    3. extend the build and gump to deal with documentation and
>       dist concerns
> 
> The first step is easy and will be done shortly.  The second is where I
> intend to separate build solutions for phoenix and merlin and will take
> a little longer.  The third is preconditioned on the second.
OK. That makes sense. Thanks Steve. I was getting out of my depth :)
> 
> Cheers, Steve.
> 
> --
> 
> |---------------------------------------|
> | Magic by Merlin                       |
> | Production by Avalon                  |
> |                                       |
> | http://avalon.apache.org              |
> |---------------------------------------|
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
> For additional commands, e-mail: server-dev-help@james.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org


Re: Cornerstone confusion

Posted by Stephen McConnell <mc...@apache.org>.
Jason Webb wrote:

> <snip'd> 
> 
>>Don't think so.  What are the actual compile errors that come 
>>up when you exclude cornerstone store impl?
>>
> 
> 
> Of all these errors (ignoring the warnings) the last error in the list
> (SimpleConnectionManager) seems to be the strangest as there appears to be a
> conflict between package names for the ThreadPool class

Thats probably because your using cornerstone products at 1.0.0 and not 
2.0.0 - I have a build file locally that is working nicely but I have a 
couple of things to take care of before committing (mainly the setup of 
a antlib that handles dependency management properly).

Current process I'm following is :

   1. get a build working that is base on latest releases of the
      respective projects (done locally - commits in a few days)
   2. extend the build and gump to deal with container concerns
   3. extend the build and gump to deal with documentation and
      dist concerns

The first step is easy and will be done shortly.  The second is where I 
intend to separate build solutions for phoenix and merlin and will take 
a little longer.  The third is preconditioned on the second.

Cheers, Steve.

-- 

|---------------------------------------|
| Magic by Merlin                       |
| Production by Avalon                  |
|                                       |
| http://avalon.apache.org              |
|---------------------------------------|

---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org


RE: Cornerstone confusion

Posted by Jason Webb <jw...@inovem.com>.
<snip'd> 
> Don't think so.  What are the actual compile errors that come 
> up when you exclude cornerstone store impl?
> 

Of all these errors (ignoring the warnings) the last error in the list
(SimpleConnectionManager) seems to be the strangest as there appears to be a
conflict between package names for the ThreadPool class

-- Jason

----------------------------------------------------------------------------
----

C:\java\james-server\src\java\org\apache\james\mailrepository\filepair\Abstr
actF
ileRepository.java:37: package
org.apache.avalon.cornerstone.blocks.masterstore
does not exist
import org.apache.avalon.cornerstone.blocks.masterstore.ExtensionFileFilter;
                                                        ^
C:\java\james-server\src\java\org\apache\james\mailrepository\filepair\File_
Pers
istent_Object_Repository.java:32: package
org.apache.avalon.cornerstone.blocks.m
asterstore does not exist
import
org.apache.avalon.cornerstone.blocks.masterstore.ClassLoaderObjectInputSt
ream;
                                                        ^
C:\java\james-server\src\java\org\apache\james\mailrepository\filepair\File_
Pers
istent_Stream_Repository.java:34: package
org.apache.avalon.cornerstone.blocks.m
asterstore does not exist
import org.apache.avalon.cornerstone.blocks.masterstore.IOUtil;
                                                        ^
C:\java\james-server\src\java\org\apache\james\nntpserver\repository\Article
IDRe
pository.java:20: package org.apache.avalon.cornerstone.blocks.masterstore
does
not exist
import org.apache.avalon.cornerstone.blocks.masterstore.IOUtil;
                                                        ^
C:\java\james-server\src\java\org\apache\james\nntpserver\repository\NNTPArt
icle
Impl.java:29: package org.apache.avalon.cornerstone.blocks.masterstore does
not
exist
import org.apache.avalon.cornerstone.blocks.masterstore.IOUtil;
                                                        ^
C:\java\james-server\src\java\org\apache\james\nntpserver\repository\NNTPGro
upIm
pl.java:23: package org.apache.avalon.cornerstone.blocks.masterstore does
not ex
ist
import org.apache.avalon.cornerstone.blocks.masterstore.ExtensionFileFilter;
                                                        ^
C:\java\james-server\src\java\org\apache\james\nntpserver\repository\NNTPGro
upIm
pl.java:24: package org.apache.avalon.cornerstone.blocks.masterstore does
not ex
ist
import org.apache.avalon.cornerstone.blocks.masterstore.IOUtil;
                                                        ^
C:\java\james-server\src\java\org\apache\james\nntpserver\repository\NNTPSpo
oler
.java:30: package org.apache.avalon.cornerstone.blocks.masterstore does not
exis
t
import org.apache.avalon.cornerstone.blocks.masterstore.IOUtil;
                                                        ^
C:\java\james-server\src\java\org\apache\james\core\AbstractJamesService.jav
a:42
9: warning: org.apache.avalon.excalibur.thread.ThreadPool in
org.apache.avalon.e
xcalibur.thread has been deprecated
 
(org.apache.avalon.excalibur.thread.ThreadPool)threadPool);
                                                       ^
C:\java\james-server\src\java\org\apache\james\mailrepository\filepair\Abstr
actF
ileRepository.java:114: cannot resolve symbol
symbol  : class ExtensionFileFilter
location: class
org.apache.james.mailrepository.filepair.AbstractFileRepository
        m_filter = new ExtensionFileFilter(m_extension);
                       ^
C:\java\james-server\src\java\org\apache\james\mailrepository\filepair\File_
Pers
istent_Object_Repository.java:96: cannot resolve symbol
symbol  : class ClassLoaderObjectInputStream
location: class
org.apache.james.mailrepository.filepair.File_Persistent_Object_
Repository
                final ObjectInputStream stream = new
ClassLoaderObjectInputStrea
m( classLoader, inputStream );
                                                     ^
C:\java\james-server\src\java\org\apache\james\mailrepository\filepair\File_
Pers
istent_Stream_Repository.java:135: cannot resolve symbol
symbol  : variable IOUtil
location: class
org.apache.james.mailrepository.filepair.File_Persistent_Stream_
Repository
                IOUtil.shutdownStream( (InputStream)o );
                ^
C:\java\james-server\src\java\org\apache\james\mailrepository\filepair\File_
Pers
istent_Stream_Repository.java:144: cannot resolve symbol
symbol  : variable IOUtil
location: class
org.apache.james.mailrepository.filepair.File_Persistent_Stream_
Repository
                    IOUtil.shutdownStream( (InputStream)list.get( i ) );
                    ^
C:\java\james-server\src\java\org\apache\james\mailrepository\filepair\File_
Pers
istent_Stream_Repository.java:154: cannot resolve symbol
symbol  : variable IOUtil
location: class
org.apache.james.mailrepository.filepair.File_Persistent_Stream_
Repository
                IOUtil.shutdownStream( (OutputStream)o );
                ^
C:\java\james-server\src\java\org\apache\james\mailrepository\filepair\File_
Pers
istent_Stream_Repository.java:163: cannot resolve symbol
symbol  : variable IOUtil
location: class
org.apache.james.mailrepository.filepair.File_Persistent_Stream_
Repository
                    IOUtil.shutdownStream( (OutputStream)list.get( 0 ) );
                    ^
C:\java\james-server\src\java\org\apache\james\nntpserver\repository\Article
IDRe
pository.java:100: cannot resolve symbol
symbol  : variable IOUtil
location: class org.apache.james.nntpserver.repository.ArticleIDRepository
            IOUtil.shutdownStream(fout);
            ^
C:\java\james-server\src\java\org\apache\james\nntpserver\repository\Article
IDRe
pository.java:154: cannot resolve symbol
symbol  : variable IOUtil
location: class org.apache.james.nntpserver.repository.ArticleIDRepository
            IOUtil.shutdownStream(fin);
            ^
C:\java\james-server\src\java\org\apache\james\nntpserver\repository\NNTPArt
icle
Impl.java:86: cannot resolve symbol
symbol  : variable IOUtil
location: class org.apache.james.nntpserver.repository.NNTPArticleImpl
            IOUtil.shutdownStream(fin);
            ^
C:\java\james-server\src\java\org\apache\james\nntpserver\repository\NNTPGro
upIm
pl.java:165: cannot resolve symbol
symbol  : class ExtensionFileFilter
location: class org.apache.james.nntpserver.repository.NNTPGroupImpl
             new InvertedFileFilter(new ExtensionFileFilter(".id"))));
                                        ^
C:\java\james-server\src\java\org\apache\james\nntpserver\repository\NNTPGro
upIm
pl.java:240: cannot resolve symbol
symbol  : variable IOUtil
location: class org.apache.james.nntpserver.repository.NNTPGroupImpl
            IOUtil.copy(newsStream,fout);
            ^
C:\java\james-server\src\java\org\apache\james\nntpserver\repository\NNTPGro
upIm
pl.java:243: cannot resolve symbol
symbol  : variable IOUtil
location: class org.apache.james.nntpserver.repository.NNTPGroupImpl
            IOUtil.shutdownStream(fout);
            ^
C:\java\james-server\src\java\org\apache\james\nntpserver\repository\NNTPSpo
oler
.java:272: cannot resolve symbol
symbol  : variable IOUtil
location: class
org.apache.james.nntpserver.repository.NNTPSpooler.SpoolerRunnab
le
                    IOUtil.shutdownStream(fin);
                    ^
C:\java\james-server\src\java\org\apache\james\nntpserver\repository\NNTPSpo
oler
.java:308: cannot resolve symbol
symbol  : variable IOUtil
location: class
org.apache.james.nntpserver.repository.NNTPSpooler.SpoolerRunnab
le
                        IOUtil.shutdownStream(fout);
                        ^
C:\java\james-server\src\java\org\apache\james\nntpserver\repository\NNTPSpo
oler
.java:332: cannot resolve symbol
symbol  : variable IOUtil
location: class
org.apache.james.nntpserver.repository.NNTPSpooler.SpoolerRunnab
le
                            IOUtil.shutdownStream(newsStream);
                            ^
C:\java\james-server\src\java\org\apache\james\remotemanager\RemoteManagerHa
ndle
r.java:504: warning: getUserByName(java.lang.String) in
org.apache.mailet.UsersR
epository has been deprecated
        User user = users.getUserByName(username);
                         ^
C:\java\james-server\src\java\org\apache\james\remotemanager\RemoteManagerHa
ndle
r.java:680: warning: getUserByName(java.lang.String) in
org.apache.mailet.UsersR
epository has been deprecated
        JamesUser user = (JamesUser) users.getUserByName(username);
                                          ^
C:\java\james-server\src\java\org\apache\james\remotemanager\RemoteManagerHa
ndle
r.java:685: warning: getUserByName(java.lang.String) in
org.apache.mailet.UsersR
epository has been deprecated
        JamesUser aliasUser = (JamesUser) users.getUserByName(alias);
                                               ^
C:\java\james-server\src\java\org\apache\james\remotemanager\RemoteManagerHa
ndle
r.java:733: warning: getUserByName(java.lang.String) in
org.apache.mailet.UsersR
epository has been deprecated
        User baseuser = users.getUserByName(username);
                             ^
C:\java\james-server\src\java\org\apache\james\remotemanager\RemoteManagerHa
ndle
r.java:785: warning: getUserByName(java.lang.String) in
org.apache.mailet.UsersR
epository has been deprecated
        JamesUser user = (JamesUser) users.getUserByName(username);
                                          ^
C:\java\james-server\src\java\org\apache\james\remotemanager\RemoteManagerHa
ndle
r.java:818: warning: getUserByName(java.lang.String) in
org.apache.mailet.UsersR
epository has been deprecated
        JamesUser user = (JamesUser) users.getUserByName(username);
                                          ^
C:\java\james-server\src\java\org\apache\james\transport\mailets\LocalDelive
ry.j
ava:132: warning:
storeMail(org.apache.mailet.MailAddress,org.apache.mailet.Mail
Address,javax.mail.internet.MimeMessage) in org.apache.mailet.MailetContext
has
been deprecated
                    getMailetContext().storeMail(
                                    ^
C:\java\james-server\src\java\org\apache\james\transport\mailets\LocalDelive
ry.j
ava:154: warning: getUserByName(java.lang.String) in
org.apache.mailet.UsersRepo
sitory has been deprecated
                if ((JamesUser) localusers.getUserByName(username)
                                          ^
C:\java\james-server\src\java\org\apache\james\transport\mailets\LocalDelive
ry.j
ava:157: warning: getUserByName(java.lang.String) in
org.apache.mailet.UsersRepo
sitory has been deprecated
                        (JamesUser) localusers.getUserByName(username);
                                              ^
C:\java\james-server\src\java\org\apache\james\util\connection\SimpleConnect
ionM
anager.java:40: org.apache.james.util.connection.SimpleConnectionManager is
not
abstract and does not override abstract method
connect(java.lang.String,java.net
.ServerSocket,org.apache.avalon.cornerstone.services.connection.ConnectionHa
ndle
rFactory,org.apache.avalon.excalibur.thread.ThreadPool) in
org.apache.avalon.cor
nerstone.services.connection.ConnectionManager
public class SimpleConnectionManager
       ^

<snip'd>


---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org


Re: Cornerstone confusion

Posted by Stephen McConnell <mc...@apache.org>.
jw@inovem.com wrote:

> I'm confused about the Cornerstone setup in 3.0. 
> To get 3.0a1 to compile (mostly) I need to add the following to the 
> include.properties file (and build.xml):
> 
> cornerstone-connection-impl.jar=${candidates.dir}/cornerstone-connection-impl-
> 1.0.jar
> cornerstone-store-impl.jar=${candidates.dir}/cornerstone-store-impl-1.0.jar
> 
> This doesn't seem right. James should need no knowledge of the implementation 
> at all.

I haven't looked into this myself - but your right - in principal james 
should not need to reference cornerstone-store-impl for compilation.

> 
> What confuses me is that Gump now builds James OK (I assume at version 3.0a1)

Yep - gump is building 3.0a1 against the current Avalon trunk.

> 
> And after these steps 3.0 still won't compile 
> 
> C:\java\james-
> server\src\java\org\apache\james\util\connection\SimpleConnectionManager.java:4
> 0: org.apache.james.util.connection.SimpleConnectionManager is not abstract 
> and does not override abstract method connect(java.lang.String,java.net
> .ServerSocket,org.apache.avalon.cornerstone.services.connection.ConnectionHandl
> erFactory,org.apache.avalon.excalibur.thread.ThreadPool) in 
> org.apache.avalon.cornerstone.services.connection.ConnectionManager
> public class SimpleConnectionManager
> 
> Are there more James changes that need to be made with regards to Cornerstone?

Don't think so.  What are the actual compile errors that come up when 
you exclude cornerstone store impl?

Stephen.


-- 

|---------------------------------------|
| Magic by Merlin                       |
| Production by Avalon                  |
|                                       |
| http://avalon.apache.org              |
|---------------------------------------|

---------------------------------------------------------------------
To unsubscribe, e-mail: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org