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

Fix bad permissions of calibre when installing as administrator - issue #14282 #14312

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

Conversation

arberg
Copy link

@arberg arberg commented Oct 28, 2024

Fixes the problem of regular user unreadable calibre install.

Comments from issue

Possible Solution
The problem is its being unpacked in Temp storage and then moved.
The temp storage of the Administrator accoun to does not give files Users readable permissions.

I have fixed this in a pull-request by running this afterwards:

Icacls $dir /t /c /reset

I have tested it on my environment and it works fine.

I have also tested behaviour of windows writing files to temp-storage, and then manually copying to scoop-install-dir, and can see that all files and folders recursively do not get the user-readable permission after copy, so a recursive reset is required.

I'm running an up2date win11.

Scoop and Buckets Version

Closes #14282

Copy link
Contributor

All changes look good.

Wait for review from human collaborators.

calibre

  • Description
  • License
  • Hashes
  • Checkver
  • Autoupdate
  • Autoupdate Hash Extraction

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

Successfully merging this pull request may close these issues.

[Bug] Calibre unreadable permission for standard user after ever install/upgrade with for global install
1 participant