laggykiller / sticker-convert

Convert (animated) stickers to/from WhatsApp, Telegram, Signal, Line, Kakao, Viber, Discord, iMessage. Written in Python.
GNU General Public License v2.0
286 stars 16 forks source link

macOS 14 FileExistsError #111

Closed sindastra closed 9 months ago

sindastra commented 9 months ago

Sometimes on macOS 14, clicking on the "Start" button will result in an instant crash.

I'm not sure why this would happen, since the folder is newly created when hitting "Start". Even though the folder stickers_output is empty. Input source is Telegram, output source is Signal.

I already tried deleting all files in stickers_input and stickers_output and restarting the application.

sticker-convert 2.2.6
Launching GUI...
Moving old files in input directory to /Users/nat/Desktop/stickers_output/archive_2024-28-01_23-23-10 as input source is not local
Moving old files in output directory to /Users/nat/Desktop/stickers_output/archive_2024-28-01_23-23-10
Exception in thread Thread-1 (start_process):
Traceback (most recent call last):
  File "/Applications/sticker-convert.app/Contents/MacOS/threading.py", line 1045, in _bootstrap_inner
  File "/Applications/sticker-convert.app/Contents/MacOS/threading.py", line 982, in run
  File "/Applications/sticker-convert.app/Contents/MacOS/sticker_convert/gui.py", line 434, in start_process
  File "/Applications/sticker-convert.app/Contents/MacOS/sticker_convert/job.py", line 76, in start
  File "/Applications/sticker-convert.app/Contents/MacOS/sticker_convert/job.py", line 231, in cleanup
  File "<frozen os>", line 225, in makedirs
FileExistsError: [Errno 17] File exists: '/Users/nat/Desktop/stickers_output/archive_2024-28-01_23-23-10'
laggykiller commented 9 months ago

Are you setting input directory and output directory to the same directory? Please set them as different directory.

sindastra commented 9 months ago

Ah, you are correct! I think this shows it is time for me to go to bed... Apologies for the redundant issue!

sindastra commented 9 months ago

Actually, I just came to think. Perhaps it would be a good idea to implement a little check and output an error message if the input and output directory are the same?

You know, in case the user is sleepy...

Such a check should fix both #111 and #112

laggykiller commented 9 months ago

Done