一通りできたっぽいのですが、このWipにぶら下げるにはどうしたらいいでしょうか？
m-tmatma:feature/issue209-wip-installer-english
↑これをフォークして↑にPRする？？？

一つだけうまくいってないところがありまして、info.txtこれを言語でだしわけるのがうまくいきません。
現在は、英語と日本語を一つのinfo.txtにして、utf-8のbom付にして逃げてる状況です。

それ以外は、翻訳結果は別（全部google翻訳)にして、iniファイルの英語時のDLL初期設定等一通りできたかなと。

他人のプルリクエストに対して修正をする方法
https://www.ncaq.net/2017/10/16/

他人のプルリクエストに対して修正をする方法

やってみます。

できたっぽいですが、以下変えたことと課題。

• 日本語部分をCustomMessageに追い出し（ただし翻訳はほぼルー語）
• DisableStartupPrompt=yesにしないと、[setup]セクションで{cm:が使えないってエラーになる。
• InfoBeforeFile="instmaterials\info.txt" これを言語で出し分けようとしたんですが{cm:を記述するとCustomMessageに展開されずエラーになる。なのでしょうがなく、info.txtをUTF-8にして日本語と英語両方記述。
• 起動時の常駐のショートカットが(&B)となってたが、戻る(&B)と同じだったので、(&R)に変更。
• なぜかもともと日本語なのに[Types]のEditor with HelpとEditor Onlyが英語だったので日本語追加
• Icon追加用のメニューでInnoに備わってる「{cm:CreateQuickLaunchIcon}」「{cm:CreateDesktopIcon}」があるが、日本語にはショートカットがついてるが英語にはついてない。
• 初期iniファイルを作成し英語の時には、[common]にszLanguageDll=sakura_lang_en_US.dllを追加
• iniファイルを作っちゃうのでアンインストール時に削除する設定追加

info.txtが解決できればほぼほぼ元通りできそう。
ただし英語OSだと、アンインストーラーが消えちゃうのが頻発しちゃうのはなんでだろう。

あと皆様PR内でDIFF結果のソースをインラインで引用してますが、やり方がわからずどなたかおしえてくださいませ。

あと皆様PR内でDIFF結果のソースをインラインで引用してますが、やり方がわからずどなたかおしえてくださいませ。

これって「Files changed」の画面で直接コメント書けばいいって感じですかね？

あと皆様PR内でDIFF結果のソースをインラインで引用してますが、やり方がわからずどなたかおしえてくださいませ。

これって「Files changed」の画面で直接コメント書けばいいって感じですかね？

ぼくもよく分っていませんが...（笑

「Files changed」の画面で行番号のあたりにマウスカーソルを持ってくと「＋」になるので、そこをクリックしてコメントを書けばクリックした行を含む数行(4行くらい？)が引用される仕組みのようです。

マウスカーソルを持ってくと「＋」になるので

やっぱそこなんすね。今度はそれ使ってみます。

英訳についてざっくり突っ込み入れてみました。
こういうの作るときはサクラエディタがすごく便利です。

長いやつを一緒に入れると表が崩れるので外に出しました。

MultiUser
互換モードでインストールします。
システムファイルおよびプログラムをユーザ権限で編集すると
編集結果が他のアプリケーションから参照できないことがあります。(VirtualStore機能)
Install in compatibility mode.
When editing system files and programs with user authority,
editing results may not be accessible from other applications. (VirtualStore function)

これ、日本語の説明が怪しい気がします。
現状の仕様ではシステムファイルを編集することはできないです。
編集結果が他アプリから見えないではなく、意図しない場所に保存されるのはず。
ちょっと隙を見てissue立てる感じで行きたいです。

InitWiz2
設定をユーザ毎に保存するか実行ファイルの場所へ保存するかを選択してください
Please choose whether you want to save the settings for each user or to the location of the executable file

Please choose [選択するモノ] [選択肢]...

うまい代替案が浮かびませんでした。

@berryzplus さん
翻訳ありがとうございます。
設定は「Setting」と以前アプリではしてたのですが、
「Preference」の方が適切そうですよね。
InitWiz1はGoogle先生に伺ったのでSettingになっちゃった感じです。
ひとまず英語メニュー化は、こんな感じで情報共有。

InfoBeforeFile="instmaterials\info.txt" これを言語で出し分けようとしたんですが

これの解決方法を見つける、週末試してみる予定。これで技術的なところは一通りクリアできそう。
後は文言ですね。

sakura_install2-2-0-1.exe の動きと比べてたのですが、この元のとなったissの時点で既に動きが違うみたいです。。。それから、なぜかアンインストールのショートカットが目の前で消えてきます。。。ひとまずエビデンス。

2-2-0-1

日本語

英語

sakura_install2-2-0-1.exeのインストーラの動きから変わってるのはコミットログ見ると、

2015/02/11 23:13:35のInst: インストール時のライセンス表示

の時に変ってるようですが、現在リリースされているsakura_install2-2-0-1.exe（2015-03-01　Ver.2.2.0.1 (Unicode版) ）にはそれが反映されていないようです。
また反映の修正方法が、
InfoBeforeFile="instmaterials\info.txt"
に書いてありますが、licenseなら、
LicenseFile=license.txt
こっちに書くのがただしそう、ただ2-2-0-1のインストーラーでは表示されていなかったのと、
文章も怪しいと @berryzplus さんがいわれてたので、いっそこれ表示しなくしてもいいのかなと思うのですが、皆様いかがでしょう？？？

また、Inno setupの仕様が変わったのか、2-2-0-1では表示されていた、Welcome画面が表示されません。

DisableWelcomePage=no

を入れると同じ振る舞いをします。Inno setupのデフォルトの振る舞いがどっかから変わってしまったのかもしれません。

:

まだ見落としがありました、スタートアップのショートカット名に不具合あり修正しました。
:

文章も怪しいと @berryzplus さんがいわれてたので、いっそこれ表示しなくしてもいいのかなと思うのですが、皆様いかがでしょう？？？

すいません、 @berryzplus さん指摘箇所とは違いますね。失礼しました。
とはいえ、sakura_install2-2-0-1.exe の時には出てなかったメッセージなんすけども。
とりあえずインストーラーの英語（手段）化はひとまず区切りで、内容はまた別途って感じですかね。

@m-tmatma さん、これ次のアクションどうします？
WIPなのでこのままクローズ？（誰かが別PRする？)
もうちょっとまってこのままコミット？

一応アンインストーラーが消えるのは最近再現しない模様、不思議。。。

@m-tmatma さん、これ次のアクションどうします？
WIPなのでこのままクローズ？（誰かが別PRする？)
もうちょっとまってこのままコミット？

状況的にマージできる状態ですか?
それとも何か調査すること残ってますか?

それとも何か調査すること残ってますか?

調査するところはないです。
既存（本修正前）とあとでほぼ動きは変わらないところまで実装できてるとおもいます。

2-2-0-1のインストーラとすでに動きが違ってる点が気になりますが(重要な情報画面が表示される)、多分当時手動でインストーラ作ってると思うので経緯はわからないきがします。
#359 (comment)
調べた結果記載。

できたら第三者でインストール　アンインストールの疎通日本語英語両方確認していただけると安心かと思ってます。

2-2-0-1のインストーラとすでに動きが違ってる点が気になりますが、多分当時手動でインストーラ作ってると思うので経緯はわからないきがします。

この PR の対象外だけど気になっている点を別の issue として登録していただけますか?
この PR の目標が達成できているのであれば、タイトルの WIP を取ってみんなでレビューしましょう。

この PR の対象外だけど気になっている点を別の issue として登録していただけますか?

了解です。
翻訳漏れてるところは無いかと思うので（翻訳が正しいかは別（笑））、別Issue立てたら、こっちへリンクします。
週末になりそうですが。

{cm:AutoStartProgram}はインストーラを起動したときに{cm:AppName} を自動的に起動するか? ということなので、もともとの 起動時に常駐 とは意味が違うように思います。

常駐するかというはなしと、インストーラーを終了したときに、起動するかが意味が違うということをおっしゃってるイルの認識でよろしいでしょうか？

これを分けたいということでしょうか？（常駐するしないと、すぐに起動したいのどうかってことを両方ともユーザーに聞き分けたいということでいいでしょうか？）

ごめんなさい、それをみつけられないです。ReadyMemoってどこにあるキーワードですか？

ここら辺です。

{cm:AutoStartProgram}はインストーラを起動したときに{cm:AppName} を自動的に起動するか? ということなので、もともとの 起動時に常駐 とは意味が違うように思います。

常駐するかというはなしと、インストーラーを終了したときに、起動するかが意味が違うということをおっしゃってるイルの認識でよろしいでしょうか？

そうです。実際の動作とメッセージが異なっています。
仕様を変えるという話ではないです。

ここら辺です。

了解です。私がつけた名前ですね。
単なる名前なので、適切なものがあれば改変願います。

そうです。実際の動作とメッセージが異なっています。
仕様を変えるという話ではないです。

了解しました。

このPRは英語インストーラーにするPRかと存じます。
ご指摘の挙動は、残念ながら私が調査するにはそれなりの時間を要する気がしています。
ご要望の挙動を満足しつつこのPRをフィックスするには時間的リソースが私にはないので、
かなりおまちいただけることなるかとぞんじます。
すいません。

ReadyMemo ってどういう意味ですか?

これって UpdateReadyMemo という関数名から来てるんですか?
何をする関数かわかりますか?

ご指摘の挙動は、残念ながら私が調査するにはそれなりの時間を要する気がしています。
ご要望の挙動を満足しつつこのPRをフィックスするには時間的リソースが私にはないので、

日本語のメッセージに関しては、もともとの 起動時に常駐(&B) がそのまま使われるように
すればいいと思います。

これって UpdateReadyMemo という関数名から来てるんですか?

単に名前です。各ダイアログのページの命名から来ている認識でおります。

function UpdateReadyMemo

これからですね。

日本語のメッセージに関しては、もともとの 起動時に常駐(&B) がそのまま使われるようにすればいいと思います。

そうですよね。
それがですね、なんだか、時と場合で表示されないようなきがしておりまして、それを調べないと決着しない気がしております・・・

そうなんです、日本語のリソースを単に英語にするってところは終わってる認識なんですが(英語があってるかどうかは置いておいて)、
そもそもの動きでどうだって話＋Windows10でメニューが勝手に消えちゃう問題とあわさると、
こいつってずっとクローズしなと思います。
いやいいんですよ、ずっとクロースしなくても、それが正しい動きを追い求めているのであれば。

そうなんです、日本語のリソースを単に英語にするってところは終わってる認識なんですが(英語があってるかどうかは置いておいて)、

{cm:AutoStartProgram} というのは、InnoSetup が定義しているメッセージのようです。
http://www.jrsoftware.org/ishelp/index.php?topic=custommessagessection

動きがよくわからないものは、InnoSetup が定義している値を使わずにカスタムの
メッセージを使った方がいいと思います。

その上で、この PR が終わった後に、標準のメッセージに差し替えが可能なものがあれば
標準のメッセージに差し替えるようにしてはいかがでしょうか?
その方が対応が楽だと思います。

@m-tmatma さん、了解です。
また週末になりそうですがやってみます。

修正コミット積みました。
{cm:CreateQuickLaunchIcon}
{cm:CreateDesktopIcon}
{cm:UninstallProgram,{cm:AppName}}
はInnoSetupのキーワードですが仕様が明示かと思われるのでそのままにしてます。

対応ありがとうございました。
問題ないです。

もともと私の投げた PR なので、approve できないみたいです。
@KENCHjp さん以外方、確認および approve お願いします。

説明欄の WIP 削除したのですが、
WIP のチェック更新されない。

Marge ボタン押せたので押してみました。
なんか、不都合あったら、再度後追いPRしましょう。