Skip to content

Check that sphdistance -Q is given a file#7619

Merged
PaulWessel merged 2 commits intomasterfrom
sph-Q-file
Jul 7, 2023
Merged

Check that sphdistance -Q is given a file#7619
PaulWessel merged 2 commits intomasterfrom
sph-Q-file

Conversation

@PaulWessel
Copy link
Member

Somehow forgot to use gmt_get_required_file for the -Q option. Now we get message like

sphdistance [ERROR]: Option -Q: No argument provided

instead of

sphdistance [ERROR]: Cannot find file 
sphdistance [ERROR]: Cannot find file 
sphdistance [ERROR]: File  not found
[Session gmt (0)]: Error returned from GMT API: GMT_FILE_NOT_FOUND (16)

Somehow forgot to use gmt_get_required_file for the -Q option.
@PaulWessel PaulWessel added enhancement Improving an existing feature maintenance Boring but important stuff for the core devs labels Jul 7, 2023
@PaulWessel PaulWessel added this to the 6.5.0 milestone Jul 7, 2023
@PaulWessel PaulWessel requested a review from joa-quim July 7, 2023 13:30
@PaulWessel PaulWessel self-assigned this Jul 7, 2023
@PaulWessel PaulWessel merged commit 5d0f7c6 into master Jul 7, 2023
@PaulWessel PaulWessel deleted the sph-Q-file branch July 7, 2023 13:40
@maxrjones maxrjones added the add-changelog Add PR to the changelog label Dec 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

add-changelog Add PR to the changelog enhancement Improving an existing feature maintenance Boring but important stuff for the core devs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants