レビューお願いします。

いっちゃいましょう。

‪コードの正当性はチェックできていませんが、サクラエディタ本体に影響するものではないので今回のPR内容に仮に問題があったとしても害はほぼ無いと考えています (必要であれば後から修正すれば良い)。‬

‪ただ、動作チェックレビューはしたほうが良いと思っています。‬

‪以下URLから今回の成果物であるインストーラが含まれるzipを入手できるはずですので、それのインストーラ挙動が想定通りになっていることが確認できれば approve で良いと思います。‬

‪https://ci.appveyor.com/project/sakuraeditor/sakura/build/1.0.194/job/u2gs8i05pp2g2rg7/artifacts‬

‪すみませんが僕はインストーラ関連の議論に参加していなかったため、どのような挙動が正解であるのか判断できません。‬

‪そのあたり分かる方、チェックいただけると助かります。‬

動作チェックレビューはしたほうが良いと思っています。

そっすね😅

パッケージラス版と今回版をまっさらで（AppDataもクリアして)インストールした状態を各々とっといて、Diffすかね。

パッケージラス版と今回版をまっさらで（AppDataもクリアして)インストールした状態を各々とっといて、Diffすかね。

そこまでしていただけると超絶理想ですが、
自分としては「インストールがちゃんとできること」「インストールしたものが動くこと」「インストールによりできあがったスタートメニューのスクショを共有」「インストールによりできあがったフォルダ内容物をスクショで共有」くらいで良いと思ってました。

diff するとして、仮に差分が出るようなことがあったとしても、今回の PR ではあくまでも「インストーラのビルドが通ること（そしてちゃんとそれが動くこと）」がクリアできていれば LGTM としてマージしちゃって良いと思います。

‪https://ci.appveyor.com/project/sakuraeditor/sakura/build/1.0.194/job/u2gs8i05pp2g2rg7/artifacts‬

Build not found or access denied.
ってでちゃいます。。。

あと、現インストーラーがなぜかアンインストールのショートカットがスタートメニューに追加されるときとされないときがある(イミフ)。。。これ落ち着いて、再現するようならIssueたてよかな。。。

https://ci.appveyor.com/project/sakuraeditor/sakura/build/1.0.194
あれ、ここからとんでみたら見れますか？

artifacts のところ。

URLを直に貼りましたが、実際にはこのPR内のAll checks have passed
AppVeyor build succeeded
の横のDetailsから成果物が見れるようになっている、はずです

ひとまず、ダウンロードできたことを報告。実行できたのでエビデンスはります。

ランダム文字列のついてるURLは一時リンクだったぽいですね。失礼しました（　´'Ａ'｀）

まだ公知じゃないから、インストーラー動かすとWindows Defenderで警告出ちゃう。

詳細情報表示で実行ってやると後は同じ。

sakura_install2-2-0-1.exe動かすとたまにこうなっちゃう。

sakura_install2-1-1-2.exe今回ビルドしたやつ。

展開された中身。

DFで中身比較。

全部開いてみました。

意図した感じでしょうか？
アンインストールも正常に全部消せることを確認（AppDateはサクラエディタを実行するとiniファイルがアンインストール後も残り続けます、現バージョンも同じ動き）

ご確認ありがとうございます！

気になるところは数点あるにしろ、一旦の叩き台としてはこのPRはこれでOKってことでも個人的には良いと思います。叩き台がないと諸々微調整をする場所がなくて困りますし。(このPR単体だけで理想挙動を仕上げるのは現実的ではない)

ただ、インストーラ議論に自分が参加していなかった以上は自分以外の方の判断で approve するかを決めていただきたいです。僕の話はあくまでも判断のヒント程度に受け取ってください。

#151 の対応を入れないとマージのときに
ビルド通らないかも

ディフェンダーが反応するのは別の理由だと思います。回避策は出来たexeをzipに詰める。だったような。

ちなみにこの件は、ぼくも内容をウォッチ出来てないです。今のところ動くならええんちゃう？としか言えない感じ。

#151 の対応を入れないとマージのときに
ビルド通らないかも

rebase しました。

@KENCHjp さん

まだ公知じゃないから、インストーラー動かすとWindows Defenderで警告出ちゃう。

https://ci.appveyor.com/project/sakuraeditor/sakura/build/1.0.208/job/fhbs8mhk9s8qlrlb/artifacts
で試しましたが、Windows Defender は出なかったです。

これでも出ますか?

今日はもう出先なのでまた明日に。

@berryzplus さん

ディフェンダーが反応するのは別の理由だと思います。回避策は出来たexeをzipに詰める。だったような。

いま成果物はzip圧縮されてるものをダウンロードしてからデスクトップに解凍（ドラッグドロップ）して実行しているのでこれじゃなさそうです。

@m-tmatma さん

https://ci.appveyor.com/project/sakuraeditor/sakura/build/1.0.208/job/fhbs8mhk9s8qlrlb/artifacts
で試しましたが、Windows Defender は出なかったです。

なにか環境が違うのかなぁ、会社のVMマシンと自宅のマシン両方ともでます。（会社は32bit、自宅は64bit)両方とも、Windows Defender以外のワクチンソフトは入ってない環境です。

単体の sakura.exe を起動しても出ますか?

@m-tmatma さん

単体の sakura.exe を起動しても出ますか?

でますね。
仕様じゃないっすかね。ネット(ファイル共有も含む)からダウンロードした実行バイナリって警告でますよね。

このチェックつけたら警告でません。

zip版やインストーラをダウンロードしたら、解凍or実行する前に「許可する」のチェックを入れるというのを、インストール手順書か何かに書いておくのがよいと思います。

zipでそれをせずに解凍してしまうと、解凍後のファイルもすべてダウンロードしたファイルとして扱われます。

ブラウザって何使ってますか？
私は Edge です

私はChromeですね。Edgeでも試しましたが、状況同じです。

ちなみに、Windows10だとWindows Defenderの画面がでて派手ですが、Windows7の以下と動きは同じです。

検証してませんが、Windows7だとネットでダウンロードしたものはかならず警告がでて、
Windows10の場合「警告が出る可能性があります」とあるので、Windows Defenderが対象が公知であればこの警告を出さなくしてくれるのかなと。
エクスプローラーで解凍やドラッグドロップするとフラグが付いたままですが、ファイラー等でzipを解凍するとこのフラグが消えてるファイルができあがります。

エクスプローラーで解凍やドラッグドロップするとフラグが付いたままですが、ファイラー等でzipを解凍するとこのフラグが消えてるファイルができあがります。

なるほど。7z で解凍してました。エクスプローラで解凍すると警告出ました。

警告の件は #166 で登録しました。

zipでそれをせずに解凍してしまうと、解凍後のファイルもすべてダウンロードしたファイルとして扱われます。

これマジで知りませんでした。
そういう仕組みだったんですね。

なるほど。7z で解凍してました。エクスプローラで解凍すると警告出ました。

なかーまｗ

ダウンロードした zip は一旦開いて階層構造を確認してから展開するようにしています。
圧縮方法によって、最上位フォルダがあったりなかったりしてどれが展開ファイルか分からなくなるとあれなので。

インターネットからダウンロードしたファイルに、データとは別にNTFSストリームをくっ付けるのはwindowsの仕様だった気がします。
http://www.atmarkit.co.jp/ait/articles/1407/11/news111.html
https://qiita.com/minr/items/c2393f532b2df35f7a9d

インターネットからダウンロードしたファイルに、データとは別にNTFSストリームをくっ付けるのはwindowsの仕様だった気がします。

ですねー。昔はサブストリームって言ってたと思ったのに、最近は代替データストリーム（ADS）っていうんですよねぇ。。。

ダウンロードした zip は一旦開いて階層構造を確認してから展開するようにしています。

私は、DWってファイラーで中を確認してたりします(笑)。
http://st.sakura.ne.jp/~higashi/dw.htm