Skip to content
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.

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

Better error output when logging cannot create folder #1063

Closed
nordmoen opened this issue May 23, 2017 · 2 comments
Closed

Better error output when logging cannot create folder #1063

nordmoen opened this issue May 23, 2017 · 2 comments

Comments

@nordmoen
Copy link
Contributor

Summary:
In the function configure_logging it would be nice if in addition to printing that the folder cannot be created print the exception being thrown.

Longer explanation:
I am having a problem where I need to run ROS on a limited environment. To allow for logging the system lets me create a folder and expose that to ROS. The problem I have been having is that when the system creates the folder it is created with different group than ROS is running under. This resulted in the situation where ROS was able to create the logging directory, but not change the folders permissions (parent folder had group root). I have used quite some time debugging why this was happening and it took some time before I found the function responsible since the folder was created successfully, but ROS still stopped with "cannot create directory".

@dirk-thomas
Copy link
Member

That sounds like a good idea. Please consider to provide a pull request for this improvement (against the latest development branch, currently lunar-devel).

@dirk-thomas
Copy link
Member

Addressed by #1068.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants