comments inline:

On 14/07/06, Jean-Sebastien Delfino <[EMAIL PROTECTED]> wrote:


I tried the RC2 binary distribution, and was able to run the Calculator
samples including the Calculator Web Service.

The binary distributions look pretty good to me, I just found a few
minor issues that should be easy to fix or document.

SDO binary install:

In INSTALL, the Linux binary install section should describe steps to
configure TUSCANY_SDOCPP and make LD_LIBRARY_PATH point to
$TUSCANY_SDOCPP/lib. I'm actually not sure if TUSCANY_SDOCPP is
required, but the doc asks to configure it on Windows, so I'm guessing
that it should be the same on Linux...

bin/test/tuscany_sdo_test fails to load some XSDs, I think it should be
removed, or the missing XSDs added.


This test is really for verifying the source build. It was not intended to
be in ht bin distro.

It would be nice to have an SDO sample, or add the missing XSDs to get
tuscany_sdo_test working.


Working on this now!


SCA binary install:

In INSTALL, the Linux binary install section should describe steps to
configure TUSCANY_SCACPP and make LD_LIBRARY_PATH point to
$TUSCANY_SCACPP/lib. Note that if you don't set TUSCANY_SCACPP, client
programs still work, but Web Services running in the Axis2 HTTP server
fail with a "SystemConfigurationException has been caught:
TUSCANY_SCACPP environment variable not set" error.


TUSCANY_SCACPP is required to be set for the process where the runtime is
running. So it must be set when using a client as the runtime will be in the
same process. Not sure how it managed to work without this! The WSClient
does not need this set as it is a simple Axis2C client. TUSCANY_SCACPP must
be set for the process that will host the sca runtime, in this instance that
will be the axis server. All these instructions are already in the
samples/INSTALL instructions.

bin/test/tuscany_sca_test fails complaining that
TUSCANY_SCACPP_SYSTEM_ROOT is not set.


again this test program is not intended for bin release. for src release
there is a script to run it.

Just a suggestion, in
samples/Calculator/CalculatorModule/Calculator.wsdl, the soap:address
http://localhost/axis/CalculatorService does not match the actual
address of the Calculator service
http://localhost:9090/axis2/services/Calculator. This is not preventing
the sample to run because the Calculator client overrides the endpoint,
but I think that having the correct endpoint in the WSDL in the first
place will avoid confusion when people use that WSDL with a Web Service
test tool for example or build a variation of the Calculator sample.


Will fix this.


I'll try the source distributions as well tomorrow.


Many thanks!





--
Pete

Reply via email to