theypsilon / Update_All_MiSTer

All-in-one script for updating your MiSTer
GNU General Public License v3.0
605 stars 27 forks source link

Remove empty folders when running update_all #76

Closed JosVerheij closed 2 years ago

JosVerheij commented 2 years ago

Issue: folders that are removed in the source repositories (at least for Shadowmasks_Mister) are not locally removed by update_all.

I currently have multiple empty folders which previously contained shadow masks that have since been reorganised by the maintainers. These folders have either been moved or removed by the maintainers. Update_all apparently correctly removes the individual files, but leaves the folders in which they lived intact. This results in clutter which is confusing to navigate; it does not make sense to encounter empty folders.

Solution: remove folders that contain no files when running update_all.

theypsilon commented 2 years ago

Thanks for reporting this. It seems to be a bug in the downloader/distribution db. I'll see what can I do to fix it. You can follow the progress here: https://github.com/MiSTer-devel/Downloader_MiSTer/issues/7