Fix permissions and minor changes #19
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request introduces significant updates to improve Linux compatibility by dynamically handling user permissions in the Docker environment, along with several other enhancements related to usability and configuration. The most important changes include refactoring the Docker setup to support dynamic UID/GID, modifying the default behavior of Makefile commands, and introducing a new script for detecting user permissions.
Linux Compatibility Enhancements:
Dockerfile
: Refactored user creation to dynamically handle UID/GID for Linux systems, ensuring compatibility with existing users and groups. Added logic to adjust permissions for home directories and symlinks.docker-compose.yml
: AddedUSER_UID
andUSER_GID
arguments to support dynamic user permissions in the Docker container.scripts/detect-permissions.sh
: Introduced a new script to detect and export user permissions (UID/GID) based on the operating system.Makefile Improvements:
Makefile
: Changed the default target fromhelp
todefault
, which displays all available commands. Removed thelogs
target and added logic to detect user permissions for Linux compatibility. [1] [2] [3]Documentation Updates:
README.md
: Updated references to the defaultmake
command and adjusted examples to reflect the removal of thehelp
target. [1] [2]docs/COMMANDS.md
: Replaced references tomake help
withmake
and removed thelogs
command from the list of available commands. [1] [2] [3]Version Bump:
VERSION
: Incremented the project version from1.3.2
to1.3.3
.Script Adjustments:
scripts/build.sh
: Integrated the newdetect-permissions.sh
script to dynamically configure user permissions during the build process. Updated Docker commands to passUSER_UID
andUSER_GID
as environment variables. [1] [2] [3] [4]scripts/configure.sh
andscripts/destroy.sh
: Replacedecho
withprintf
for consistent formatting and improved script robustness. [1] [2] [3] [4]