Skip to content

Comments

fix: initialize rosout publisher on distros newer than Humble#593

Merged
esteve merged 1 commit intoros2-rust:mainfrom
esteve:rosout-test
Feb 4, 2026
Merged

fix: initialize rosout publisher on distros newer than Humble#593
esteve merged 1 commit intoros2-rust:mainfrom
esteve:rosout-test

Conversation

@esteve
Copy link
Collaborator

@esteve esteve commented Jan 28, 2026

This PR initializes the rosout publisher for distros after Humble.

Fixes #590

@esteve esteve force-pushed the rosout-test branch 3 times, most recently from 2a8abc1 to ce02a7f Compare January 28, 2026 22:16
@esteve esteve marked this pull request as ready for review January 30, 2026 22:22
@esteve esteve requested review from jhdcs and maspe36 January 30, 2026 22:22
@esteve
Copy link
Collaborator Author

esteve commented Feb 3, 2026

@EmeraldWither friendly ping 🙂 Can you check if this change fixes your issue with rosout? I believe the test covers it and I've been able to reproduce it for distros newer than Humble. Thanks.

@EmeraldWither
Copy link

Hi, thanks for taking a look at this.

This change does work on Kilted, and using the following code snippet does result in the expected behavior:

use anyhow::{Error, Result};
use rclrs::*;

fn main() -> Result<(), Error> {
    let context = Context::default_from_env()?;
    let mut executor = context.create_basic_executor();
    let node = executor.create_node("log_test_node")?;
    log!(node.error(), "This is a log message!");
    executor.spin(SpinOptions::default());
    Ok(())
}

@esteve esteve changed the title feat: enable rosout publishing fix: initialize rosout publisher on distros newer than Humble Feb 4, 2026
@esteve esteve merged commit a3fef7f into ros2-rust:main Feb 4, 2026
1 of 9 checks passed
@esteve esteve deleted the rosout-test branch February 4, 2026 21:55
@esteve
Copy link
Collaborator Author

esteve commented Feb 4, 2026

@EmeraldWither thanks for the feedback 🙂

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.

Kilted: log!() macros do not publish to "/rosout" topic

2 participants