Skins parser produces "parse error" when encountering __MACOSX folders

[jolohaga]

Bug Description:
When folders are zipped using the Finder, MacOS can add a __MACOSX folder to the contents of the zip archive. If such a zip file is passed to the skins parser, the parser will fail on the __MACOSX folder and issue a "parse error" message in the skins list.

Surge XT Version

• Version: 1.2.3
• Plugin Type: AU
• Bitness: 32-bit

Reproduction Steps:
Steps to reproduce the behavior:

1. Using Safari, download a skin archive from the library
2. The download will be automatically unzipped. In the Finder find the download, right-click on it and select "Compress"
3. In Surge, drag-and-drop the Finder zipped file
4. Open the skins list and a list will appear tagged with "parse error". (see attached screenshot for an example)

Expected Behavior:
I expected to see a single set of skins to appear without "parse error" appearing.

Screenshots:

Computer Information (please complete the following!):

• OS: MacOS Ventura 13.4.1
• Host: Logic 10.7.9
• Version: 10.7.9

Additional Information:
Please let me know if you need additional info. I can be reached in Discord.

[baconpaul]

Thanks. This will be easy to fix and I'll get it for 1.3. Excellent bug report. Appreciated.

[baconpaul]

Confirmed bug

[baconpaul]

Ahh there's two ways to solve it. The easy way is ignore __MACOSX directories form the zip file. The harder but better way is do that but also don't create them. I'll try and do both.