You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@shindig.apache.org by do...@apache.org on 2008/02/22 03:52:20 UTC

svn commit: r630074 - /incubator/shindig/trunk/features/opensocial-reference/feature.xml

Author: doll
Date: Thu Feb 21 18:52:10 2008
New Revision: 630074

URL: http://svn.apache.org/viewvc?rev=630074&view=rev
Log:
Removed accidental dependency of opensocial-reference on ifpc. If your opensocial-0.7 feature needs this dependency you should add it to your feature.xml file.  


Modified:
    incubator/shindig/trunk/features/opensocial-reference/feature.xml

Modified: incubator/shindig/trunk/features/opensocial-reference/feature.xml
URL: http://svn.apache.org/viewvc/incubator/shindig/trunk/features/opensocial-reference/feature.xml?rev=630074&r1=630073&r2=630074&view=diff
==============================================================================
--- incubator/shindig/trunk/features/opensocial-reference/feature.xml (original)
+++ incubator/shindig/trunk/features/opensocial-reference/feature.xml Thu Feb 21 18:52:10 2008
@@ -19,8 +19,6 @@
 -->
 <feature>
   <name>opensocial-reference</name>
-	<!-- all known opensocial implementations assume that ifpc is present. -->
-	<dependency>ifpc</dependency>
   <gadget>
     <script src="opensocial.js"/>
     <script src="activity.js"/>



Re: svn commit: r630074 - /incubator/shindig/trunk/features/opensocial-reference/feature.xml

Posted by Kevin Brown <et...@google.com>.
It might happen already because of a bug, but I'll look into it as soon as I
get a chance.

On Thu, Feb 21, 2008 at 7:05 PM, Cassie <do...@apache.org> wrote:

> Very good point... do you think you can make that happen? :)
>
>
> On Thu, Feb 21, 2008 at 7:03 PM, Kevin Brown <et...@google.com> wrote:
>
> > We probably need to make it so that you can overwrite an existing
> feature
> > when calling registerFeature; as well -- otherwise the new
> opensocial-0.7is
> > going to conflict with existing opensocial implementations, and
> > maintaining
> > patches sucks. This will go away as the opensocial feature
> implementation
> > is
> > fleshed out and people migrate to actually using it for their real data,
> > of
> > course.
> >
> > On Thu, Feb 21, 2008 at 6:52 PM, <do...@apache.org> wrote:
> >
> > > Author: doll
> > > Date: Thu Feb 21 18:52:10 2008
> > > New Revision: 630074
> > >
> > > URL: http://svn.apache.org/viewvc?rev=630074&view=rev
> > > Log:
> > > Removed accidental dependency of opensocial-reference on ifpc. If your
> > > opensocial-0.7 feature needs this dependency you should add it to your
> > > feature.xml file.
> > >
> > >
> > > Modified:
> > >    incubator/shindig/trunk/features/opensocial-reference/feature.xml
> > >
> > > Modified:
> > > incubator/shindig/trunk/features/opensocial-reference/feature.xml
> > > URL:
> > >
> >
> http://svn.apache.org/viewvc/incubator/shindig/trunk/features/opensocial-reference/feature.xml?rev=630074&r1=630073&r2=630074&view=diff
> > >
> > >
> >
> ==============================================================================
> > > --- incubator/shindig/trunk/features/opensocial-reference/feature.xml
> > > (original)
> > > +++ incubator/shindig/trunk/features/opensocial-reference/feature.xml
> > Thu
> > > Feb 21 18:52:10 2008
> > > @@ -19,8 +19,6 @@
> > >  -->
> > >  <feature>
> > >   <name>opensocial-reference</name>
> > > -       <!-- all known opensocial implementations assume that ifpc is
> > > present. -->
> > > -       <dependency>ifpc</dependency>
> > >   <gadget>
> > >     <script src="opensocial.js"/>
> > >     <script src="activity.js"/>
> > >
> > >
> > >
> >
> >
> > --
> > ~Kevin
> >
> > If you received this email by mistake, please delete it, cancel your
> mail
> > account, destroy your hard drive, silence any witnesses, and burn down
> the
> > building that you're in.
> >
>



-- 
~Kevin

If you received this email by mistake, please delete it, cancel your mail
account, destroy your hard drive, silence any witnesses, and burn down the
building that you're in.

Re: svn commit: r630074 - /incubator/shindig/trunk/features/opensocial-reference/feature.xml

Posted by Cassie <do...@apache.org>.
Very good point... do you think you can make that happen? :)


On Thu, Feb 21, 2008 at 7:03 PM, Kevin Brown <et...@google.com> wrote:

> We probably need to make it so that you can overwrite an existing feature
> when calling registerFeature; as well -- otherwise the new opensocial-0.7is
> going to conflict with existing opensocial implementations, and
> maintaining
> patches sucks. This will go away as the opensocial feature implementation
> is
> fleshed out and people migrate to actually using it for their real data,
> of
> course.
>
> On Thu, Feb 21, 2008 at 6:52 PM, <do...@apache.org> wrote:
>
> > Author: doll
> > Date: Thu Feb 21 18:52:10 2008
> > New Revision: 630074
> >
> > URL: http://svn.apache.org/viewvc?rev=630074&view=rev
> > Log:
> > Removed accidental dependency of opensocial-reference on ifpc. If your
> > opensocial-0.7 feature needs this dependency you should add it to your
> > feature.xml file.
> >
> >
> > Modified:
> >    incubator/shindig/trunk/features/opensocial-reference/feature.xml
> >
> > Modified:
> > incubator/shindig/trunk/features/opensocial-reference/feature.xml
> > URL:
> >
> http://svn.apache.org/viewvc/incubator/shindig/trunk/features/opensocial-reference/feature.xml?rev=630074&r1=630073&r2=630074&view=diff
> >
> >
> ==============================================================================
> > --- incubator/shindig/trunk/features/opensocial-reference/feature.xml
> > (original)
> > +++ incubator/shindig/trunk/features/opensocial-reference/feature.xml
> Thu
> > Feb 21 18:52:10 2008
> > @@ -19,8 +19,6 @@
> >  -->
> >  <feature>
> >   <name>opensocial-reference</name>
> > -       <!-- all known opensocial implementations assume that ifpc is
> > present. -->
> > -       <dependency>ifpc</dependency>
> >   <gadget>
> >     <script src="opensocial.js"/>
> >     <script src="activity.js"/>
> >
> >
> >
>
>
> --
> ~Kevin
>
> If you received this email by mistake, please delete it, cancel your mail
> account, destroy your hard drive, silence any witnesses, and burn down the
> building that you're in.
>

Re: svn commit: r630074 - /incubator/shindig/trunk/features/opensocial-reference/feature.xml

Posted by Kevin Brown <et...@google.com>.
We probably need to make it so that you can overwrite an existing feature
when calling registerFeature; as well -- otherwise the new opensocial-0.7 is
going to conflict with existing opensocial implementations, and maintaining
patches sucks. This will go away as the opensocial feature implementation is
fleshed out and people migrate to actually using it for their real data, of
course.

On Thu, Feb 21, 2008 at 6:52 PM, <do...@apache.org> wrote:

> Author: doll
> Date: Thu Feb 21 18:52:10 2008
> New Revision: 630074
>
> URL: http://svn.apache.org/viewvc?rev=630074&view=rev
> Log:
> Removed accidental dependency of opensocial-reference on ifpc. If your
> opensocial-0.7 feature needs this dependency you should add it to your
> feature.xml file.
>
>
> Modified:
>    incubator/shindig/trunk/features/opensocial-reference/feature.xml
>
> Modified:
> incubator/shindig/trunk/features/opensocial-reference/feature.xml
> URL:
> http://svn.apache.org/viewvc/incubator/shindig/trunk/features/opensocial-reference/feature.xml?rev=630074&r1=630073&r2=630074&view=diff
>
> ==============================================================================
> --- incubator/shindig/trunk/features/opensocial-reference/feature.xml
> (original)
> +++ incubator/shindig/trunk/features/opensocial-reference/feature.xml Thu
> Feb 21 18:52:10 2008
> @@ -19,8 +19,6 @@
>  -->
>  <feature>
>   <name>opensocial-reference</name>
> -       <!-- all known opensocial implementations assume that ifpc is
> present. -->
> -       <dependency>ifpc</dependency>
>   <gadget>
>     <script src="opensocial.js"/>
>     <script src="activity.js"/>
>
>
>


-- 
~Kevin

If you received this email by mistake, please delete it, cancel your mail
account, destroy your hard drive, silence any witnesses, and burn down the
building that you're in.