Skip to content

Humble Backport: Fix registration of types when comming from C Type Support (#35)#37

Draft
Mario-DL wants to merge 6 commits intovulcanexus-humblefrom
backport/fix-registration-c-typesupport
Draft

Humble Backport: Fix registration of types when comming from C Type Support (#35)#37
Mario-DL wants to merge 6 commits intovulcanexus-humblefrom
backport/fix-registration-c-typesupport

Conversation

@Mario-DL
Copy link

No description provided.

jsan-rt and others added 6 commits June 21, 2024 12:29
Signed-off-by: Javier Santiago <javiersantiago@eprosima.com>
* Refs #20596: Cherry pick Keys Feature from iron

* Refs #20164: rmw_fastrtps_shared: prepare api for keys support

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

* Refs #20164: rmw_fastrtps_shared: getKey() implementation

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

* Refs #20164: rmw_fastrtps* interface methods update for keys support

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

* Refs #20164: rmw_fastrtps_cpp empty implementation

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

* Refs #20164: rmw_fastrtps_dynamic_cpp Message & Service Typesupport construction updates

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

* Refs #20164: rmw_fastrtps_dynamic_cpp type_support_proxy updates implementation

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

* Refs #20164: rmw_fastrtps_dynamic_cpp type_support implementation

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

* Refs #20164: Move typesupport key vars to rmw_fastrtps_shared_cpp

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

* Refs #20164: Initialize Typesupport key members in rmw_fastrtps_cpp

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

* Refs #20310: rmw_fastrtps_shared_cpp: add abi_version enum and type support member

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

* Refs #20310: rmw_fastrtps_cpp: adopt abi version v2

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

* Refs #20310: rmw_fastrtps_dynamic_cpp: adopt abi version v2

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

* Refs #20310: rmw_fastrtps_shared_cpp: apply_qos_resource_limits_for_keys() free function

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

* Refs #20310: rmw_fastrtps_cpp: apply qos for keys

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

* Refs #20310: rmw_fastrtps_dynamic_cpp: apply qos for keys

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

* Refs #20310: Review suggestions

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

* Refs #20310: fix: include checking for the v2 identifier when trying to register the typesupport_introspection. This is useful when builtin types are also built with v2 abi and rmw_fastrtps_cpp is used

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

* Refs #20310: rmw_fastrtps_cpp review 2 suggestion

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

* Refs #20310: Updates according latest message_type_support structure (no initial alignment)

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

* Refs #20310: Compile for Fast CDR 1 only

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

* Refs #20310: Comment typo

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

---------

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

* Refs #20596: Adapt rmw_fastrtps_shared_cpp: adapt getKey() method to humble

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>

---------

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>
Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>
Signed-off-by: eduponz <eduardoponz@eprosima.com>
Signed-off-by: eduponz <eduardoponz@eprosima.com>
…Support (#35)

Signed-off-by: Mario Dominguez <mariodominguez@eprosima.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants