Re: [Orocos-users] [Orocos] Orocos Toolchain 2.6.0 Released !

Hi Paul,

On Tue, Dec 11, 2012 at 11:09 AM, Paul Chavent <[email protected]> wrote:
> Hello Peter.
>
> Some words about the last release :
>
> (1) Thank you for having managed this new release.
>
> (2) The downloads of the archives didn't work today : http://people.mech.kuleuven.be/~orocos/pub/stable/toolchain/v2.6.0/orocos-toolchain-2.6.0-src.tar.bz2 returns 404. Is it still possible to access independent archives of rtt and ocl ?

There is an issue with the file server which caused all our new
uploads to be not available on the webserver. We've reported it and
hope it will be resolved soon. You can get rtt and ocl as archives
from github or gitorious. The complete toolchain we could try to
upload to a different place and see if we can manually move it on the
webserver...

>
> (3) I'm sorry, but i have submitted a wrong patch for the bug 979. I thought that you have read my comment http://bugs.orocos.org/show_bug.cgi?id=979#c6 , but the master repo still contains the error (a class keyword to replace by a struct).

I did, but then I forgot.. patch is king  :-) Fixed on gitorious.

>
> (4) The bug 950 have been partially merged. I added some modifications (see http://bugs.orocos.org/show_bug.cgi?id=950#c8). Do you have some reviews on it ?

What really sucks is that the rttlua deployer is called 'deployer' and
the taskbrowser deployer is called 'Deployer'.  Changing either of
both is going to break a lot of scripts.

Especially since every rttlua script contains a line like 'dp =
tc:getPeer("deployer")'

What we could do is try to use the alias mechanism such that
'deployer' and 'Deployer' resolve to the same TC, *but keep the
official name as 'deployer' in rttlua*.

For example (note that the DC is created as "deployer"):

diff --git a/lua/LuaComponent.cpp b/lua/LuaComponent.cpp
index 0a56197..22d03b9 100644
--- a/lua/LuaComponent.cpp
+++ b/lua/LuaComponent.cpp
@@ -345,6 +345,8 @@ int ORO_main(int argc, char** argv)
        dc = new DeploymentComponent("deployer");

        lua.connectPeers(dc);
+    // Alias for rttscript compatibility.
+    lua.addPeer(dc, "Deployer");

        /* run init file */
        wordexp(INIT_FILE, &init_exp, 0);

Would this work for you ? Or is it also for the
CorbaDeploymentComponent + NamingService necessary ? How do you set
the the name of different CorbaDeploymentComponents ???

>
> (3) The bug 926 haven't been merged. Do you have suggestions to make the patch acceptable ?

It was. I just assumed it had been merged already, and somehow also
overlooked the bug report. This is annoying since it adds a virtual
function to the ChannelElements. Is this patch in use on the rock-rtt
repository ?

>
> (5) I think you can close bug 853

Thanks.

Peter
-- 
Orocos-Dev mailing list
[email protected]
http://lists.mech.kuleuven.be/mailman/listinfo/orocos-dev