For more infomation about this bug, visit

Peter Soetens
<peter [dot] soetens [..] ...> changed:

What |Removed |Added
--------------------------------------------------------------------------
Status|REOPENED |RESOLVED
Resolution| |FIXED

--- Comment #6 from Peter Soetens
<peter [dot] soetens [..] ...> 2008-05-15 23:37:28 ---
Recent changes in the deployer required updateProperties usage for merging the
new XML file into the current property structure. updateProperties did not
support duplicate names (like "StateMachineScript") in the target or source
bag.

$ svn ci PropertyBag.hpp PropertyBag.cpp -m"Fix bug 510: loading multiple
programscripts and statemachines
Rewrite updateProperties such that a PropertyBag with n times the same
propertyname is updated correctly n times, instead of all n being projected to
a single property in the target bag."
Sending PropertyBag.cpp
Sending PropertyBag.hpp
Transmitting file data ..
Committed revision 29286.

For more infomation about this bug, visit

Ruben Smits <ruben [dot] smits [..] ...> changed:

What |Removed |Added
--------------------------------------------------------------------------
Status|RESOLVED |REOPENED
Resolution|FIXED |

--- Comment #5 from Ruben Smits <ruben [dot] smits [..] ...> 2008-05-15 09:21:32 ---
Apparently this functionality is removed from trunk.

Ruben

For more infomation about this bug, visit

Peter Soetens
<peter [dot] soetens [..] ...> changed:

What |Removed |Added
--------------------------------------------------------------------------
Status|ASSIGNED |RESOLVED
Resolution| |FIXED

--- Comment #4 from Peter Soetens
<peter [dot] soetens [..] ...> 2008-02-22 13:07:20 ---
Patch applied on trunk/ocl

$ svn ci deployment -m"Fixes bug #510: loading multiple programscripts,
statemachines and property files."
Sending deployment/DeploymentComponent.cpp
Transmitting file data .
Committed revision 28993.

For more infomation about this bug, visit

Peter Soetens
<peter [dot] soetens [..] ...> changed:

What |Removed |Added
--------------------------------------------------------------------------
AssignedTo|orocos- |peter [dot] soetens [..] ...
|dev [..] ... |

--- Comment #3 from Peter Soetens
<peter [dot] soetens [..] ...> 2008-02-22 13:05:27 ---
Created an attachment (id=241)
--> (https://www.fmtc.be/bugzilla/orocos/attachment.cgi?id=241)
Fixes bug.

This patch allows to load multiple scripts, in the order as they are listed.
Multiple property files can be loaded as well, and the 'UpdateProperties'
(compare with 'PropertyFile') element has been added which allows to configure
only some properties of a component (analogous to
marshalling.updateProperties() ). The 'PropertyFile' element enforces that all
component properties are listed in the XML file.

For more infomation about this bug, visit

Peter Soetens
<peter [dot] soetens [..] ...> changed:

What |Removed |Added
--------------------------------------------------------------------------
Summary|loading multiple |loading multiple
|programscripts and |programscripts,
|statemachines |statemachines and property
| |files

--- Comment #2 from Peter Soetens
<peter [dot] soetens [..] ...> 2008-02-22 13:02:49 ---
The same problem occurs when specifying multiple property files: a component
can only be configured from one XML file.

For more infomation about this bug, visit

Peter Soetens
<peter [dot] soetens [..] ...> changed:

What |Removed |Added
--------------------------------------------------------------------------
Status|NEW |ASSIGNED
CC| |peter [dot] soetens [..] ...
Target Milestone|--- |1.4.1

--- Comment #1 from Peter Soetens
<peter [dot] soetens [..] ...> 2008-02-13 17:06:52 ---
(In reply to comment #0)
> Apparently the deployer only supports the loading of only one statemachine and
> one programscript for each component.
>
> If you add more ProgramScripts/StateMachines in the xml-configuration, only the
> first one is loaded.

When I implemented this, I naively thought that 'one state machine would be
enough for every component', guided by the 'one state machine can load another'
principle. But I understand now that this is flawed for various reasons.