file writing access error

[MinnieTheMoocher]

I am running Blender 2.69 32bit on Windows 7.
I unpacked its .zip file to C:\Program Files (x86)\Blender 2.69
and can run it fine there.

I added the LDR Importer script to it and can execute it successfully.

However, I cannot import files using it.
The script seems to try to write to files below the C:\Program Files (x86) folder,
which it isn't allowed to. Here is the output Blender generates on stdout:

Traceback (most recent call last):
File "c:\Program Files (x86)\Blender 2.69\2.69\scripts\addons\import_ldraw.py", line 1083, in execute
saveInstallPath(self)
File "c:\Program Files (x86)\Blender 2.69\2.69\scripts\addons\import_ldraw.py", line 1101, in saveInstallPath
os.makedirs(config_path)
File "c:\Program Files (x86)\Blender 2.69\2.69\python\lib\os.py", line 266, in makedirs
mkdir(name, mode)
PermissionError: [WinError 5] Access is denied: 'c:\Program Files (x86)\Blender 2.69\2.69\scripts\presets\io_import_ldraw'

I think the script needs some fixing. On Windows, it should not try to write to the tool installation path. Instead, a tool's options are stored in a special folder per-user, and that is below

C:\Users\YourUserNameGoesHere\AppData\Local\LDRImporter

or

C:\Users\YourUserNameGoesHere\AppData\Local\Blender

As a workaround, I am moving my Blender installation folder outside of
C:\Program Files (x86), but that's not something I really want to do.
All my programs shall be below that.

[Banbury]

Blender plugins should be installed in:

c:\Users\Username\AppData\Roaming\Blender Foundation\Blender\2.x\scripts\addons\

[le717]

Hello!

Thank you for the bug report. I implemented the patch to save the configuration where it did, but failed to think what may happen if the user installs it to Program Files. My bad.

I'll get to fixing this. It may not be this week, but I'll get it done and it will ship in the next release. 😉

As a workaround, I recommend installing LDR Importer to C:\Users\MyUser\AppData\Roaming\Blender Foundation\Blender\2.x\scripts\addons, which is the suggested installation path and where I designed it to work. 😃

Again, thank you for your interest in LDR Importer and the bug report! 😄

P.S. If you don't mind my asking (I'm curious), how did you learn of this project?

Banbury ninja'd me. 😛

[MinnieTheMoocher]

thank you very much for your quick reply!

I found a link to this wonderful script on the ldraw.org forums.

[le717]

I have a fix for this, pending commit. I'll let you know when it is pushed for you to try. 😉

[le717]

@MinnieTheMoocher I patched this in bb84f54. Download that revision and tell me if it solves the issue or not. Of course I tested it myself, and it worked when installed both in program files and app data. Basically, it looks for and saves config.py in the same place (app data), but it uses a process to detect the newest available version of Blender and save it there. I'm about to revise it to get the currently used Blender version instead (my local copy of the API docs went missing...). However, if you want to go ahead and test this version, please do. It'd actually help ensure I've done correctly. :)

[le717]

Alright, updated the process in eff7232. @MinnieTheMoocher Will you please test this? I'll merge it tonight if it works for you. 😉

[le717]

I will assume by lack of reply this works, and I will merge it into the master.

Thanks for your interest in LDR Importer! :D

[MinnieTheMoocher]

sorry, didn't see your request in time.
I will not have time to test each change you submit. But if you have a release candidate for 1.2 ready,
I'll be happy to test it for you before release if you want.

[le717]

Closed in #45. If you download the script from the master branch right now, the patch be present. ;)