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

成果物の ZIP 内部のフォルダを短くする (「リリースファイルのファイル名が長すぎるのでは?」 に対応する) #815

Merged

Conversation

m-tmatma
Copy link
Member

@m-tmatma m-tmatma commented Mar 27, 2019

成果物の ZIP 内部のフォルダを短くする (「リリースファイルのファイル名が長すぎるのでは?」 に対応する)

sakura-editor/management-forum#66 (comment) (sakura-editor/management-forum#52 (comment))

@m-tmatma m-tmatma added this to the next release milestone Mar 27, 2019
@m-tmatma m-tmatma added the CI appveyor など CI 関連 【ChangeLog除外】 label Mar 27, 2019
@m-tmatma m-tmatma changed the title ZIP 内部のフォルダを短くする (「リリースファイルのファイル名が長すぎるのでは?」 に対応する) 成果物の ZIP 内部のフォルダを短くする (「リリースファイルのファイル名が長すぎるのでは?」 に対応する) Mar 27, 2019
KENCHjp
KENCHjp previously approved these changes Mar 27, 2019
Copy link
Member

@KENCHjp KENCHjp left a comment

Choose a reason for hiding this comment

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

https://ci.appveyor.com/project/sakuraeditor/sakura/builds/23410869/job/6mv7gdkpqkh8oq75/artifacts

ここ確認しただけですが、短くなっているのを確認しました。

@ds14050
Copy link
Contributor

ds14050 commented Mar 28, 2019

実は PR #787 「[WIP]build-installerとzipArtifacts.batの簡略化」では、EXE や Installer といった zip 内部フォルダも省略されています。自分はそれでいいと思っていましたし、@KageShiron さんに修正を求めるつもりもありませんでした。

EXE や Installer といったサブフォルダは全部入りのアーティファクトを分割した経緯から存在しているだけで、もう後戻りするつもりもないでしょうから不要ではないでしょうか。ビルドナンバーも PR ナンバーもない EXE というだけのフォルダを残してもどのみち「可能であれば zip を解凍したときに Win32 のようなフォルダができるような形ではなく zip ファイル名に近い形のフォルダができるようになっていると解凍時にファイルが行方不明になったり混在しなくて助かります。」という要望には応えられないのですから。

さらに短縮するもしないも PR #787 まかせでいいとも思っています。

@KENCHjp
Copy link
Member

KENCHjp commented Mar 28, 2019

改修はより良い改修で上書きすることで進化すると思っています。
より良いとはその時のスナップショットでしかなくて過去を否定するものではありません。
そしてまた明日より良いスナップショットで上書きすればいいじゃないっすか

@m-tmatma
Copy link
Member Author

ZIP 内部の上位ファルダをなくしました。

Copy link
Contributor

@ds14050 ds14050 left a comment

Choose a reason for hiding this comment

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

早いですね。生成物から階層が減っているのを確認しました。RELDIR_* の使用場所が1か所だけになったので変数を削ってもいいと思いますが、そこはご自由に。

@m-tmatma m-tmatma merged commit 059661f into sakura-editor:master Mar 28, 2019
@m-tmatma m-tmatma deleted the feature/artifacts_path_name_shorten branch March 28, 2019 21:24
takke added a commit to takke/sakura that referenced this pull request Apr 3, 2019
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 11, 2019
…path_name_shorten

成果物の ZIP 内部のフォルダを短くする (「リリースファイルのファイル名が長すぎるのでは?」 に対応する)
HoppingTappy pushed a commit to HoppingTappy/sakura that referenced this pull request Jun 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI appveyor など CI 関連 【ChangeLog除外】
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants