search

casualsnek / onthespot

qt based music downloader written in python
GNU General Public License v2.0
745 stars 54 forks source link

Stuck at 99% - PermissionError #5

Closed dylanrsmith closed 1 year ago

dylanrsmith commented 1 year ago

Was getting this error when downloading a track.

Progress bar would be stuck 'converting' at 99%.

Not sure if this is a widespread issue, I think I found a potential fix.

Traceback (most recent call last):
  File "spotutils.py", line 394, in run
    status = self.download_track(
  File "spotutils.py", line 328, in download_track
    os.remove(filename)
PermissionError: [WinError 32] The process cannot access the file because it is being used by another process: 'C:/Music/Song.mp3'
Miollnir commented 1 year ago

i have the same issue did u found a fix ?

Ragriod commented 1 year ago

image

After download the track is stuck with converting at 99%

casualsnek commented 1 year ago

i have the same issue did u found a fix ?

Maybe yes, i will push a commit in a while which you can test !

casualsnek commented 1 year ago

image

After download the track is stuck with converting at 99%

Can you test it now ? I have binary release for windows which you can use ! Also follow Installing on windows section in README.md to install ffmpeg which should solve these issues

Miollnir commented 1 year ago

Can you test it now ? I have binary release for windows which you can use ! Also follow Installing on windows section in README.md to install ffmpeg which should solve these issues

it works now 👍

casualsnek commented 1 year ago

Was getting this error when downloading a track.

Progress bar would be stuck 'converting' at 99%.

Not sure if this is a widespread issue, I think I found a potential fix.

Traceback (most recent call last):
  File "spotutils.py", line 394, in run
    status = self.download_track(
  File "spotutils.py", line 328, in download_track
    os.remove(filename)
PermissionError: [WinError 32] The process cannot access the file because it is being used by another process: 'C:/Music/Song.mp3'

Can you please test this binary release: https://anonfiles.com/69zbldBey8/onthespot_win_exe built against current main branch ! Apparently there were multiple reasons why this could happen and this binary should work as expected. The build instructions and scripts were changed so while i write documentation on it you can try the binary and check if it works !

dylanrsmith commented 1 year ago

Just tested that binary and it works ! I'll close this issue. Thanks!