Simply-Love / Simply-Love-SM5

a StepMania 5 theme for the post-ITG community
Other
196 stars 145 forks source link

Judgement fonts broken with latest SM5 git builds. #7

Closed Xaero252 closed 9 years ago

Xaero252 commented 9 years ago

Pretty much what the title says: 2014-12-02-033327_2561x1440_scrot As you can see the Judgement Font is completely gone. Basically the same as playing with it disabled via option in ScreenGameplay. Here's what I can snab from the log: Theme element "Graphics/_judgments" could not be found in "Themes/Simply Love/" or "Themes/_fallback/". This is with a fresh copy from git; furthermore:  stat "/home/xaero/.stepmania-5.0/Themes/Simply Love/Graphics/_judgments/" File: ‘/home/xaero/.stepmania-5.0/Themes/Simply Love/Graphics/_judgments/’ Size: 4096 Blocks: 8 IO Block: 4096 directory Device: 812h/2066d Inode: 14682716 Links: 2 Access: (0700/drwx------) Uid: ( 1000/ xaero) Gid: ( 100/ users) Access: 2014-12-02 03:22:32.736447745 -0700 Modify: 2014-11-18 19:43:54.380018646 -0700 Change: 2014-11-18 19:43:54.380018646 -0700 Birth: - As you can see, the file is there - and usable by my current user. Only thing that has changed is the version of sm5 installed.

I just chmod'd my entire stepmania 5 directory 0755 to make sure it wasn't a permissions issue (shouldn't have been to begin with - same result.) That's the only reference to Judgements in the log provided. I'll be checking the SM5 git to see what could have changed to affect this.

quietly-turning commented 9 years ago

Good catch. Using bisection, I was able to narrow it down to this particular merge: https://github.com/stepmania/stepmania/commit/3b36858cf8c2e92d625947ed65b3f5f0e2ac264b

I'll look into a bit, but I suspect I'll end up just filing a bug report with StepMania.

kyzentun commented 9 years ago

Whoops, my fault. Fixed in this commit: https://github.com/stepmania/stepmania/commit/89864fc890ed7e1f8874e7fc6be1df072d06027e

Xaero252 commented 9 years ago

Confirmed fixed with above commit. Closing this.

quietly-turning commented 9 years ago

Thanks, @kyzentun!