scottrice / Ice

Application to automatically add ROMs to Steam
http://scottrice.github.io/Ice/
MIT License
835 stars 96 forks source link

Combine console.txt and emulators.txt & Console specific rom folder #95

Closed saftle closed 10 years ago

saftle commented 10 years ago

Steam Category Name="NES" Console Grid Category (http://consolegrid.com) ="Nintendo Entertainment System" Emulator Path="C:\Emulation\NES\Nestopia\Nestopia.exe" Emulator Launch Command="%rom%" Rom Path="C:\Emulation\NES\Roms\"


This removes the need for two files and removes a bit of the bloat in my opinion. I don't think it is necessary for Ice to know what the name of an emulator is, what the nickname is, etc. All that matters is what is being passed to Steam.

Also, I think it would be a great feature to specify console specific rom paths. I may have my PSX isos shared via a network drive in order to save space, but I may save my SNES games locally. This also again makes it to where Ice no longer needs to create a roms folder.

scottrice commented 10 years ago

I am all for the idea of having console specific ROMs folders. That is definitely something that can be done.

As for splitting up emulators and consoles, I want to keep them the way they are. There are two big reasons for this:

I really like the idea of letting people specify separate ROM folders for each console though. I will keep this issue open to track the progress of that feature.

saftle commented 10 years ago

Thanks for the feedback. Regarding the Console grid, I specified the console grid name in the second line. I understand your reasoning though regarding not combining the two files. Thanks for considering the console exclusive rom folder.

scottrice commented 10 years ago

You're right, I totally missed the ConsoleGrid name. My bad. I removed that from my previous comment as you did take that into account.

shidarin commented 10 years ago

It's also possible that an end user would want to use different emulators for different games- if Ice eventually supports multiple emulators for the same console.

scottrice commented 10 years ago

I forgot to reference this issue in the commit, but 6f9be21f2d0d108f573089ddf50dd8695d9942d0 adds the console specific ROMs directory functionality.