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

The AddFLACs script stops after converting one file #2

Closed
dmitryleskov opened this Issue Jun 14, 2013 · 1 comment

Comments

Projects
None yet
1 participant
@dmitryleskov
Owner

dmitryleskov commented Jun 14, 2013

Several people reported this problem in the comments to my blog post For instance, a visitor who identified himself as Brendan wrote:

Love your script, real time saver.
But since reinstalling Windows (from Windows 8 to Windows 8, so nothing new there, just a reinstall) it’s converting 1 file each time I run it.
The temp.meta files that are created contain the correct Artist, Album, track nr. etc.
The files I’m trying to add of course also contain this information
Do you have any idea what causes this?

I’m running Windows 8 and I use your script as per the instructions on this blog.

But I could not reproduce that issue until another reader, dave granic, provided his insight:

The problem Brendan is having is the same one I am having with Permission Denied when the script tries to execute

fso.DeleteFile(tempWAVPath)

I modified the script to show the TempWAVPath and I have set permissions on the folder to allow EVERYONE full control, could the file be open?

Deleting intermediate track
Deleting temporary WAV file C:\temp\AddFLACs-2013-05-19-00-08-11\temp.wav
C:\downloads\install\AddFLACs-0.91\AddFLACs.js(275, 13) Microsoft JScript runtime error: Permission denied

and followed up with a solution:

fixed the issue… changed line 276 of js file to

fso.DeleteFile(tempWAVPath, true)

@ghost ghost assigned dmitryleskov Jun 14, 2013

@dmitryleskov

This comment has been minimized.

Show comment
Hide comment
@dmitryleskov

dmitryleskov Jun 15, 2013

Owner

Indeed, flac.exe preserves the readonly attribute and there seems to be no way to alter this behavior.

Owner

dmitryleskov commented Jun 15, 2013

Indeed, flac.exe preserves the readonly attribute and there seems to be no way to alter this behavior.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment