Re: rFSM catkinization

On Fri, May 16, 2014 at 4:14 PM, Sylvain Joyeux <[email protected]>wrote:

> > I'm not very enthusiastic about this patch, IMNSHO it's plain wrong that
>> > a generic software library needs to be changed for use in a specific
>> > framework.
>>
>> I fully agree! This is an unmaintainable path...
>>
> It is obviously unmaintainable as the orocos packages already migrated
> into rosbuild, then from rosbuild to catkin for the sake of ROS
> compatibility. But it seems that spending time on build systems is fun
> [/sarcasm]
>

I'd say that the intrusiveness of ROS compatibility has improved
substantially since rosbuild. If adding the package.xml file to an upstream
repository is undesirable, bloom also allows to "inject" it into the
release (not upstream) repository. More info here:

http://wiki.ros.org/bloom/Tutorials/ReleaseThirdParty#Modifying_the_Release_Repository

Cheers,

Adolfo.


>
> Sylvain
>
>
> 2014-02-14 9:39 GMT+01:00 Herman Bruyninckx <
> [email protected]>:
>
>> On Fri, 14 Feb 2014, Markus Klotzbuecher wrote:
>>
>> > Hi Ruben,
>> >
>> > On Do, Feb 13, 2014 at 04:28:47 +0100, Ruben Smits wrote:
>> >
>> >> we've cleanup our fork to remove the ugly guard changes we did. But we
>> >> did do a catkinization of rFSM. It just adds functionality, we do not
>> >> break any old use-cases. It just makes it possible now to include rFSM
>> >> in a catkin workspace and have all the LUA paths set up automatically.
>> >
>> > I'm not very enthusiastic about this patch, IMNSHO it's plain wrong that
>> > a generic software library needs to be changed for use in a specific
>> > framework.
>>
>> I fully agree! This is an unmaintainable path...
>>
>> >> Using this patch we can also seamlessly release rFSM into the ROS
>> >> ecosystem. (apt-get install ros-hydro-rFSM)
>> >>
>> >> pull request is available here: https://github.com/kmarkus/rFSM/pull/4
>> >
>> > I've merged it, since it seems to make your and other users life
>> > easier. But I think the long term goal should be to have a generic
>> > non-ros debian package.
>> >
>> > Markus
>>
>> Herman
>> --
>> Orocos-Dev mailing list
>> [email protected]
>> http://lists.mech.kuleuven.be/mailman/listinfo/orocos-dev
>>
>
>
> --
> Orocos-Dev mailing list
> [email protected]
> http://lists.mech.kuleuven.be/mailman/listinfo/orocos-dev
>
>


-- 
Adolfo Rodríguez Tsouroukdissian
Senior robotics engineer
[email protected]
http://www.pal-robotics.com

PAL ROBOTICS S.L
c/ Pujades 77-79, 4º4ª
08005 Barcelona, Spain.
Tel. +34.93.414.53.47
Fax.+34.93.209.11.09
Skype: adolfo.pal-robotics
Facebook <http://www.facebook.com/palrobotics1> -
Twitter<http://twitter.com/#%21/palrobotics>- PAL
Robotics YouTube Channel <http://www.youtube.com/user/PALRobotics>

AVISO DE CONFIDENCIALIDAD: Este mensaje y sus documentos adjuntos, pueden
contener información privilegiada y/o confidencial que está dirigida
exclusivamente a su destinatario. Si usted recibe este mensaje y no es el
destinatario indicado, o el empleado encargado de su entrega a dicha
persona, por favor, notifíquelo inmediatamente y remita el mensaje original
a la dirección de correo electrónico indicada. Cualquier copia, uso o
distribución no autorizados de esta comunicación queda estrictamente
prohibida.

CONFIDENTIALITY NOTICE: This e-mail and the accompanying document(s) may
contain confidential information which is privileged and intended only for
the individual or entity to whom they are addressed.  If you are not the
intended recipient, you are hereby notified that any disclosure, copying,
distribution or use of this e-mail and/or accompanying document(s) is
strictly prohibited.  If you have received this e-mail in error, please
immediately notify the sender at the above e-mail address.

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