Skip to content

Conversation

qryxip
Copy link
Member

@qryxip qryxip commented Jan 7, 2023

内容

model/ディレクトリを保持したまま内容物を展開するようにします。

  • Bash
  • PowerShell

関連 Issue

Fixes #368.

その他

@qryxip qryxip marked this pull request as ready for review January 7, 2023 21:16
@qryxip qryxip changed the title [wip] model/を考慮してダウンロードする model/を考慮してダウンロードする Jan 7, 2023
@qryxip
Copy link
Member Author

qryxip commented Jan 7, 2023

0.14.0-preview.3への動作は未確認です。

@qryxip
Copy link
Member Author

qryxip commented Jan 7, 2023

#367 も似た様な感じで落ちてますね。
まさかとは思いますが/区切りだと思っていたのが\とか…

@qryxip
Copy link
Member Author

qryxip commented Jan 8, 2023

その通りだったみたいです。しかも.NETが勝手に/\に変換したわけでもなく、そもそもファイル名自体が\区切りだったらしい? (今テザリング環境なのでダウンロードして確かめてはいませんが)

追記:

zipinfo -1 ~/Downloads/voicevox_core-windows-x64-cpu-0.14.0-preview.3.zip
voicevox_core-windows-x64-cpu-0.14.0-preview.3\onnxruntime.dll
voicevox_core-windows-x64-cpu-0.14.0-preview.3\onnxruntime_providers_shared.dll
voicevox_core-windows-x64-cpu-0.14.0-preview.3\README.txt
voicevox_core-windows-x64-cpu-0.14.0-preview.3\VERSION
voicevox_core-windows-x64-cpu-0.14.0-preview.3\voicevox_core.dll
voicevox_core-windows-x64-cpu-0.14.0-preview.3\voicevox_core.h
voicevox_core-windows-x64-cpu-0.14.0-preview.3\voicevox_core.lib
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\.gitignore
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\d0.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\d1.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\d10.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\d11.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\d2.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\d3.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\d4.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\d5.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\d6.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\d7.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\d8.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\d9.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\metas.json
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\pd0.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\pd1.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\pd10.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\pd11.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\pd2.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\pd3.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\pd4.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\pd5.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\pd6.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\pd7.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\pd8.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\pd9.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\pi0.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\pi1.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\pi10.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\pi11.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\pi2.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\pi3.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\pi4.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\pi5.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\pi6.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\pi7.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\pi8.bin
voicevox_core-windows-x64-cpu-0.14.0-preview.3\model\pi9.bin

#117 からあるこの部分で\になっているらしい? .NETでは/に直しているはずなのですが、Compress-Archiveはどうなんでしょう? そもそもpowershell.exeを呼んだら確かPowerShell 5になった気が...

powershell Compress-Archive -Path "artifact/${{ env.ASSET_NAME }}" -DestinationPath "${{ env.ASSET_NAME }}.zip"

まあこれを直すべきかというと微妙かもしれませんが。明確な規格違反でありunzip(1)とかでも警告が出るとしても、\区切りのZIPファイルは世に溢れてはいますし。

Copy link
Contributor

@qwerty2501 qwerty2501 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTMですが
実行可能バイナリ版ダウンローダーがあればそもそもスクリプト版ダウンローダーは不要そうですね

Copy link
Member

@PickledChair PickledChair left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!(download_test がまだ model の存在をチェックしていないと思いますが、この対応はこの PR とは別に行う感じでしょうか?)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ダウンローダーがmodel/の内容をぶちまけてしまう
4 participants