-
Notifications
You must be signed in to change notification settings - Fork 477
Error running 0.12.87 with --genpoi #1271
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
Comments
Perhaps another instance of overviewer is running? Something is holding open the yuid cache file |
Definitely only one instance running. And I immediately reverted to 0.12.82 and re-ran Overviewer in the same command window with no problems. .82 to .87 involved changing the cache processing, so maybe that's where the problem lies. |
Previously, the files were not closed after reading or writing; by using a "with" statement, the file handles will be closed as soon as they go out of scope. Possible fix for #1271.
I just pushed a change which may fix this, let us know if the problem still persists. Thanks for your bug report. |
Sorry for the bad news, but running Win 64-bit 12.90 gets the following error. (EDIT: This error is different than the original error).
|
Oh, here it is. I am getting this same script. OverViewer 0.12.93 Minecraft 1.9 Microsoft Windows [Version 6.1.7601] C:\Users\Shi>cd c:\Users\Shi\Desktop\overviewer c:\Users\Shi\Desktop\overviewer>overviewer.exe --config=C:/Users/Shi/Desktop/ove This is the error that occurred: c:\Users\Shi\Desktop\overviewer> |
I was having this problem as well. I resolved it by renaming uuidcache.dat. Unfortunately the file must be renamed/removed before each run or the error will still occur. |
It appears to be an issue with how Windows does atomic file renames, I'll try to look into it soon™. |
Still an issue in version 0.12.137. |
I had the same problem on my server. IIS had changed the file permissions of several files to not allow overwrite. Since I didn't want to check hundreds of files, I deleted the entire output folder content and re-rendered the map. Solved my problem and subsequent passes. |
@LeadMagnet I'm fairly sure this is a different issue though, related to atomic file renames, and nothing to do with permissions. |
With the [Error 183] can we get the name of the file/files that are causing the error, maybe we can delete them manually. |
It happens whenever you try to rename a file to another file which already exists, on Windows. |
The fix I just pushed should work, but since we currently don't have automatic Windows builds and I'm not feeling like getting my dev env set up on Windows again, I can't confirm that it works right now. If someone on Windows can build from git and show that it's still broken, I'll re-open. |
@eminence hooked me up with an up-to-date Windows build from the not-yet-finished new build infrastructure and I can confirm that this bug is indeed fixed now. Hooray! |
Windows 8.1 64-bit overviewer-0.12.87.zip gets the following error with --genpoi:
The following is the output with version 0.12.82:
The text was updated successfully, but these errors were encountered: