Ticket #39661

Dridiさんのパッチ

Open Date: 2019-10-07 22:29 Last Update: 2019-11-16 10:32

Reporter:
Owner:
Status:
Open [Owner assigned]
Component:
MileStone:
(None)
Priority:
5 - Medium
Severity:
5 - Medium
Resolution:
None
File:
1
Vote
Score: 0
No votes
0.0% (0/0)
0.0% (0/0)

Details

Dridiさんから提供されたパッチを適用する。以下彼からのコメント。

This summer I managed to build DTXMania on Fedora, report bugs and send patches to Wine, and got it to the point where I can select a song. If I select a song it crashes, but I figured I should probably send you the patches for inclusion so that I don't have to rebase what is done every time you push new features.

I'm making progress and I think that soon I should be able to run DTXMania without winetricks! (I have WAV problems with winetricks).

I hope you will find that the patches work fine on Windows too, because I have no Windows machine to try them myself. For more information, please refer to the commit message of each patch, and don't hesitate to send me your questions if any.

Ticket History (3/9 Histories)

2019-10-07 22:29 Updated by: yyagi
  • New Ticket "Dridiさんのパッチ" created
2019-10-07 23:26 Updated by: yyagi
Comment

添付ファイルにある8つのパッチの内、0001, 0004, 0005, 0006, 0007, 0008 を適用。

0002と0003は以下の理由で一旦適用を保留。

  • 00002: 元々のhintpathはnugetで取得したpackages以下のDLLを参照しており、これをRuntimeResouces以下のDLLを参照するよう変更しない方がnuget管理上よいのではとの懸念が残る。(nugetは別管理で問題ない気もするが、まずは要調査)
  • 00003: NVorbis.csproj をmsbuild2003用に全面書き換えするパッチだが、もともとは.NET Standard用のライブラリということもあり、.NET Framework用のcsprojにしてしまうのには少々抵抗がある。
2019-10-08 00:47 Updated by: yyagi
Comment

0002の方は問題なさそうなので適用しました。

2019-10-30 20:24 Updated by: kairera0467
Comment

パッチ0004適用(Rev:32d90ac83153c134f38159d1c95a6f152cb0acc9)の際に、FDK.csprojの144行目に「<Compile Include="コード\00.共通\CWindowsTheme.cs" />」を追加されているのですが、
コミットログを見る限りCWindowsTheme.csがリポジトリに入っておらず、ビルドができなくなっています。確認よろしくお願いします。

2019-10-30 22:55 Updated by: yyagi
Comment

ご指摘恐縮です。(Dridiさんからも同じ指摘をいただいていました...)

先ほど当該ファイルをコミットしました。ご確認ください。 ついでに、Windowsの文字色を取得する良い方法をご存じでしたら、ぜひ教えてください。

2019-11-04 22:57 Updated by: kairera0467
Comment

CWindowsTheme.csのコミット確認しました。ビルドについては...すみません、まだ試せていない状況です。

ついでに、Windowsの文字色を取得する良い方法をご存じでしたら、ぜひ教えてください。

存じ上げないですね...
取得の前に、まだWindows10に慣れていなかったりします。

2019-11-15 22:10 Updated by: kairera0467
Comment

ビルドが通ることを確認できました。(Windows10 Pro 1903 Build:18362.418、VisualStudio 2019 16.3.8)

2019-11-16 10:32 Updated by: yyagi
Comment

ビルドのご確認ありがとうございました。安心しました。

Attachment File List

Edit

You are not logged in. I you are not logged in, your comment will be treated as an anonymous post. » Login