Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Game INI Updates #3103

Merged
merged 3 commits into from Oct 4, 2015
Merged

Game INI Updates #3103

merged 3 commits into from Oct 4, 2015

Conversation

Linktothepast
Copy link
Contributor

This is mainly a virtual console update. Removes progressive scan option
from snes, genesis, etc. games since it is no longer needed for proper
speed. Adds gameinis for Neo geo, Arcade and missing gameinis for
genesis.

This is mainly a virtual console update. Removes progressive scan option
from snes, genesis, etc. games since it is no longer needed for proper
speed. Adds gameinis for Neo geo, Arcade and missing gameinis for
genesis.
@Linktothepast
Copy link
Contributor Author

This is a great git failure from my part and i don't have a clue how to fix this apart starting from scratch.

@BhaaLseN
Copy link
Member

git rebase dolphin-emu/master master and cross your fingers. If it worked, git push -f origin master (or whatever your own fork is called, if its not origin).
In the future, you might want to consider creating branches for that, instead of using master (to avoid this).

@Linktothepast
Copy link
Contributor Author

I created a branch gameinis locally but for some weird reason it didn't showed up in the repo online from the beginning instead every time i committed something to gameinis locally it committed it to master in the online repo...

@BhaaLseN
Copy link
Member

Only branches you pushed will show up on GitHub, otherwise they're only local to your clone. Maybe try git rebase dolphin-emu/master gameinis followed by git push origin gameinis (again, replace origin if necessary).
Maybe drop by on IRC, easier to discuss there if anything else comes up.

@dolphin-emu-bot
Copy link
Contributor

FifoCI detected that this change impacts graphical rendering. Here are the behavior differences detected by the system:

automated-fifoci-reporter

@Tilka Tilka changed the title Master Game INI Updates Sep 29, 2015
@shellashock
Copy link

Since you have a pr for game inis open, I'll just leave this here.

Conflict - Desert Storm 2: Back to Baghdad NTSC (GC2E9G) will display videos if Real XFB is enabled and will only produce the audio with a black screen with either Virtual XFB or Disabled XFB.

At the time of writing, Real XFB is not enabled in the .ini for this game and it probably should be considering that it makes cutscenes visible without any side effects that I am aware of. This change makes this game fully playable out of the box with no issues whatsoever. (wiki can be changed to 5 star rating after adding the requirement for Real XFB to the video issue?)

@Linktothepast
Copy link
Contributor Author

Oh thanks i will try to add it, it seems there is another game that the hybrid xfb branch can help. Although i am pissed at git, might as well pack all the inis and ask someone else to do a pr about it.

shuffle2 added a commit that referenced this pull request Oct 4, 2015
@shuffle2 shuffle2 merged commit 43879eb into dolphin-emu:master Oct 4, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
5 participants