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

管理者権限非保持ユーザがインストーラーを使用するとエラーが発生する問題を修正(#1919) #1962

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

iqc-cpi
Copy link

@iqc-cpi iqc-cpi commented Jul 8, 2024

PR対象

  • インストーラ

カテゴリ

  • 不具合修正

PR の背景

#1919

仕様・動作説明

PR の影響範囲

インストーラ

テスト内容

build-all.batで作成したインストーラ(32bit版)で、以下動作確認
・管理者権限でのインストール
・非管理者権限でのインストール

テストOS環境: Windows11 Pro

関連 issue, PR

#1919

参考資料

Copy link

sonarcloud bot commented Jul 8, 2024

@AppVeyorBot
Copy link

Build sakura 1.0.4359 failed (commit 92b22bfcb7 by @iqc-cpi)

berryzplus
berryzplus previously approved these changes Jul 8, 2024
Copy link
Contributor

@berryzplus berryzplus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTMです。

課題的なもの2点、一応書き留めておきます。

  • push元ブランチ名が日本語を含むため、警告が出ています。国際化対応が不十分であるの「GitHubの不具合」ですが、利用者側で対応するものだと思います。個人的にはどちらでもよいのでスルーします。
  • 従来 HKLM に登録していたものを HKCU に登録する対応です。旧バージョンをインストール済みの端末に新バージョンをインストールした場合、HKLM のエントリーが削除されません。考慮すべき点がいくつかあると思うのですがスルーします。

@berryzplus berryzplus dismissed their stale review July 10, 2024 23:51

えいや、で入れられるほど軽くない変更のようなので、apporove外しておきます。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants