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

Include errno (and strerror) in sys errors #5193

Closed
lucasb-eyer opened this issue Dec 19, 2013 · 2 comments
Closed

Include errno (and strerror) in sys errors #5193

lucasb-eyer opened this issue Dec 19, 2013 · 2 comments

Comments

@lucasb-eyer
Copy link
Contributor

Hi,

Today I spent considerable amount of time trying to figure out why almost all julia fs functions failed after a certain (nondeterministic) point in my program. The answer lies in errno: 24: "Too many files open".

If all functions included errno info in those errors which follow from a C call, this would have been a no-brainer. One example is readdir, just doing error("unable to read directory $path") instead of providing additional useful errno diagnostic.

I could change that and other occurrences to systemerror("unable to read directory $path", file_count < 0), but before I spend time on this, I'd like to know if it is at all wanted?

@StefanKarpinski
Copy link
Sponsor Member

Yes, this would be great. I've tried to be systematic (get it?) about using systemerror wherever appropriate, but there's a lot of uses in Base that don't have this. Improving that would be a valuable contribution.

@lucasb-eyer
Copy link
Contributor Author

Guess we can close this now.

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

No branches or pull requests

2 participants