You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tuscany.apache.org by Luciano Resende <lu...@gmail.com> on 2008/03/20 19:43:02 UTC

Feed Binding, was Re: svn commit: r639187 - in /incubator/tuscany/branches/sca-java-1.2: distribution/bundle/pom.xml distribution/manifest/pom.xml samples/calculator-distributed/build.xml

I'm adding back the Feed binding to the distribution as there is still
one sample that uses rome atom/rss binding.

On Thu, Mar 20, 2008 at 12:44 AM,  <js...@apache.org> wrote:
> Author: jsdelfino
>  Date: Thu Mar 20 00:44:42 2008
>  New Revision: 639187
>
>  URL: http://svn.apache.org/viewvc?rev=639187&view=rev
>  Log:
>  Added missing JARs to distribution build. Removed obsolete feed binding JARs. Fixed class names in calculator-distributed/build.xml.
>
>  Modified:
>     incubator/tuscany/branches/sca-java-1.2/distribution/bundle/pom.xml
>     incubator/tuscany/branches/sca-java-1.2/distribution/manifest/pom.xml
>     incubator/tuscany/branches/sca-java-1.2/samples/calculator-distributed/build.xml
>
>  Modified: incubator/tuscany/branches/sca-java-1.2/distribution/bundle/pom.xml
>  URL: http://svn.apache.org/viewvc/incubator/tuscany/branches/sca-java-1.2/distribution/bundle/pom.xml?rev=639187&r1=639186&r2=639187&view=diff
>  ==============================================================================
>  --- incubator/tuscany/branches/sca-java-1.2/distribution/bundle/pom.xml (original)
>  +++ incubator/tuscany/branches/sca-java-1.2/distribution/bundle/pom.xml Thu Mar 20 00:44:42 2008
>  @@ -64,7 +64,7 @@
>          </dependency>
>          <dependency>
>              <groupId>${pom.groupId}</groupId>
>  -            <artifactId>tuscany-binding-feed</artifactId>
>  +            <artifactId>tuscany-binding-atom</artifactId>
>              <version>${pom.version}</version>
>          </dependency>
>          <dependency>
>  @@ -94,6 +94,11 @@
>          </dependency>
>          <dependency>
>              <groupId>${pom.groupId}</groupId>
>  +            <artifactId>tuscany-binding-rss</artifactId>
>  +            <version>${pom.version}</version>
>  +        </dependency>
>  +        <dependency>
>  +            <groupId>${pom.groupId}</groupId>
>              <artifactId>tuscany-binding-rss-rome</artifactId>
>              <version>${pom.version}</version>
>          </dependency>
>  @@ -380,11 +385,26 @@
>          </dependency>
>          <dependency>
>              <groupId>${pom.groupId}</groupId>
>  +            <artifactId>tuscany-node2-api</artifactId>
>  +            <version>${pom.version}</version>
>  +        </dependency>
>  +        <dependency>
>  +            <groupId>${pom.groupId}</groupId>
>              <artifactId>tuscany-node-impl</artifactId>
>              <version>${pom.version}</version>
>          </dependency>
>          <dependency>
>              <groupId>${pom.groupId}</groupId>
>  +            <artifactId>tuscany-node2-impl</artifactId>
>  +            <version>${pom.version}</version>
>  +        </dependency>
>  +        <dependency>
>  +            <groupId>${pom.groupId}</groupId>
>  +            <artifactId>tuscany-node2-launcher</artifactId>
>  +            <version>${pom.version}</version>
>  +        </dependency>
>  +        <dependency>
>  +            <groupId>${pom.groupId}</groupId>
>              <artifactId>tuscany-policy</artifactId>
>              <version>${pom.version}</version>
>          </dependency>
>  @@ -507,12 +527,6 @@
>                                  </transformer>
>                                 <transformer implementation="org.codehaus.mojo.shade.resource.AppendingTransformer">
>                                      <resource>META-INF/services/org.apache.tuscany.sca.assembly.SCABindingFactory</resource>
>  -                                </transformer>
>  -                               <transformer implementation="org.codehaus.mojo.shade.resource.AppendingTransformer">
>  -                                    <resource>META-INF/services/org.apache.tuscany.sca.binding.feed.AtomBindingFactory</resource>
>  -                                </transformer>
>  -                               <transformer implementation="org.codehaus.mojo.shade.resource.AppendingTransformer">
>  -                                    <resource>META-INF/services/org.apache.tuscany.sca.binding.feed.RSSBindingFactory</resource>
>                                  </transformer>
>                                 <transformer implementation="org.codehaus.mojo.shade.resource.AppendingTransformer">
>                                      <resource>META-INF/services/org.apache.tuscany.sca.binding.atom.AtomBindingFactory</resource>
>
>  Modified: incubator/tuscany/branches/sca-java-1.2/distribution/manifest/pom.xml
>  URL: http://svn.apache.org/viewvc/incubator/tuscany/branches/sca-java-1.2/distribution/manifest/pom.xml?rev=639187&r1=639186&r2=639187&view=diff
>  ==============================================================================
>  --- incubator/tuscany/branches/sca-java-1.2/distribution/manifest/pom.xml (original)
>  +++ incubator/tuscany/branches/sca-java-1.2/distribution/manifest/pom.xml Thu Mar 20 00:44:42 2008
>  @@ -57,7 +57,7 @@
>          </dependency>
>          <dependency>
>              <groupId>${pom.groupId}</groupId>
>  -            <artifactId>tuscany-binding-feed</artifactId>
>  +            <artifactId>tuscany-binding-atom</artifactId>
>              <version>${pom.version}</version>
>          </dependency>
>          <dependency>
>  @@ -67,6 +67,11 @@
>          </dependency>
>          <dependency>
>              <groupId>${pom.groupId}</groupId>
>  +            <artifactId>tuscany-binding-rss</artifactId>
>  +            <version>${pom.version}</version>
>  +        </dependency>
>  +        <dependency>
>  +            <groupId>${pom.groupId}</groupId>
>              <artifactId>tuscany-binding-rss-rome</artifactId>
>              <version>${pom.version}</version>
>          </dependency>
>  @@ -343,7 +348,22 @@
>          </dependency>
>          <dependency>
>              <groupId>${pom.groupId}</groupId>
>  +            <artifactId>tuscany-node2-api</artifactId>
>  +            <version>${pom.version}</version>
>  +        </dependency>
>  +        <dependency>
>  +            <groupId>${pom.groupId}</groupId>
>              <artifactId>tuscany-node-impl</artifactId>
>  +            <version>${pom.version}</version>
>  +        </dependency>
>  +        <dependency>
>  +            <groupId>${pom.groupId}</groupId>
>  +            <artifactId>tuscany-node2-impl</artifactId>
>  +            <version>${pom.version}</version>
>  +        </dependency>
>  +        <dependency>
>  +            <groupId>${pom.groupId}</groupId>
>  +            <artifactId>tuscany-node2-launcher</artifactId>
>              <version>${pom.version}</version>
>          </dependency>
>          <dependency>
>
>  Modified: incubator/tuscany/branches/sca-java-1.2/samples/calculator-distributed/build.xml
>  URL: http://svn.apache.org/viewvc/incubator/tuscany/branches/sca-java-1.2/samples/calculator-distributed/build.xml?rev=639187&r1=639186&r2=639187&view=diff
>  ==============================================================================
>  --- incubator/tuscany/branches/sca-java-1.2/samples/calculator-distributed/build.xml (original)
>  +++ incubator/tuscany/branches/sca-java-1.2/samples/calculator-distributed/build.xml Thu Mar 20 00:44:42 2008
>  @@ -58,7 +58,7 @@
>      </target>
>
>      <target name="runNodeA">
>  -        <java classname="node.LaunchNodeA"
>  +        <java classname="node.LaunchCalculatorNodeA"
>                fork="true">
>              <classpath>
>                 <pathelement path="src/main/resources"/>
>  @@ -70,7 +70,7 @@
>      </target>
>
>      <target name="runNodeB">
>  -        <java classname="node.LaunchNodeB"
>  +        <java classname="node.LaunchCalculatorNodeB"
>                fork="true">
>              <classpath>
>                 <pathelement path="src/main/resources"/>
>  @@ -82,7 +82,7 @@
>      </target>
>
>      <target name="runNodeC">
>  -        <java classname="node.LaunchNodeC"
>  +        <java classname="node.LaunchCalculatorNodeC"
>                fork="true">
>              <classpath>
>                 <pathelement path="src/main/resources"/>
>
>
>
>  ---------------------------------------------------------------------
>  To unsubscribe, e-mail: tuscany-commits-unsubscribe@ws.apache.org
>  For additional commands, e-mail: tuscany-commits-help@ws.apache.org
>
>



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

---------------------------------------------------------------------
To unsubscribe, e-mail: tuscany-commits-unsubscribe@ws.apache.org
For additional commands, e-mail: tuscany-commits-help@ws.apache.org


Re: Feed Binding, was Re: svn commit: r639187 - in /incubator/tuscany/branches/sca-java-1.2: distribution/bundle/pom.xml distribution/manifest/pom.xml samples/calculator-distributed/build.xml

Posted by Jean-Sebastien Delfino <js...@apache.org>.
Jean-Sebastien Delfino wrote:
> Jean-Sebastien Delfino wrote:
>> Luciano Resende wrote:
>>> Thanks for volunteering !
>>>
>>> On Thu, Mar 20, 2008 at 12:12 PM, Jean-Sebastien Delfino
>>> <js...@apache.org> wrote:
>>>> Luciano Resende wrote:
>>>>  > I'm adding back the Feed binding to the distribution as there is 
>>>> still
>>>>  > one sample that uses rome atom/rss binding.
>>>>  >
>>>>
>>>>  I can fix the sample to use the new binding if that helps. Having both
>>>>  the old and new bindings is likely to cause confusion, as people will
>>>>  not know which one gets picked.
>>>>
>>>>  --
>>>>  Jean-Sebastien
>>
>> OK I'll do it this morning.
>>
>> I'm also in the middle of bringing-up the Eclipse plugin build and 
>> launcher, and running into issues with JARs missing I think and also 
>> issues communicating between the plugin and the SCA node, but I expect 
>> to fix it in the next few hours this morning too.
>>
> 
> Well, the morning extended to the afternoon :), I managed to bring up 
> the eclipse plugin and launcher but am still working on removing the 
> dependencies on the old Rome-based feed binding, as several demo modules 
> are still using it in addition to the two feed-aggregator samples.
> 

In SVN revisions r639931 (trunk) and r639933 (1.2 branch) the samples 
and demos that were using the old feed binding have been ported to the 
new atom-abdera and rss-rome bindings.

I had to fix the following issues:

- In the ATOM binding invoker, remove the calls to close client HTTP 
connections as Abdera does lazy parsing (requiring an open connection).

- Change the feed-aggregator sample to use spec-compliant ATOM feeds as 
the old ones were making Abdera unhappy (Abdera seems to be more strict 
than Rome when parsing feeds).

I also had to workaround a more serious issue with SCA reference 
injection, I'll post about it in a new thread.

-- 
Jean-Sebastien

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


Re: Feed Binding, was Re: svn commit: r639187 - in /incubator/tuscany/branches/sca-java-1.2: distribution/bundle/pom.xml distribution/manifest/pom.xml samples/calculator-distributed/build.xml

Posted by Jean-Sebastien Delfino <js...@apache.org>.
Jean-Sebastien Delfino wrote:
> Luciano Resende wrote:
>> Thanks for volunteering !
>>
>> On Thu, Mar 20, 2008 at 12:12 PM, Jean-Sebastien Delfino
>> <js...@apache.org> wrote:
>>> Luciano Resende wrote:
>>>  > I'm adding back the Feed binding to the distribution as there is 
>>> still
>>>  > one sample that uses rome atom/rss binding.
>>>  >
>>>
>>>  I can fix the sample to use the new binding if that helps. Having both
>>>  the old and new bindings is likely to cause confusion, as people will
>>>  not know which one gets picked.
>>>
>>>  --
>>>  Jean-Sebastien
> 
> OK I'll do it this morning.
> 
> I'm also in the middle of bringing-up the Eclipse plugin build and 
> launcher, and running into issues with JARs missing I think and also 
> issues communicating between the plugin and the SCA node, but I expect 
> to fix it in the next few hours this morning too.
> 

Well, the morning extended to the afternoon :), I managed to bring up 
the eclipse plugin and launcher but am still working on removing the 
dependencies on the old Rome-based feed binding, as several demo modules 
are still using it in addition to the two feed-aggregator samples.

-- 
Jean-Sebastien

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


Re: Feed Binding, was Re: svn commit: r639187 - in /incubator/tuscany/branches/sca-java-1.2: distribution/bundle/pom.xml distribution/manifest/pom.xml samples/calculator-distributed/build.xml

Posted by Jean-Sebastien Delfino <js...@apache.org>.
Luciano Resende wrote:
> Thanks for volunteering !
> 
> On Thu, Mar 20, 2008 at 12:12 PM, Jean-Sebastien Delfino
> <js...@apache.org> wrote:
>> Luciano Resende wrote:
>>  > I'm adding back the Feed binding to the distribution as there is still
>>  > one sample that uses rome atom/rss binding.
>>  >
>>
>>  I can fix the sample to use the new binding if that helps. Having both
>>  the old and new bindings is likely to cause confusion, as people will
>>  not know which one gets picked.
>>
>>  --
>>  Jean-Sebastien

OK I'll do it this morning.

I'm also in the middle of bringing-up the Eclipse plugin build and 
launcher, and running into issues with JARs missing I think and also 
issues communicating between the plugin and the SCA node, but I expect 
to fix it in the next few hours this morning too.

-- 
Jean-Sebastien

---------------------------------------------------------------------
To unsubscribe, e-mail: tuscany-commits-unsubscribe@ws.apache.org
For additional commands, e-mail: tuscany-commits-help@ws.apache.org


Re: Feed Binding, was Re: svn commit: r639187 - in /incubator/tuscany/branches/sca-java-1.2: distribution/bundle/pom.xml distribution/manifest/pom.xml samples/calculator-distributed/build.xml

Posted by Jean-Sebastien Delfino <js...@apache.org>.
Luciano Resende wrote:
> Thanks for volunteering !
> 
> On Thu, Mar 20, 2008 at 12:12 PM, Jean-Sebastien Delfino
> <js...@apache.org> wrote:
>> Luciano Resende wrote:
>>  > I'm adding back the Feed binding to the distribution as there is still
>>  > one sample that uses rome atom/rss binding.
>>  >
>>
>>  I can fix the sample to use the new binding if that helps. Having both
>>  the old and new bindings is likely to cause confusion, as people will
>>  not know which one gets picked.
>>
>>  --
>>  Jean-Sebastien

OK I'll do it this morning.

I'm also in the middle of bringing-up the Eclipse plugin build and 
launcher, and running into issues with JARs missing I think and also 
issues communicating between the plugin and the SCA node, but I expect 
to fix it in the next few hours this morning too.

-- 
Jean-Sebastien

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


Re: Feed Binding, was Re: svn commit: r639187 - in /incubator/tuscany/branches/sca-java-1.2: distribution/bundle/pom.xml distribution/manifest/pom.xml samples/calculator-distributed/build.xml

Posted by Luciano Resende <lu...@gmail.com>.
Thanks for volunteering !

On Thu, Mar 20, 2008 at 12:12 PM, Jean-Sebastien Delfino
<js...@apache.org> wrote:
> Luciano Resende wrote:
>  > I'm adding back the Feed binding to the distribution as there is still
>  > one sample that uses rome atom/rss binding.
>  >
>
>  I can fix the sample to use the new binding if that helps. Having both
>  the old and new bindings is likely to cause confusion, as people will
>  not know which one gets picked.
>
>  --
>  Jean-Sebastien
>
>
>
>  ---------------------------------------------------------------------
>  To unsubscribe, e-mail: tuscany-commits-unsubscribe@ws.apache.org
>  For additional commands, e-mail: tuscany-commits-help@ws.apache.org
>
>



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

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


Re: Feed Binding, was Re: svn commit: r639187 - in /incubator/tuscany/branches/sca-java-1.2: distribution/bundle/pom.xml distribution/manifest/pom.xml samples/calculator-distributed/build.xml

Posted by Luciano Resende <lu...@gmail.com>.
Thanks for volunteering !

On Thu, Mar 20, 2008 at 12:12 PM, Jean-Sebastien Delfino
<js...@apache.org> wrote:
> Luciano Resende wrote:
>  > I'm adding back the Feed binding to the distribution as there is still
>  > one sample that uses rome atom/rss binding.
>  >
>
>  I can fix the sample to use the new binding if that helps. Having both
>  the old and new bindings is likely to cause confusion, as people will
>  not know which one gets picked.
>
>  --
>  Jean-Sebastien
>
>
>
>  ---------------------------------------------------------------------
>  To unsubscribe, e-mail: tuscany-commits-unsubscribe@ws.apache.org
>  For additional commands, e-mail: tuscany-commits-help@ws.apache.org
>
>



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

---------------------------------------------------------------------
To unsubscribe, e-mail: tuscany-commits-unsubscribe@ws.apache.org
For additional commands, e-mail: tuscany-commits-help@ws.apache.org


Re: Feed Binding, was Re: svn commit: r639187 - in /incubator/tuscany/branches/sca-java-1.2: distribution/bundle/pom.xml distribution/manifest/pom.xml samples/calculator-distributed/build.xml

Posted by Jean-Sebastien Delfino <js...@apache.org>.
Luciano Resende wrote:
> I'm adding back the Feed binding to the distribution as there is still
> one sample that uses rome atom/rss binding.
> 

I can fix the sample to use the new binding if that helps. Having both 
the old and new bindings is likely to cause confusion, as people will 
not know which one gets picked.

-- 
Jean-Sebastien

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


Re: Feed Binding, was Re: svn commit: r639187 - in /incubator/tuscany/branches/sca-java-1.2: distribution/bundle/pom.xml distribution/manifest/pom.xml samples/calculator-distributed/build.xml

Posted by Jean-Sebastien Delfino <js...@apache.org>.
Luciano Resende wrote:
> I'm adding back the Feed binding to the distribution as there is still
> one sample that uses rome atom/rss binding.
> 

I can fix the sample to use the new binding if that helps. Having both 
the old and new bindings is likely to cause confusion, as people will 
not know which one gets picked.

-- 
Jean-Sebastien

---------------------------------------------------------------------
To unsubscribe, e-mail: tuscany-commits-unsubscribe@ws.apache.org
For additional commands, e-mail: tuscany-commits-help@ws.apache.org