マルチユーザ設定を有効にする

[berryzplus]

exe.iniのデフォルト値を変更します。

このファイルはインストーラのテンプレートとして使われるファイルなので、変更によるインストーラへの影響はないと考えられます。

変更を適用すると、このファイルを Debug にコピーするだけで、
インストール済みの SAKURA editor の設定を流用することができるようになります。

設定の意味については、変更対象のiniに書かれているコメントを参照してください。

[ds14050]

「インストール済みの SAKURA editor の設定」は熟成された秘伝のタレのようなものなので、ぽっとダウンロードしてきた sakura.exe や、テキトーにビルドした sakura.exe に破壊されてはダメージが大きすぎます。

なので、マルチユーザー設定を有効に変更された resource/sakura.exe.ini がどのようにユーザーに露出されるのか、または露出することがないのかだけを知っておきたいです。

なんとなく「露出することはない」し、インストーラがユーザーの選択に応じて書き換えるので「値の違いにも意味がない」気はしています。

[berryzplus]

なんとなく「露出することはない」し、インストーラがユーザーの選択に応じて書き換えるので「値の違いにも意味がない」気はしています。

そうです。

いまは sakura.exe.ini がなかった場合に選択される設定が入ってるのでそれを変えたいです。
現状だと、ファイルをコピーしたうえで設定を編集しないといけなくて二度手間なので。

デバッグ中のini破壊事故が起こるのが怖いので、
この作業をビルドプロセスに組み込むことは考えていません。

[ds14050]

要は、コピーするだけで使える sakura.exe.ini が転がっていると便利、という目的の変更だと理解しています。

approve しようと思いましたが「このファイルはインストーラのテンプレートとして使われるファイルなので、変更によるインストーラへの影響はないと考えられます。」部分の検証が自分にはできません。

たとえば installer/sakura-common.iss に

Source: "sakura\sakura.exe.ini";       DestDir: "{app}";                  Components: main; Check: isMultiUserEnabled; Flags: onlyifdoesntexist;

という行がありますが、その意味がわからないのですが、条件と結果が不整合を起こしたりしないのでしょうか。

[berryzplus]

要は、コピーするだけで使える sakura.exe.ini が転がっていると便利、という目的の変更だと理解しています。

その通りです。

approve しようと思いましたが「このファイルはインストーラのテンプレートとして使われるファイルなので、変更によるインストーラへの影響はないと考えられます。」部分の検証が自分にはできません。

これは「たぶん大丈夫」と考えていますが、正直いうと少し怪しいです。
で、ちょっと pascal のコード読んでみました・・・。

以下は指摘部分の引用をテキトーに改行したものです。

Source: "sakura\sakura.exe.ini"; // コピー元はこのファイルだよ
  DestDir: "{app}";              // 出力先はappだよ
  Components: main;              // このファイルはmainの一部だよ
  Check: isMultiUserEnabled;     // 指定された条件がみたされたらコピーするよ
  Flags: onlyifdoesntexist;      // 出力先にファイルがないときだけコピーするよ

iss は inoo setup のコードなので、記述言語は pascal script です。

言語の構文をよく分ってなくても、これがプログラム言語であるなら、
処理結果を予想することは容易です・・・。
ソースファイルに変更を加えたら、処理結果に影響しないはずがない。

しかし、このPRは処理結果には一切の影響を与えません。
何故なら・・・

sakura/installer/sinst_src/sakura.exe.ini

Lines 26 to 28 in 18763d8

	MultiUser=1
	UserRootFolder=0
	UserSubFolder=sakura

インストーラがコピーするファイルは別のファイルだからです。

変更対象のファイルは sakura-editor/resources/sakura.exe.ini なので、使われていないファイルです。
未使用だから削除、でもいい気はしますが、せっかくこの場所にあるので有効に活用したいです。

[ds14050]

インストーラがコピーするファイルは別のファイルだからです。

そうだったんですね。

実は挙げた部分「Check: isMultiUserEnabled」がすでに resource/sakura.exe.ini の内容と整合していないではないかと不思議に思っていました。sakura/installer/sinst_src/sakura.exe.ini の内容とは整合していますね。

resource/sakura.exe.ini が sakura_rc.rc でインクルードされているということもないようなので approve してしまいます。

しかし resource/sakura.exe.ini は削除してしまって installer/sinst_src/sakura.exe.ini をコピーして利用すればいいのではありませんか。

[berryzplus]

レビューありがとうございます。

しかし resource/sakura.exe.ini は削除してしまって installer/sinst_src/sakura.exe.ini をコピーして利用すればいいのではありませんか。

だって、インストーラの資材は階層が深いんですもの、めんどくさくてｗ

[ds14050]

そんな気はしました(笑)。この PR を見られたことで resource/sakura.exe.ini が消されかけても私は関知しませんよ。

[KENCHjp]

めんどくさくてｗ

自分の守備範囲でないのを見るのは同じコード量でも心労度合い全然ちがいますよね(笑)
インストーラーをmakeするバッチ(build-installer.bat)を見ていただくと、どこにあるファイルをどうおぜん立てするかわかります。

[ds14050]

KENCHjp さん、共感してはだめですw berryzplus さんはコピペの手間を厭っています。

[berryzplus]

これが適用されてもコピペまではするんですけどね。
適用前はコピペしたあとさらに編集しないといけないのが面倒だったんです。

[berryzplus]

レビューありがとうございます。マージしてしまいます。
少なくとも設定保存機構の改善が終わるまで、消されないと嬉しいです。

[ds14050]

適用前はコピペしたあとさらに編集しないといけないのが面倒だったんです。

installer/sinst_src/sakura.exe.ini の中身は最初からマルチユーザー設定が有効ですからコピペだけで済むはずです。フォルダを１階層余分に潜るのが面倒なのだとばかり。