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 chown and permission issues #19

Merged
merged 12 commits into from
Oct 24, 2024
Merged

Fix chown and permission issues #19

merged 12 commits into from
Oct 24, 2024

Conversation

feederbox826
Copy link
Owner

@feederbox826 feederbox826 commented Oct 23, 2024

size comparison

alpine: 791.47 -> 434.53 (-356.94MB)
hwaccel: 1380 -> 1120 (-260MB)

entrypoint.sh

🔑 chown and permissions management

  • remove SKIP_CHOWN
  • only set u=rwx and owner when chown-ing
  • split into reown/reown_r for non-recursive chown

🧩 stashapp/stash compatibility mode

  • add consistent STASHAPP_STASH_CONFIG
  • more robust checks for permissions
  • don't fake ROOTLESS

🚚/🚛 Migration

  • move config backups as well

🐍 Python

  • be picky when finding python reqs
  • drop bencoder.pyx for fastbencode
  • drop build dependencies for python

📋 Startup checks

  • add check for common directories
  • be more verbose with error logs

🎭 user changing

  • default to 911:911 in docker image as well
  • log explanation if root is preserved

other

  • remove unnecessary curly brackets

Documentation

  • add develop tag
  • fix typo for NVENV -> NVENC
  • reformat readme
  • update branding

🐳 docker misc changes

  • remove unused UV_VERSION
  • fix arm64 installing intel drivers

- remove SKIP_CHOWN
- default user to 911/911
- remove curly brackets from unnecessary variables
-  only set u=rwx and owner when chown
- run directory test as PUID target
- simplify migrate_update
- add a check for common directories
- use double brackets where possible to simplify
- add STASHAPP_STASH_CONFIG
- use error when there are breaking errors
- update branding
- be picky when finding python reqs
- explain current user status with logs
- use stat to mirror stash behaviour
- don't fake ROOTLESS
- fix runas logic
@feederbox826 feederbox826 merged commit 9153713 into main Oct 24, 2024
@feederbox826 feederbox826 deleted the permission-woes branch October 24, 2024 01:03
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.

1 participant