+1

Compiled source distro with a clean maven repo OK
Tested webapp samples using Tomcat and they are all working fine
Tested couple samples, including BPEL, and couple others, and found no issues.

One minor issue, is that the LICENSE file has wrong license for SDO
API, looks like only some files are released under ASF 2.0 and other
files have OSOA license, see[1]. I would suggest fixing that in trunk
and in the release branch.

[1]https://svn.apache.org/repos/asf/incubator/tuscany/java/sdo/distribution/src/main/release/bin/LICENSE


On 11/5/07, Mark Combellack <[EMAIL PROTECTED]> wrote:
> Raymond Feng <[EMAIL PROTECTED]> writes:
>
> >
> > Hi,
> >
> > Please review and vote on the 1.0.1-incubating release artifacts of Tuscany
> > SCA for Java.
> >
> > The RC4 contains a few fixes to issues discovered in RC3. Thank you for
> > reporting and fixing the issues promptly.
> >
> > SVN Tag:
> > http://svn.apache.org/repos/asf/incubator/tuscany/tags/java/sca/1.0.1-RC4/
> >
> > Stage maven repo: http://people.apache.org/~rfeng/tuscany/maven/
> >
> > RAT report:
> > http://people.apache.org/~rfeng/tuscany/1.0.1-RC4/1.0.1-RC4.rat.txt
> >
> > Distros (zip/gz/asc/md5) :
> > http://people.apache.org/~rfeng/tuscany/1.0.1-RC4/
> >
> > Thanks,
> > Raymond
> >
> > PS: I posted this message on Friday and it was only sent to myself
> >
>
>
>
> First of all, thanks Raymond for doing the builds for the next release.
>
> I've not had time to run all the samples but I have tried some of the samples
> using the binary distribution on Linux for the testing.
>
> Generally, the release looks good. I would give it a +0.5 as it has some minor
> issues but possibly not enough to prevent it's release.
>
>
> My review comments are below:
>
>
>
>
>
> Medium - perhaps needs investigation?
> =====================================
>
> Can't get mvn to build sample projects
> -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
>
> This might simply be because the artefacts are not published to the Maven
> repository yet.
>
> Cannot build with Maven. When I run the "mvn" command, I get the following
> output:
>
> [INFO] Scanning for projects...
> Downloading: 
> http://repo1.maven.org/maven2/org/apache/tuscany/sca/tuscany-sca/1.
> 0.1-incubating/tuscany-sca-1.0.1-incubating.pom
> [INFO] 
> ------------------------------------------------------------------------
> [ERROR] FATAL ERROR
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] Failed to resolve artifact.
>
> GroupId: org.apache.tuscany.sca
> ArtifactId: tuscany-sca
> Version: 1.0.1-incubating
>
> Reason: Unable to download the artifact from any repository
>
>   org.apache.tuscany.sca:tuscany-sca:pom:1.0.1-incubating
>
> from the specified remote repositories:
>   central (http://repo1.maven.org/maven2)
>
>
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] Trace
> org.apache.maven.reactor.MavenExecutionException: Cannot find parent: 
> org.apache
> .tuscany.sca:tuscany-sca for project: null:sample-osgi-supplychain:jar:null 
> for
> project null:sample-osgi-supplychain:jar:null
>         at org.apache.maven.DefaultMaven.getProjects(DefaultMaven.java:378)
>         at org.apache.maven.DefaultMaven.doExecute(DefaultMaven.java:290)
>         at org.apache.maven.DefaultMaven.execute(DefaultMaven.java:125)
>         at org.apache.maven.cli.MavenCli.main(MavenCli.java:280)
>         at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
>         at 
> sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.
> java:39)
>         at 
> sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAcces
> sorImpl.java:25)
>         at java.lang.reflect.Method.invoke(Method.java:585)
>         at org.codehaus.classworlds.Launcher.launchEnhanced(Launcher.java:315)
>         at org.codehaus.classworlds.Launcher.launch(Launcher.java:255)
>         at 
> org.codehaus.classworlds.Launcher.mainWithExitCode(Launcher.java:430)
>         at org.codehaus.classworlds.Launcher.main(Launcher.java:375)
> Caused by: org.apache.maven.project.ProjectBuildingException: Cannot find 
> parent
> : org.apache.tuscany.sca:tuscany-sca for project: 
> null:sample-osgi-supplychain:j
> ar:null for project null:sample-osgi-supplychain:jar:null
>         at 
> org.apache.maven.project.DefaultMavenProjectBuilder.assembleLineage(D
> efaultMavenProjectBuilder.java:1261)
>         at 
> org.apache.maven.project.DefaultMavenProjectBuilder.buildInternal(Def
> aultMavenProjectBuilder.java:747)
>         at 
> org.apache.maven.project.DefaultMavenProjectBuilder.buildFromSourceFi
> leInternal(DefaultMavenProjectBuilder.java:479)
>         at 
> org.apache.maven.project.DefaultMavenProjectBuilder.build(DefaultMave
> nProjectBuilder.java:200)
>         at org.apache.maven.DefaultMaven.getProject(DefaultMaven.java:553)
>         at 
> org.apache.maven.DefaultMaven.collectProjects(DefaultMaven.java:467)
>         at org.apache.maven.DefaultMaven.getProjects(DefaultMaven.java:364)
>         ... 11 more
> Caused by: org.apache.maven.project.ProjectBuildingException: POM 
> 'org.apache.tu
> scany.sca:tuscany-sca' not found in repository: Unable to download the 
> artifact
> from any repository
>
>   org.apache.tuscany.sca:tuscany-sca:pom:1.0.1-incubating
>
> from the specified remote repositories:
>   central (http://repo1.maven.org/maven2)
>  for project org.apache.tuscany.sca:tuscany-sca
>         at 
> org.apache.maven.project.DefaultMavenProjectBuilder.findModelFromRepo
> sitory(DefaultMavenProjectBuilder.java:573)
>         at 
> org.apache.maven.project.DefaultMavenProjectBuilder.assembleLineage(D
> efaultMavenProjectBuilder.java:1257)
>         ... 17 more
> Caused by: org.apache.maven.artifact.resolver.ArtifactNotFoundException: 
> Unable
> to download the artifact from any repository
>
>   org.apache.tuscany.sca:tuscany-sca:pom:1.0.1-incubating
>
> from the specified remote repositories:
>   central (http://repo1.maven.org/maven2)
>
>         at 
> org.apache.maven.artifact.resolver.DefaultArtifactResolver.resolve(De
> faultArtifactResolver.java:197)
>         at 
> org.apache.maven.artifact.resolver.DefaultArtifactResolver.resolve(De
> faultArtifactResolver.java:73)
>         at 
> org.apache.maven.project.DefaultMavenProjectBuilder.findModelFromRepo
> sitory(DefaultMavenProjectBuilder.java:526)
>         ... 18 more
> Caused by: org.apache.maven.wagon.ResourceDoesNotExistException: Unable to 
> downl
> oad the artifact from any repository
>         at 
> org.apache.maven.artifact.manager.DefaultWagonManager.getArtifact(Def
> aultWagonManager.java:324)
>         at 
> org.apache.maven.artifact.resolver.DefaultArtifactResolver.resolve(De
> faultArtifactResolver.java:185)
>         ... 20 more
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] Total time: < 1 second
> [INFO] Finished at: Mon Nov 05 20:45:25 GMT 2007
> [INFO] Final Memory: 2M/38M
> [INFO] 
> ------------------------------------------------------------------------
>
>
>
> Classpath in simple-callback-ws/README is wrong
> -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
>
> The README has sample-simple-callback.jar in the class path but the jar file
> is actually called sample-simple-callback-ws.jar. As a result, the sample
> does not run and throws:
>
> Exception in thread "main" java.lang.NoClassDefFoundError: 
> simplecallback/Simple
> CallbackClient
>
>
> Once the jar file name is corrected, the sample runs fine.
>
>
>
>
>
>
> Trivial - does not need fixing before release
> =============================================
>
> sample/osgi-supplychain/README
> -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
>
> Line 22 incorrectly states that the expected output is:
>     Work thread Thread[Thread-1,5,main] - Order, submitted, fulfilled, shipped
>
> However, lines 131 and 154 correctly state that the expected output is:
>     Work thread Thread[pool-1-thread-1,5,main] - Order, submitted, fulfilled, 
> sh
> ipped
>
> Line 22 is has the wrong thread name
>
>
>
> Add link to Jira server on CHANGES file
> -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
>
> The CHANGES file lists the Jiras that have been fixed. It might be nice to
> add the URL of where the Jira server is located.
>
>
>
> Not all the samples are listed in samples/README
> -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
>
> The following samples are in the samples directory but not listed in the
> samples/README file.
>
> binding-echo
> binding-echo-extension
> binding-notification-broker
> binding-notification-consumer
> binding-notification-producer
> helloworld-ws-reference-jms
> helloworld-ws-reference-secure
> helloworld-ws-service
> implementation-crud
> implementation-crud-extension
> implementation-notification
> implementation-pojo-extension
>
>
>
> Instructions for running quote-xquery sample are incorrect
> -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
>
> The instructions correctly list the command "ant run-server" to run the 
> server.
>
> The instructions do not mention that you need to run the command "ant run" to
> run the client
>
>
>
> implementation-pojo-extension sample is not complete
> -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
>
> Generally, the samples have a Java command line in the README for running the
> sample without ant. However, the implementation-pojo-extension/README does 
> not.
>
> The command "ant run" does not work for the implementation-pojo-extension
> sample
>
>
>
> implementation-notification sample is not complete
> -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
>
> The jar file in the target directory of the implementation-notification sample
> is named differently (sample-implementation-notification-1.0.1-incubating.jar)
> whereas all other sample jar files do not include the "-1.0.1-incubating"
> version information
>
>
>
> implementation-notification sample no *nix instructions
> -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
>
> The implementation-notification/README file is missing the command line for
> *nix. The command is:
>
> java -cp 
> ../../lib/tuscany-sca-manifest.jar:target/sample-implementation-notific
> ation.jar notification.TrafficAdvisoryClient
>
> Note: I have assumed that the jar has been renamed (see above issue)
>
>
>
> Thanks,
>
> Mark
>
>
>
>
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
>


-- 
Luciano Resende
Apache Tuscany Committer
http://people.apache.org/~lresende
http://lresende.blogspot.com/

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to