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.

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

Jump to bottom

Improve error message for .cfg parsing #3

Closed
baagaard-usgs opened this issue Jun 11, 2019 · 1 comment
Closed

Improve error message for .cfg parsing #3

baagaard-usgs opened this issue Jun 11, 2019 · 1 comment
Labels
deficiency Undesirable behavior or annoying

Comments

@baagaard-usgs
Copy link
Contributor

Having an inadvertent space at the beginning of a line in a .cfg file specifying a facility (not a property) generates a very vague, difficult to understand error message:

 >> {command line}::
 >> pyre.inventory(error)
 -- 'str' object has no attribute 'append'
pylithapp: configuration error(s)

This could be an error in generating the error message.
@baagaard-usgs baagaard-usgs added the deficiency Undesirable behavior or annoying label Jun 11, 2019
@baagaard-usgs baagaard-usgs changed the title Improve error mesh for .cfg parsing Improve error message for .cfg parsing Jul 1, 2019
@baagaard-usgs
Copy link
Contributor Author

Fixed in 569e556.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
deficiency Undesirable behavior or annoying
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@baagaard-usgs