Skip to content

Fix use of serde #9

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Oct 13, 2025
Merged

Fix use of serde #9

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
0d7e707
Fix use of serde
mxgrey Oct 10, 2025
05552fb
Include serde for services
mxgrey Oct 11, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions rosidl_generator_rs/resource/action.rs.em
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,8 @@ for subfolder, action in action_specs:
}@

pub mod rmw {
#[cfg(feature = "serde")]
use serde::{Deserialize, Serialize};
@{
TEMPLATE(
'msg_rmw.rs.em',
Expand All @@ -47,6 +49,9 @@ TEMPLATE(
}@
} // mod rmw

#[cfg(feature = "serde")]
use serde::{Deserialize, Serialize};

@{
TEMPLATE(
'msg_idiomatic.rs.em',
Expand Down
5 changes: 5 additions & 0 deletions rosidl_generator_rs/resource/msg.rs.em
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
pub mod rmw {
#[cfg(feature = "serde")]
use serde::{Deserialize, Serialize};
@{
TEMPLATE(
'msg_rmw.rs.em',
Expand All @@ -11,6 +13,9 @@ TEMPLATE(
}@
} // mod rmw

#[cfg(feature = "serde")]
use serde::{Deserialize, Serialize};

@{
TEMPLATE(
'msg_idiomatic.rs.em',
Expand Down
2 changes: 0 additions & 2 deletions rosidl_generator_rs/resource/msg_idiomatic.rs.em
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,6 @@ from rosidl_parser.definition import UnboundedWString
@# ############ Idiomatic message types ############
@# #################################################
@# These types use standard Rust containers where possible.
#[cfg(feature = "serde")]
use serde::{Deserialize, Serialize};
@[for subfolder, msg_spec in msg_specs]@
@{
type_name = msg_spec.structure.namespaced_type.name
Expand Down
2 changes: 0 additions & 2 deletions rosidl_generator_rs/resource/msg_rmw.rs.em
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,6 @@ from rosidl_parser.definition import UnboundedSequence
from rosidl_parser.definition import UnboundedString
from rosidl_parser.definition import UnboundedWString
}@
#[cfg(feature = "serde")]
use serde::{Deserialize, Serialize};
@[for subfolder, msg_spec in msg_specs]@
@{
type_name = msg_spec.structure.namespaced_type.name
Expand Down
5 changes: 5 additions & 0 deletions rosidl_generator_rs/resource/srv.rs.em
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,3 +1,6 @@
#[cfg(feature = "serde")]
use serde::{Deserialize, Serialize};

@{
TEMPLATE(
'srv_idiomatic.rs.em',
Expand All @@ -10,6 +13,8 @@ TEMPLATE(
}

pub mod rmw {
#[cfg(feature = "serde")]
use serde::{Deserialize, Serialize};
@{
TEMPLATE(
'srv_rmw.rs.em',
Expand Down