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.

Sign up for GitHub

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

Jump to bottom

CIのArtifactsについて #1906

Closed
dep5 opened this issue Feb 3, 2023 · 7 comments · Fixed by #1911
Closed

CIのArtifactsについて #1906

dep5 opened this issue Feb 3, 2023 · 7 comments · Fixed by #1911

Comments

@dep5
Copy link
Contributor

dep5 commented Feb 3, 2023

問題内容

  • Github ActionsとAppVeyor
    Asm --- Release
    Release-Asm.zipの中身がありません。
    アセンブラ出力を有効にしませんか? #286

  • Github Actions
    osがwindows-2019とwindows-2022の2つありJobsも8個あります。
    osに合わせたVisual Studioのバージョン別にJobが用意されていると考えました。
    　
    Artifactsはどちらのosでビルドされたかわからないように思います。
    （後でアップロードされた方で上書きしている？）
    全種類アップロードするか、os情報のテキストをzipに入れるかしてもらえないですか？
    　
    そうかもと思っただけで、違っていたらすいません。

再現手順

再現頻度

問題のカテゴリ

  • ビルドの問題
    • Appveyor
    • Github Actions

環境情報

スクリーンショット
@dep5
Copy link
Contributor Author

dep5 commented Feb 3, 2023 •
edited
Loading

フォーク先のGithub ActionsではJobsが7個同時に動くようなので
windows-2022, Release, x64版が後まわしになるので、
Exe x64 Releaseがwindows-2022版になってしまう気がしています。
見分け方がわからないので、気のせいかもしれません。

@berryzplus
Copy link
Contributor

アセンブラ出力はDebugだけじゃなかったでしたっけ？
ちょっと曖昧です。

  • Github Actions
    osがwindows-2019とwindows-2022の2つありJobsも8個あります。
    osに合わせたVisual Studioのバージョン別にJobが用意されていると考えました。
    　
    Artifactsはどちらのosでビルドされたかわからないように思います。
    （後でアップロードされた方で上書きしている？）
    全種類アップロードするか、os情報のテキストをzipに入れるかしてもらえないですか？

指摘通りのバグだと思います。
2 × 2 × 2 ＝ 8セットが4セット分しかない枠にアップロードされてます。
vs2022でのビルドをできるようにしたとき、テキトーにレビューしたのが原因です。
現状は、たまたま後に終わったビルドの内容で上書きされている状態です。

@dep5
Copy link
Contributor Author

dep5 commented Feb 5, 2023

berryzplusさん
コメントありがとうございます
#1486 とそこにある #1777 も含めて知りたかったことが書いてありました。

アセンブラ出力はDebugだけじゃなかったでしたっけ？

PRによってはアセンブラ出力されているということですかね？
それならしょうがないですね。

2 × 2 × 2 ＝ 8セットが4セット分しかない枠にアップロードされてます。

ファイルサイズ的に問題なければ必要なセット分の枠が欲しいです。

@berryzplus
Copy link
Contributor

お気持ち

2つの問題を解決しないとcloseできないissueになっています。
問題が2つならissueを分けたいです。

過去のissueも複数の問題を含んだissueは放置される傾向にある気がします。

@berryzplus berryzplus mentioned this issue Feb 6, 2023
Merged
@berryzplus
Copy link
Contributor

#1911 の対応ではダメでしょうか？

  • ASMの出力先パスを変更したのをzip生成バッチに反映していなかった不具合を修正します。
  • 成果物名が複数のVisual Studioバージョンによるビルドを考慮していないのに、複数バージョンでビルドしてしまっていた誤りを是正します。

@dep5
Copy link
Contributor Author

dep5 commented Feb 10, 2023

AppVeyorについては確認できませんでしたが、zipのほうを修正してくださったんですね。
Github Actionsでは確認できました。
asmやdevについては使い方がわかるわけではなく、中身がないのが気になっただけなので、解決しました。

わたしが書いたコードがWindows Server 2022にしか対応していなかったので
Windows Server 2022に一本化されるならとても助かります。

AppVeyor
Visual Studio 2017 (Windows Server 2016)
Github Actions
Visual Studio 2019 (Windows Server 2019)
Visual Studio 2022 (Windows Server 2022)

Visual Studioのサポートバージョンを網羅しているのかと思っていたのですが、
そういうわけではなかったのでしょうか？
構わなければ、これでお願いします。

@dep5
Copy link
Contributor Author

dep5 commented Feb 12, 2023

対応が済んだようですのでクローズします

@dep5 dep5 closed this as completed Feb 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

CIのArtifactsについて berryzplus/sakura-editor
2 participants
@berryzplus @dep5