Comments inline ... On Mon, Apr 7, 2008 at 10:43 AM, Simon Laws <[EMAIL PROTECTED]> wrote: > On Mon, Apr 7, 2008 at 5:17 PM, Luciano Resende <[EMAIL PROTECTED]> > wrote: > > > > > Thanks all for the quick feedback. I'm fixing these issues and should > > have a RC3a sometime today for review. > > > > On Mon, Apr 7, 2008 at 8:51 AM, Simon Laws <[EMAIL PROTECTED]> > > wrote: > > > > > > On Mon, Apr 7, 2008 at 10:25 AM, ant elder <[EMAIL PROTECTED]> wrote: > > > > > > > On Mon, Apr 7, 2008 at 6:14 AM, Luciano Resende <[EMAIL PROTECTED]> > > > > wrote: > > > > > > > > > Please review and vote on the 1.2 release artifacts of Tuscany SCA > > for > > > > > Java. > > > > > > > > > > The artifacts are available for review at: > > > > > > http://people.apache.org/~lresende/tuscany/sca-1.2-RC3/<http://people.apache.org/%7Elresende/tuscany/sca-1.2-RC3/> > > <http://people.apache.org/%7Elresende/tuscany/sca-1.2-RC3/> > > > > <http://people.apache.org/%7Elresende/tuscany/sca-1.2-RC3/> > > > > > > > > > > > > > > > This includes the signed binary and source distributions, the RAT > > > > report, > > > > > and the Maven staging repository. > > > > > > > > > > The eclipse updatesite for the Tuscany Eclipse plugins is available > > at: > > > > > > http://people.apache.org/~lresende/tuscany/sca-1.2-RC3/updatesite/<http://people.apache.org/%7Elresende/tuscany/sca-1.2-RC3/updatesite/> > > <http://people.apache.org/%7Elresende/tuscany/sca-1.2-RC3/updatesite/> > > > > <http://people.apache.org/%7Elresende/tuscany/sca-1.2-RC3/updatesite/ > > > > > > > > > > > > > The release tag is available at : > > > > > > > http://svn.apache.org/repos/asf/incubator/tuscany/tags/java/sca/1.2-RC3/ > > > > > > > > > > > > > > > Looks OK to me, here is my +1. > > > > > > > > > > -- > > > > > Luciano Resende > > > > > > > > > > > > > This looks pretty good to me but the problem with building with mvn > > > > prevents > > > > complete testing and there's a problem with the legal files being > > > > duplicated > > > > in the src distro so we probably need a respin to fix those things. > > > > > > > > The things I've found so far are: > > > > > > > > - src distro has got two copies of legal files in top level folder > > > > > > > > - The update site jars NOTICE file has the project name as ${pom.name > > } > > > > instead of Apache Tuscany > > > > > > > > - building samples with mvn in bin dsitro fails with > > > > java.lang.NoClassDefFoundError: > > org/apache/tuscany/sca/work/WorkScheduler > > > > at > > > > > > > > > > > org.apache.tuscany.sca.host.embedded.impl.ReallySmallRuntime.start(ReallySmallRuntime.java:102) > > > > > > > > - building src distro fails in modules/core with missing class > > > > WorkScheduler > > > > > > > > - INSTALL file mentions doc folder which missing and does not mention > > > > tutorials folder > > > > > > > > - tutorial catalog includes a built webapp is that intentional? And > > trying > > > > to deploy that webapp in tomcat gives a SEVERE: Error Starting SCA > > WebApp > > > > Node > > > > java.lang.ClassNotFoundException: > > > > > > org.apache.tuscany.sca.implementation.node.webapp.NodeWebAppServletHost > > > > > > > > - the README in the tutorial folder doesn't give much info and just > > points > > > > to a website page but that page doesn't look finished and gives > > little > > > > help > > > > > > > > ...ant > > > > > > > > > > Hi Luciano > > > > > > Just started checking through RC3. The samples I tried look good. I had > > > problems with the maven build of the samples but by adding > > > > http://people.apache.org/~lresende/tuscany/sca-1.2-RC3/maven/<http://people.apache.org/%7Elresende/tuscany/sca-1.2-RC3/maven/>as > a maven > > > > > mirror I was able to make progress. > > > > > > Looking at the LICENSE file. Things in the lib dir but not in the > > license > > > file are; > > > > > > stax-api-1.0-2.jar > > > > > > Things in the LICENSE file but not in the lib dir are; > > > > > > commons-cli-1.0.jar > > > activemq-core-4.1.1.jar > > > axis2-adb-codegen-1.3.jar > > > maven-artifact-2.0.2.jar > > > maven-artifact-manager-2.0.2.jar > > > maven-error-diagnostics-2.0.2.jar > > > maven-model-2.0.2.jar > > > maven-profile-2.0.2.jar > > > maven-project-2.0.2.jar > > > maven-repository-metadata-2.0.2.jar > > > maven-settings-2.0.2.jar > > > opensaml-1.1.jar > > > wagon-file-1.0-alpha-7.jar > > > wagon-http-lightweight-1.0-alpha-6.jar > > > wagon-provider-api-1.0-alpha-6.jar > > > xbean-2.1.0.jar > > > xmlParserAPIs-2.6.2.jar > > > jaxen-1.1-1.jar > > > stax-api-1.0.2.jar > > > > > > Before I carry on checking other things, what's the protocol for making > > > changes to the 1.2 branch? Shall I report them here and go in and fix > > them > > > or do you want to make the changes yourself? > > > > > > Regards > > > > > > Simon > > > > > > > > > > > -- > > Luciano Resende > > Apache Tuscany Committer > > http://people.apache.org/~lresende <http://people.apache.org/%7Elresende> > > > http://lresende.blogspot.com/ > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > > Some more comments on RC3... > > bin > LICENSE > Difference between LICENSE/lib already reported
Fixed in branch already. > NOTICE > The way that the sca xsd license is referred to is inconsistent when > compared to the SDO license. Can we replace > > "This product also includes software under the OSOA license > (see the LICENSE file contained in this distribution), with > the following copyright:" > > with > > "This product also includes software under the Service Component > Architecture JavaDoc, > Interface Definition files and XSD files license, with the following > copyright:" > Could you elaborate more on your proposal ? Also, looks like the proposed text might be missing some text ? Maybe a pointer to the SDO place you used as comparison would help me understand better. I was looking at the latest 1.1 SDO tag, but not sure I found the right place. > > Need to add > > "This product includes software under the Apache License Version 2.0 > with the following copyright: > > Copyright (c) 2003-2004 Jan-Klaas Kollhof > Copyright (c) 2005 Michael Clark, Metaparadigm Pte Ltd > > Please see the LICENSE file present in the root directory of this > distribution. " > Where is this notice coming from ? > > /modules now includes tuscany-node2-launcher-webapp-1.2-incubating.war and > it also appears as a separate maven artifact > (Have to admit that I don't know what this is for, although I can guess) > It doesn't have license and notice files and the manifest information is > different from the module jars. > Should this really ship as a separate maven artifact? > As for the war license, etc, it's following the same as our sample apps. But you are right, we should probably not ship this in maven. > src > ships itest/transaction/derbydb This is required by transaction (it has the derby db in svn) > Got the mvn build problem that has already been reported so can't test much > more Fixed in branch > > updatesite > Add a README Fixed in the branch. > > Regards > > Simon > -- 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]
