Skip to content
This repository was archived by the owner on May 31, 2025. It is now read-only.

[rosgraph] now allowing python_logging.yaml for logging configuration#1061

Merged
dirk-thomas merged 2 commits intoros:lunar-develfrom
asmodehn:yaml_logging_conf_lunar
Jul 17, 2017
Merged

[rosgraph] now allowing python_logging.yaml for logging configuration#1061
dirk-thomas merged 2 commits intoros:lunar-develfrom
asmodehn:yaml_logging_conf_lunar

Conversation

@asmodehn
Copy link
Copy Markdown
Contributor

#1033 rebased on lunar.

@dirk-thomas
Copy link
Copy Markdown
Member

Thank you!

Please consider adding some documentation about this new feature in Lunar to the wiki and post a link to the diff here.

@dirk-thomas dirk-thomas merged commit 7af8c60 into ros:lunar-devel Jul 17, 2017
sputnick1124 pushed a commit to sputnick1124/ros_comm that referenced this pull request Jul 30, 2017
…ros#1061)

* [rosgraph] now allowing python_logging.yaml for logging configuration

* get package path only once, minor style
@dirk-thomas
Copy link
Copy Markdown
Member

@asmodehn Did you have a chance to document this new feature in the wiki? Please post a link to the diff here for future reference.

@asmodehn
Copy link
Copy Markdown
Contributor Author

Well I thought I did, but it is obviously not there... I ll add something about it soon.

@asmodehn
Copy link
Copy Markdown
Contributor Author

done.

@dirk-thomas
Copy link
Copy Markdown
Member

@dirk-thomas
Copy link
Copy Markdown
Member

See #1292 on how the lookup order was changed later.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants