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

v2.4.0 alpha1 pre-release 確認issue #66

Closed
4 of 6 tasks
KENCHjp opened this issue Mar 27, 2019 · 66 comments
Closed
4 of 6 tasks

v2.4.0 alpha1 pre-release 確認issue #66

KENCHjp opened this issue Mar 27, 2019 · 66 comments
Labels
Release リリース作業チケット

Comments

@KENCHjp
Copy link
Member

KENCHjp commented Mar 27, 2019

前のIssueが長くなってきたので新たに立てます。

#52


コメントで上げられた検討事項

@KENCHjp
Copy link
Member Author

KENCHjp commented Mar 27, 2019

#52 (comment)
転記。
sakura-tag-v2.4.0-alpha1-build1699-2582c34c-Win32-Release-Exe.zip をダウンロードしましたが
このファイルをフォルダ名にして展開すると
R:\sakura-tag-v2.4.0-alpha1-build1699-2582c34c-Win32-Release-Exe\sakura-tag-v2.4.0-alpha1-build1699-2582c34c-Win32-Release\EXE\sakura.exe
sakura.exe までのパスが137文字あって長いのは少し対策が必要じゃないか?と思いました。

R:\sakura-v2.4.0-alpha1-Win32-Release-Exe\sakura-v2.4.0-alpha1-Win32-Release\EXE\sakura.exe
になったらちょっとマシかな?と思いました。

ちなみにインストーラーだと
R:\sakura-tag-v2.4.0-alpha1-build1699-2582c34c-Win32-Release-Installer\sakura-tag-v2.4.0-alpha1-build1699-2582c34c-Win32-Release\Installer\sakura_install2-4-0-1699-x86.exe
EXE ファイルまでのパスが137文字あります。

@takke takke added the Release リリース作業チケット label Mar 27, 2019
@takke
Copy link
Member

takke commented Mar 27, 2019

早速ありがとうございます。

EXE の ZIP が Installer よりもデカイ(18.6MB)のが気になっていまして、そのままアップロードしていいのか躊躇していました。

sakura.pdb や sakura_lang_en_US.pdb は(debugビルドはともかく)Releaseビルドには不要かなと思うのですがいかがでしょうか。

@takke
Copy link
Member

takke commented Mar 27, 2019

R:\sakura-v2.4.0-alpha1-Win32-Release-Exe\sakura-v2.4.0-alpha1-Win32-Release\EXE\sakura.exe
になったらちょっとマシかな?と思いました。

私もリリース版は sakura-v2.4.0-alpha1-Win32-Release-Exe.zip くらいにリネームしてアップロードでもかまわないと思います。Artifacts へのURLも貼っていただいているのでどのビルドかは追跡できると思いますし。

@KENCHjp
Copy link
Member Author

KENCHjp commented Mar 27, 2019

sakura-editor/sakura#263

これが治ってない気がします。
うち的にはこれキラーコンテンツなのでどなたか検証対応をお願いします。
キャプチャ

@KENCHjp
Copy link
Member Author

KENCHjp commented Mar 27, 2019

sakura.pdb や sakura_lang_en_US.pdb は(debugビルドはともかく)Releaseビルドには不要かなと思うのですがいかがでしょうか。

私も思ったのですが、一旦そのままあげてみました。
releaseビルドに必要なければ、削りたいですね。

@k-takata
Copy link
Member

sakura.pdb や sakura_lang_en_US.pdb は(debugビルドはともかく)Releaseビルドには不要かなと思うのですがいかがでしょうか。

Releaseビルドを実行中に起きたエラーを調べるには、Releaseの .pdb も必要だと思います。ただ、同じ .zip ファイルに入れるべきかは疑問です。ついでに、sakura-doxygen.chm/chi も別zipでいいのではないかという気がします。
なお、sakura_lang_en_US.dll には実行コードがないので sakura_lang_en_US.pdb はRelease/Debugともに不要だと思います。

@KENCHjp
Copy link
Member Author

KENCHjp commented Mar 27, 2019

ファイル名の長さおよび何を梱包するかについては、ci系手を入れるのが工数かかるようであれば(またはセンシティブであれば)、一旦手動対応でも構わないかなと思っています。
何を梱包するかだけの議論でいいかと思うので。
最悪インストラーのみ公開でも構わないかなと。

@KENCHjp
Copy link
Member Author

KENCHjp commented Mar 27, 2019

sakura-tag-v2.4.0-alpha1-build1699-2582c34c-Win32-Release-Installer.zip
sakura-tag-v2.4.0-alpha1-build1699-2582c34c-Win32-Release-Exe.zip

このこらの中のパスが、
\sakura-tag-v2.4.0-alpha1-build1699-2582c34c-Win32-Release\EXE
\sakura-tag-v2.4.0-alpha1-build1699-2582c34c-Win32-Release\Installer
がついてるので、まるっと取ってて直下をzip化してもいいかもですね。

@KENCHjp
Copy link
Member Author

KENCHjp commented Mar 27, 2019

インストーラーをWindows10(32bit)で、新規インストールアインインストールの動確しました。

@KENCHjp
Copy link
Member Author

KENCHjp commented Mar 27, 2019

備忘録
https://github.com/sakura-editor/sakura-editor.github.io
こっちもv2.4.0リリース用のブランチ切る。

@m-tmatma
Copy link
Member

sakura-editor/sakura#263

これが治ってない気がします。
うち的にはこれキラーコンテンツなのでどなたか検証対応をお願いします。
キャプチャ

これは新規インストールのみです。
アップデートインストールでは、設定値は変更されないので。

@KENCHjp
Copy link
Member Author

KENCHjp commented Mar 27, 2019

これは新規インストールのみです。

新規インストールしたつもりです。

C:\Program Files\sakura

このsakuraフォルダと、

C:\Users<user>\AppData\Roaming\sakura

このsakuraフォルダが無いのは確認して新規インストールしたのですが、
他にありますでしょうか?

@KENCHjp
Copy link
Member Author

KENCHjp commented Mar 27, 2019

起動して、
C:\Users<user>\AppData\Roaming\sakura\sakura.ini

ここにiniファイルが出来上がってるのを確認して一度落として、iniファイル削除して再度たちあげてもみたのですが。

@m-tmatma
Copy link
Member

32bit os ですか?
64bit os だとアプリフォルダが違います。

@m-tmatma
Copy link
Member

↑ 書いてた。

@KENCHjp
Copy link
Member Author

KENCHjp commented Mar 27, 2019

32bit osです。
VirtulaBoxで32bitのまっさらな環境作って確認中。

@m-tmatma
Copy link
Member

sakura-tag-v2.4.0-alpha1-build1699-2582c34c-Win32-Release-Installer.zip
sakura-tag-v2.4.0-alpha1-build1699-2582c34c-Win32-Release-Exe.zip

このこらの中のパスが、
\sakura-tag-v2.4.0-alpha1-build1699-2582c34c-Win32-Release\EXE
\sakura-tag-v2.4.0-alpha1-build1699-2582c34c-Win32-Release\Installer
がついてるので、まるっと取ってて直下をzip化してもいいかもですね。

これは sakura-editor/sakura#114 (comment) で対応したものです。

パスを短くするのは zipartifacts.batをちょこっと変えるだけです。
ただ、私は今日は時間は取れないです。

今日リリースすることにこだわらなくてもいいんじゃないかとは思ってます。
見つかった問題を対処してからリリースする方がいいと思います。

@KENCHjp
Copy link
Member Author

KENCHjp commented Mar 27, 2019

すいません、今日releaseにはこだわってないです。
@takke さんが頑張っていただいたので、私もやれる範囲で、一通りreleaseまでやってみようかなって思って手を動かしてみましたが、
まだalpha版なので、こっから1weekぐらいみんなで埃叩いて再来週ぐらいリリース出来たらなってイメージです。
4月中でもいいかなっと。

@KENCHjp
Copy link
Member Author

KENCHjp commented Mar 27, 2019

もういっそ新元号にあわせてもいいかなとかか。

@beru
Copy link

beru commented Mar 27, 2019

https://sakura-editor.github.io/download.html

こちらのページの更新もリリース作業に加えるのはどうでしょうか?

@m-tmatma
Copy link
Member

https://github.com/sakura-editor/sakura/releases/tag/v2.4.0-alpha1

にダウンロード数のカウンタつけました。

こんな感じの指定です。

![Github Releases v2.4.0-alpha1](https://img.shields.io/github/downloads/sakura-editor/sakura/v2.4.0-alpha1/total.svg "v2.4.0-alpha1")

@m-tmatma
Copy link
Member

パス名を短くする PR sakura-editor/sakura#815 を作成しました。
ZIP 内部のフォルダ名に関して、こうしたほうがいいとかあったらコメントください。

@m-tmatma
Copy link
Member

ファイル名の長さおよび何を梱包するかについては、ci系手を入れるのが工数かかるようであれば(またはセンシティブであれば)、一旦手動対応でも構わないかなと思っています。

sakura-editor/sakura#815 が終わったら PR 投げます。

@KENCHjp
Copy link
Member Author

KENCHjp commented Mar 27, 2019

@m-tmatma さん

sakura-editor/sakura#815 が終わったら PR 投げます。

了解です!

@m-tmatma
Copy link
Member

あと、https://github.com/sakura-editor/sakura/milestone/1 の扱いを考える必要があります。

https://github.com/sakura-editor/sakura/releases/tag/v2.4.0-alpha1 を作成していますが、

alpha1 ~ 正式な v2.4.0 までのバージョンの間にいくつか変更が入ると思いますが、
どの修正が alpha1 ~ 正式な v2.4.0 の間に入ったか、わかるようになんらかの形で
管理したいです。

@KENCHjp
Copy link
Member Author

KENCHjp commented Mar 27, 2019

@beru さん

こちらのページの更新もリリース作業に加えるのはどうでしょうか?

もち!

@KENCHjp
Copy link
Member Author

KENCHjp commented Mar 27, 2019

@m-tmatma さん

>どの修正が alpha1 ~ 正式な v2.4.0 の間に入ったか、わかるようになんらかの形で
管理したいです。

ですよね。
正式なv2.4.0とalpha1のタグ付けの間は、Add CHANGELOG.md に再度反映されるかなって考えております。
もしくは、alpha1はあくまで検証で、2.3.0~2.4.0のCHANGELOGでもいいのかなとか。

@m-tmatma
Copy link
Member

あと、https://github.com/sakura-editor/sakura/milestone/1 の扱いを考える必要があります。

名前を "next release" から v2.4.0 に変更しました。

@KENCHjp
Copy link
Member Author

KENCHjp commented Mar 28, 2019

appeyor で sha 系のハッシュをすでに生成してたような気がします。

https://github.com/sakura-editor/sakura/blob/master/appveyor.md

ここに説明ありましたね。
sha256.txtってファイルが出来るように見えるのですが、appveyorの成果物から見つけられず・・・

@m-tmatma
Copy link
Member

2.4.0 の alpha 1 用のページ、alpha 2 用のページ、正式版のページとリリース毎に作ったらいいと思います。

えーっと、これは wiki の CHANGELOG のことですか?

そうです。
そうすればどの alpha でどの機能をいれたか追跡可能でかつ、生成を
自動かできて楽かと思います。

@m-tmatma
Copy link
Member

* md5ファイル生成も appveyor で自動化したい

コマンドラインでやる方法はいっぱいあるけど、ハッシュだけ出力はないのだろうか?
https://www.atmarkit.co.jp/ait/articles/0507/30/news017.html

https://docs.python.org/ja/3/library/hashlib.html を使えば python でも書けるが。

@m-tmatma
Copy link
Member

https://docs.python.org/ja/3/library/hashlib.html を使えば python でも書けるが。

既存の sha256.txt は hashlib を使って calc-hash.py で生成している。

@takke
Copy link
Member

takke commented Mar 28, 2019

2.4.0 の alpha 1 用のページ、alpha 2 用のページ、正式版のページとリリース毎に作ったらいいと思います。
そうすればどの alpha でどの機能をいれたか追跡可能でかつ、生成を
自動かできて楽かと思います。

現状の changelog-sakura の設定だと、タグを打った時刻で区切って Issue/PR が抽出されるので、そのままでも alpha1 と alpha2 ~ 正式版のそれぞれの修正が載ることになります。

例えば Release/v2.4.0-alpha1 タグを打った後に閉じたリリース作業のPR #814 は、今朝生成された CHANGELOG.md には Unreleased すなわち alpha2 相当のところに載っています。
(sakura-editor/sakura#814 自体は management ラベルを付けたので次回からは抽出されなくなりますけど)

2019-03-29_00h24_40

CHANGELOG.md には alpha1 等は含めず、正式版の分だけ載せるほうがいいですかね。そうであれば CHANGELOG.md のコミット前の整形がもうひと手間必要になりますね。

@KENCHjp
Copy link
Member Author

KENCHjp commented Mar 29, 2019

OS標準ではいっているので、certutilつかうのがハッシュ値とるだけならいいのかなと。

@m-tmatma
Copy link
Member

sakura.pdb や sakura_lang_en_US.pdb は(debugビルドはともかく)Releaseビルドには不要かなと思うのですがいかがでしょうか。

Releaseビルドを実行中に起きたエラーを調べるには、Releaseの .pdb も必要だと思います。ただ、同じ .zip ファイルに入れるべきかは疑問です。ついでに、sakura-doxygen.chm/chi も別zipでいいのではないかという気がします。
なお、sakura_lang_en_US.dll には実行コードがないので sakura_lang_en_US.pdb はRelease/Debugともに不要だと思います。

sakura-editor/sakura#816

@m-tmatma
Copy link
Member

サクラエディタ側、web 側ともに release ブランチの protect 設定しました。

protect3

protect4

@KENCHjp
Copy link
Member Author

KENCHjp commented Apr 4, 2019

このIssueで決めてた方がいい気がしてきたのですが、2.4.0リリース直後に、一旦バージョンを2.4.1にして、マジリリースしようとするときに、2.4.1でいいくのか、2.5.0にするのか再協議って運用どうでしょう。

@m-tmatma
Copy link
Member

m-tmatma commented Apr 5, 2019

https://github.com/sakura-editor/sakura/milestone/2
v2.4.0 以降にいれるための Milestone を作成しました。
v2.4.0 には入れないことを明示して目標スケジュールの共有に使用ください。

@arigayas
Copy link

v2.4.0 リリースにあたって次のリリース(アルファ2かベータ1)を作成されないのでしょうか?

GitHubに移行して1年以内にv2.4.0がリリースされるのが良いと思っております。

@KENCHjp
Copy link
Member Author

KENCHjp commented May 12, 2019

@arigayas さん コメントありがとうございます。
今見つかってるクリティカル不具合(と私がリクエストしたもの)修正と他の方からの動確あつまったら、alpha2かベータ1リリースをしたいとおもっておりますが、本プロジェクトは強制しないのがモットーかと思っておりますので、旗振り力&誘導力弱い私めはひとまずAzure対応を追いかけ中でございます。

正式リリースを20周年でだせるといいよねとか、桜の日にリリースできればいいよねとか、一瞬もりあがりましたがこればっかりはメンバーの都合というかパワー次第かと。
気長にお待ちいただければ幸いです^^;

@arigayas
Copy link

ユーザーは、のんびり待つしかないのですね(´;ω;`)

@berryzplus
Copy link

「ファイルの排他制御モード」のデフォルトを 「排他制御しない」に変更する by m-tmatma · Pull Request #263 · sakura-editor/sakura が治ってない?

win8.1 pro 64bit, win10 pro 64bit で「再現なし」を確認しました。

再現環境が残っていたら sakura.ini 内の nFileShareMode の値がどうなってるか調べてみてほしいです。予想では =1 になっているはず。(コード的には =0 を書くように変更してます。)

@KENCHjp
Copy link
Member Author

KENCHjp commented May 16, 2019

>win8.1 pro 64bit, win10 pro 64bit で「再現なし」を確認しました。
お、ではもう一度確認してみます。
再現方法は、常駐していないことを確認したのち、iniファイルを消して起動、なのですが、あってますでしょうか。

@berryzplus
Copy link

再現方法は、常駐していないことを確認したのち、iniファイルを消して起動、なのですが、あってますでしょうか。

手順は、①sakura.exe と同じフォルダに sakura.ini がないこと、②起動中の sakura.exe がないことを確認したのち、③sakura.exe を起動、④共通設定で「排他制御しない」を確認、⑤「サクラエディタの全終了」をさせてから⑥sakura.iniを開いてnFileShareModeの値を確認、でした。

こちらでも HyperV のOSイメージをダウンロードして再現テストしてみます。

確か既出だと思いますが、MSがEdgeブラウザのテスト用に無償イメージを提供してるみたいですね。
https://eng-notebook.com/post-2199/


sakura.exe.ini の設定によって挙動が変わることに、今気付きました。
インストーラーはどう制御しているんでしたっけ・・・(詳細は調べてみてから書きます。

@berryzplus
Copy link

Win7 32bitで「再現なし」を確認しました。

先述blog記事のリンク先にあった IE11 - Win7(x86)VirtualBox 版を使いました。
(Hyper-V版は動かなかったのでVirtualBox落としてきて入れました:smile:)
英語OSのまま、D&Dでinstaller.zipをコピって検証してみました。
Win32 Releaseのインストーラー版で日本語にし、オプションはデフォルトのままインストールしました。

初回起動で not exclusive になっていること、
初回起動の終了時に常駐アイコンが残らないことが確認できました。

sakura.exe.ini はデフォルトで「個人用設定にする」だったみたいです。チェックを外してUACに弾かれたとしても、初期設定が反映されないようには見えない(設定が読めないと「排他制御なし」になる)ので原因は別っぽい気がしました。

@KENCHjp
Copy link
Member Author

KENCHjp commented May 16, 2019

@berryzplus さん 確認どうもです。
私も追試します。お手数おかけします。

@KENCHjp
Copy link
Member Author

KENCHjp commented May 17, 2019

別PCがあったのでEXEを展開して確認しましたが、再現しませんでした(ただしく動作します)
私の確認方法に問題があったか環境に何が問題があるようなので、一旦私の確認ミス(おま環)ということにさせてください。
皆様お手数おかけしてすいません。

あとは特に不具合的なの発生してないということですかね。

一件、 
sakura-editor/sakura#909
こういうのはありましたが。
ぼちぼちベータ?

@takke
Copy link
Member

takke commented May 17, 2019

あとは特に不具合的なの発生してないということですかね。

現時点でダウンロード数も798ありますし、alpha版にしては割とたくさん使われているような気がします。

2019-05-17_13h24_02

そろそろ beta1 ですかね?

@KENCHjp
Copy link
Member Author

KENCHjp commented May 17, 2019

releaseバイナリをAzureからにするのかな?、AppVeyorでいいのかな。
正直どっち(AzureとAppveyor)が何の役割になっているか、追い切れてなくて^^;
シーケンス図っぽいのあると嬉しいかな。

@takke
Copy link
Member

takke commented May 17, 2019

Azure-Pipelines は AppVeyor よりも速い(並列ビルドができる)し、柔軟な設定が出来たり比較的分かりやすいUIだったりするので皆さんの好みのような気がしています。「お試しで使ってみたら案外良かった」という感想を皆さん持たれてるんじゃないですかね。

ただ、現時点では sakura-editor/sakura#821 が完了していないので(かな?)、Azure-Pipelines の成果物のフォルダ名が sakura-buildLocal-Win32-Release-Installer になっていたりバージョン番号にビルド番号が含まれないなど、暫定的な対応が残っている状況ですよね。

というわけで今リリースするならまだ AppVeyor からですね。
(Azure-Pipelines と AppVeyor の使い分けについて一度別途Issueで整理したほうが良さそう?)

@KENCHjp
Copy link
Member Author

KENCHjp commented May 17, 2019

@takke さん了解っす。
Web修正しなくっちゃですね、バッサリ過去を捨てるか、最低限だけ修正するかどうしようかなと。。。

@m-tmatma
Copy link
Member

sakura-editor/sakura#906 を修正する必要があります。

@KENCHjp
Copy link
Member Author

KENCHjp commented May 24, 2019

v2.4のヘルプ キーワード検索画面のリストが文字化けしている
sakura-editor/sakura#922

追記しました。pre-releaseと最近のappveyorの成果物見ましたが化けてる模様。

@KENCHjp
Copy link
Member Author

KENCHjp commented May 24, 2019

長くなったので次立てました。
#72
こっちはクローズします。

@KENCHjp KENCHjp closed this as completed May 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Release リリース作業チケット
Projects
None yet
Development

No branches or pull requests

7 participants