Purge of ANGBAND_DIR_DEBUG_SAVE for non-Windows systems

• 報告者: backwardseric
• 担当者: habu
• チケットの種類: パッチ
• 状況: 完了
• コンポーネント: (未割り当て)
• マイルストーン: 変愚蛮怒3.0.0Alpha5リリース (完了済み)
• 優先度: 3
• 重要度: 3
• 解決法: 修正済み
• ファイル: なし

詳細

Recent changes to src/main/angband-initializer.c include code to purge old entries from ANGBAND_DIR_DEBUG. That code appears to be Windows-specific and does not compile on macOS.

This commit, https://osdn.net/projects/hengbandforosx/scm/git/hengbandosx/commits/bcd48d9ec0ea82ef0e45dcd283985bec5176eb4a , (unix-debug-save-purge branch on the hengbandforosx project; branch is based off Hengband's develop branch) brackets the Windows-specific code with "#ifdef WINDOWS" and adds a Unix-specific implementation in the "#else" clause.

チケットの履歴 (4 件中 3 件表示)

2021-02-06 06:21 更新者: backwardseric

• 新しいチケット "Purge of ANGBAND_DIR_DEBUG_SAVE for non-Windows systems" が作成されました

2021-02-06 08:12 更新者: backwardseric

コメント

Two follow-up commits on the same branch, https://osdn.net/projects/hengbandforosx/scm/git/hengbandosx/commits/f92080b275de8e848e1c72bca1b24014b7c95d7d and https://osdn.net/projects/hengbandforosx/scm/git/hengbandosx/commits/9ebca88726d9c5dcc38c4ba749593c6e5884a0e0 (the latter is the head of the branch), correct two deficiencies with the original:

• Was not initializing default_savefile on non-Windows systems.
• Was only passing the last part of the path and not the full path to stat() and remove().

This all relates to the changes in your feature/Fix-Delete-Logfile branch. I don't know if my branch will merge cleanly with that one.

2021-02-06 13:11 更新者: habu

• 担当者 が (未割り当て) から habu に更新されました
• 解決法 が なし から 修正済み に更新されました

コメント

I've merged your commits into the feature/Debug-save-for-non-Windows branch and made a minor fix to the path generation part. This branch will be merged into develop.

I think the feature/Fix-Delete-Logfile will be discarded.

Thanks for your implementation.

2021-02-07 18:38 更新者: deskull

• 状況 が オープン から 完了 に更新されました
• マイルストーン が (未割り当て) から 変愚蛮怒3.0.0Alpha5リリース (完了済み) に更新されました

コメント

解決しているようですので閉じます。