Skip to content

Fix issue 169#184

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

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

Conversation

@colisee
Copy link
Collaborator

@colisee colisee commented Mar 15, 2026

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
Copy link
Collaborator

@ikke-t ikke-t left a comment

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

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
Copy link
Collaborator

@JohnVillalovos JohnVillalovos left a comment

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