Skip to content

Fix issue 169 - #184

Merged
colisee merged 8 commits into
LibreBooking:masterfrom
colisee:issues/169
Mar 16, 2026
Merged

Fix issue 169#184
colisee merged 8 commits into
LibreBooking:masterfrom
colisee:issues/169

Conversation

@colisee

@colisee colisee commented Mar 15, 2026

Copy link
Copy Markdown
Collaborator

Changes

  • git is not needed in the image
  • setup.sh does not need to be in the image
  • librebooking files retrieved by Dockerfile
  • Dockerfile fills file config/custom-version.txt with the output of git describe --tags --long

Close #169

Robin Alexander added 4 commits March 13, 2026 17:42
Dockerfile: in charge of retrieving librebooking files and SHA
setup.sh: librebooking customization
The last commit id matches the last 7 characters of
`git describe --tags --long`
@colisee colisee added the enhancement New feature or request label Mar 15, 2026

@ikke-t ikke-t left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Only the cosmetic change of regexp to comment. Looks good overall. Change it or not, I approve

Comment thread Dockerfile Outdated
Robin Alexander added 3 commits March 15, 2026 17:45
The length of the short commit id is not always 7 characters.
- Argument "--keep-git-dir" not recognized by podman build
- Implement git clone inside the alpine/git image section
- Implement "config/custom-version.txt"
- Remove ".git" directory upstream version is saved
Comment thread Dockerfile Outdated
Comment thread Dockerfile Outdated
Comment thread Dockerfile Outdated
Comment thread Dockerfile

@JohnVillalovos JohnVillalovos left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks @colisee

LGTM

@colisee
colisee marked this pull request as ready for review March 16, 2026 07:47
@colisee
colisee merged commit f96c573 into LibreBooking:master Mar 16, 2026
3 checks passed
@colisee
colisee deleted the issues/169 branch March 16, 2026 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Check Dockerfile does not install useless software

3 participants