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

Error in mb-util #111

Open
mrg218 opened this issue Sep 4, 2019 · 0 comments
Open

Error in mb-util #111

mrg218 opened this issue Sep 4, 2019 · 0 comments

Comments

@mrg218
Copy link

mrg218 commented Sep 4, 2019

I am pretty sure that the line of code in mb-util that is now

if os.path.isdir(args[0]) and not os.path.isfile(args[0]):

should be:

if os.path.isdir(args[0]) and not os.path.isfile(args[1]):

Weird that no one has noticed this while using. It now checks if a directory is not a file, while it should check if the second argument is not an existing mbtiles file.

What is the reason that updates of mbtiles file is now allowed?

@mrg218 mrg218 changed the title But in mb-util Errorin mb-util Sep 4, 2019
@mrg218 mrg218 changed the title Errorin mb-util Error in mb-util Sep 4, 2019
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

No branches or pull requests

1 participant