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

Change if statement for $SRC_DIR #58

Closed
wants to merge 1 commit into from

Conversation

mikenye
Copy link
Contributor

@mikenye mikenye commented Jul 30, 2020

I was having an issue where tar1090 would output "source directory (2nd argument) not specified, fatal error!" when the 2nd argument was given, and was a directory.

Changing the if statement as-per the commit 462c855 seems to have solved this.

The original behaviour (displaying the error and exiting if the user does not provide the 2nd argument) still works.

One futher change could be adjusting the error message to something like "source directory (2nd argument) not specified or does not exist, fatal error!", however I will leave this up to you.

Thanks!

@wiedehopf wiedehopf closed this Jul 30, 2020
@wiedehopf
Copy link
Owner

not the intent.

gonna fix another way.

@wiedehopf
Copy link
Owner

fixed.

@mikenye mikenye deleted the mikenye-patch-1 branch July 30, 2020 10:04
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

Successfully merging this pull request may close these issues.

None yet

2 participants