dh4 / mupen64plus-qt

A customizable launcher for Mupen64Plus
BSD 3-Clause "New" or "Revised" License
95 stars 10 forks source link

ROMs do not import into Debian 10 "Buster" #61

Closed eduardomozart closed 3 years ago

eduardomozart commented 3 years ago

Mupen64plus-qt do not imports ROMs after selecting a directory through "ROMS Directories" from GUI into Deepin 20 (based on Debian 10 "Buster"). I tried the mupen64plus-qt version from Debian repositories (1.1.2), latest stable (1.1.4) and development releases (1.14-git:f1ea589).

After adding a directory into "ROMs directory" from the Settings and click on "Ok", it do not import any ROMs and when I go back into settings the "ROMs directory" list is empty.

I'm not sure, but maybe this issue is related to #57?

The only workaround I could find so Mupen64plus-qt start scanning my ROMs would be editing it's config file directly (''/home/leo/.config/mupen64plus/mupen64plus-qt'') and set the ''roms'' key from ''Paths'' section manually (e.g. ''roms=/home/leo/Games/Nintendo - Nintendo 64 (N64)''). So I had to open the "Settings" from Mupen64plus-qt. If I click into the "OK" button, it clears the "ROMs directory" list, but if I press the "X" close button to close the "Settings" window, it starts importing the ROMs normally.

dh4 commented 3 years ago

Let me know if the commit I just pushed fixes the issue. If so, I'll go through the process of creating a new Debian release when I get a chance.

eduardomozart commented 3 years ago

Hello @dh4, thank you for your fast response! I compiled the modified version and it worked normally - the ROMs directory isn't being excluded anymore when opening and changing the Settings through Mupen64plus-qt UI. I noticed two small issues:

Thank you for adding support for ZIP files. I know it just extracts the ''*.n64'' file from zip to a temp folder, but it's a pretty neat feature for Mupen64plus and make loading ROMs compatible with Parallel64 (RetroArch). It would be amazing updating the Debian repo to a newer Mupen64plus-qt build so the Game information from the TheGamesDB would be starting working again into Debian repo build.

Thank you for still working on this project, it's amazing. The only feature I really miss is allowing configuring joypad controls from UI. M64Py allows this.

dh4 commented 3 years ago

Thanks! Glad that fixed it. I think your 2nd and 3rd bullet points are good suggestions. I'll try to add those features/fixes when I get a chance!

Mavarice commented 2 years ago

@dh4 By the way, the version in apt is still pulling 1.14.1 which has this bug

mupen64plus-qt is already the newest version (1.14-1).