[Nitro] [OG] eval and style

George Moschovitis george.moschovitis at gmail.com
Mon Feb 5 15:06:31 EST 2007


the problem was with the Aspects#wrap method, see my other post for
details...
now your patch seems to work. I will have to polish the file a little bit
more but it is definitely a step to the right direction.
I hope to see more patches from you ;-)

-g

On 2/5/07, Judson Lester <nyarly at gmail.com> wrote:
>
> That's really strange.  I can't find anywhere that og_allocate is
> defined with 0 arguments.  Can you point me to a test that drives that
> behavior?  If I could see it in a ruby on my machine, I might be abel
> to hunt it down.
>
> Judson
>
> On 2/5/07, George Moschovitis <george.moschovitis at gmail.com> wrote:
> >
> > >
> > > What sort of errors are you getting?  I know that the new write_attr
> > > wasn't working with the old adaptors - mysql.rb had to be updated.
> > > But the update is really simple.
> >
> >
> >
> > I am using mysql. this is the error I am getting:
> >
> > DEBUG: Rendering '/'.
> > DEBUG: SELECT COUNT(*) AS COUNT FROM oglink WHERE live = 1
> > DEBUG: SELECT * FROM oglink WHERE live = 1 ORDER BY create_time DESC
> LIMIT
> > 20
> > ERROR: wrong number of arguments (2 for 0)
> > ERROR:
> > /home/gmosx/code/public/og/lib/og/store/sql.rb:1114:in
> > `og_read'
> > /home/gmosx/code/public/og/lib/og/store/sql.rb:1114:in
> > `read_all'
> > /home/gmosx/code/public/og/lib/og/adapter/mysql/override.rb:17:in
> > `each_row'
> > /home/gmosx/code/public/og/lib/og/adapter/mysql/override.rb:16:in
> > `each_row'
> > /home/gmosx/code/public/og/lib/og/store/sql.rb:1112:in
> > `read_all'
> > /home/gmosx/code/public/og/lib/og/store/sql.rb:374:in
> > `find'
> > /home/gmosx/code/public/og/lib/og/entity.rb:266:in `all'
> >
> >
> > -g.
> >
> >
> > On 2/5/07, Judson Lester <nyarly at gmail.com> wrote:
> > > Feel free to benchmark it.  It probably possible to find some middle
> > > ground, where a eval-able string gets created for the class, and
> > > og_insert does the eval-ing.  Something like that.
> > >
> >
> > > Judson
> > >
> > > On 2/5/07, George Moschovitis <george.moschovitis at gmail.com> wrote:
> > > > I added your files and get errors. I will investigate your patch
> some
> > more.
> > > > On thing that worries my are the potential speed penalty of this
> patch
> > (I am
> > > > not sure there is a penalty, will have to think about this).
> > > >
> > > > thanks,
> > > > -g.
> > > >
> > > >
> > > > On 2/5/07, George Moschovitis < george.moschovitis at gmail.com> wrote:
> > > > > I was going to suggest the same to you ;-)
> > > > >
> > > > > I will add your files,
> > > > >
> > > > > thanks
> > > > >
> > > > >
> > > > >
> > > > >
> > > > > On 2/5/07, Judson Lester < nyarly at gmail.com> wrote:
> > > > > >
> > > > > > Huh.  I get the same result: I approve 11 patches to pull, and
> then
> > it
> > > > > > never returns.
> > > > > >
> > > > > > Tell you what: attached are the two files I've made changes
> to.  I'm
> > > > > > just going to trash my repo and start over, unless you have a
> better
> > > > > > idea.
> > > > > > (they're og/store/sql.rb and og/adaptor/mysql.rb)
> > > > > >
> > > > > > Judson
> > > > > >
> > > > > > _______________________________________________
> > > > > > Nitro-general mailing list
> > > > > > Nitro-general at rubyforge.org
> > > > > > http://rubyforge.org/mailman/listinfo/nitro-general
> > > > > >
> > > > > >
> > > > >
> > > > >
> > > > >
> > > > >
> > > > > --
> > > > > http://blog.gmosx.com
> > > > > http://cull.gr
> > > > > http://www.joy.gr
> > > > > http://nitroproject.org
> > > >
> > > >
> > > >
> > > > --
> > > > http://blog.gmosx.com
> > > > http://cull.gr
> > > > http://www.joy.gr
> > > > http://nitroproject.org
> > > > _______________________________________________
> > > > Nitro-general mailing list
> > > > Nitro-general at rubyforge.org
> > > > http://rubyforge.org/mailman/listinfo/nitro-general
> > > >
> > > _______________________________________________
> > > Nitro-general mailing list
> > > Nitro-general at rubyforge.org
> > > http://rubyforge.org/mailman/listinfo/nitro-general
> > >
> >
> >
> >
> > --
> > http://blog.gmosx.com
> > http://cull.gr
> > http://www.joy.gr
> > http://nitroproject.org
> > _______________________________________________
> > Nitro-general mailing list
> > Nitro-general at rubyforge.org
> > http://rubyforge.org/mailman/listinfo/nitro-general
> >
> _______________________________________________
> Nitro-general mailing list
> Nitro-general at rubyforge.org
> http://rubyforge.org/mailman/listinfo/nitro-general
>



-- 
http://blog.gmosx.com
http://cull.gr
http://www.joy.gr
http://nitroproject.org
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://rubyforge.org/pipermail/nitro-general/attachments/20070205/a867564f/attachment.html 


More information about the Nitro-general mailing list