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

Pake parsing error does not indicate where error occurs #10

Open
Quasek opened this issue Mar 24, 2015 · 2 comments
Open

Pake parsing error does not indicate where error occurs #10

Quasek opened this issue Mar 24, 2015 · 2 comments
Assignees
Labels

Comments

@Quasek
Copy link
Contributor

Quasek commented Mar 24, 2015

It'd be great if pake told where an parsing error is located.

@podusowski
Copy link
Owner

We need a test case for this, because most of the time it tries to, there is a test which checks whenever pake report any kind of error on ill-formed scripts:

expecting to fail: /home/wolverine/Development/pake/__build/pake.py hello
fatal: /home/wolverine/Development/pake/tests/ErrorsInScripts/__build/001/001.pake:1: expected directive
expecting to fail: /home/wolverine/Development/pake/__build/pake.py hello
fatal: /home/wolverine/Development/pake/tests/ErrorsInScripts/__build/002/002.pake:1: unexpected (
expecting to fail: /home/wolverine/Development/pake/__build/pake.py hello
fatal: /home/wolverine/Development/pake/tests/ErrorsInScripts/__build/003/003.pake:1: unexpected literal: unknown
expecting to fail: /home/wolverine/Development/pake/__build/pake.py hello
fatal: $004.$non_existing does not exist
expecting to fail: /home/wolverine/Development/pake/__build/pake.py hello
fatal: target hello not found
expecting to fail: /home/wolverine/Development/pake/__build/pake.py hello
fatal: /home/wolverine/Development/pake/tests/ErrorsInScripts/__build/006/006.pake:1: unknown target type: dummy_target_type
expecting to fail: /home/wolverine/Development/pake/__build/pake.py hello
fatal: /home/wolverine/Development/pake/tests/ErrorsInScripts/__build/007/007.pake:1: unexpected variable: $hello
expecting to fail: /home/wolverine/Development/pake/__build/pake.py hello
fatal: expecting { after $
expecting to fail: /home/wolverine/Development/pake/__build/pake.py hello
fatal: no such module: nonexisting
expecting to fail: /home/wolverine/Development/pake/__build/pake.py hello
fatal: /home/wolverine/Development/pake/tests/ErrorsInScripts/__build/010/010.pake:1: expected variable

Currently there is this case when quoted literal is parsed and it doesn't know where it came from, this needs to be fixed.

@podusowski podusowski self-assigned this Mar 24, 2015
@Quasek
Copy link
Contributor Author

Quasek commented Mar 24, 2015

Yeah, this is the case i am talking about :P.

@podusowski podusowski added the bug label Mar 25, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants