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

pdal command line mistake doesn't return EXIT_FAILURE #3354

Closed
schwehr opened this issue Feb 8, 2021 · 1 comment · Fixed by #3355
Closed

pdal command line mistake doesn't return EXIT_FAILURE #3354

schwehr opened this issue Feb 8, 2021 · 1 comment · Fixed by #3355
Milestone
2.3.0

Comments

@schwehr
Copy link
Contributor

schwehr commented Feb 8, 2021

Using a debian_testing build from head.

This looks good.

$ ./pdal info /does/not/exist ; echo $?
PDAL: Cannot determine reader for input file: /does/not/exist

1

I would expect a failure code (e.g. 1).

$ ./pdal /does/not/exist ; echo $?
(PDAL Error) Command '/does/not/exist' not recognized

0
@hobu hobu added this to the 2.3 milestone Feb 8, 2021
abellgithub added a commit that referenced this issue Feb 9, 2021
@abellgithub
Return an error code if no command is provided.
ed8ab9f 
Close #3354
@abellgithub abellgithub mentioned this issue Feb 9, 2021
Merged
abellgithub added a commit that referenced this issue Feb 17, 2021
@abellgithub
Return an error code if no command is provided. (#3355)
ede0e8c 
Close #3354
@schwehr
Copy link
Contributor Author

schwehr commented Feb 17, 2021

@abellgithub Thank you!

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

Successfully merging a pull request may close this issue.

Return an error code if no command is provided. PDAL/PDAL
2 participants
@hobu @schwehr