LibreBooking / docker

Librebooking as a docker container
GNU General Public License v3.0
11 stars 12 forks source link

Container can not be restarted #65

Closed yenyen closed 1 year ago

yenyen commented 1 year ago

Hello,

When the container is restarted, it throws an error and stops :

/config /var/www/html
/var/www/html
+ '[' -f /config/config.php ']'
+ '[' -f /var/www/html/config/config.php ']'
++ find /var/www/html/plugins -type f -name '*dist*'
+ for source in $(find /var/www/html/plugins -type f -name "*dist*")
++ echo /var/www/html/plugins/Authentication/Moodle/Moodle.config.dist.php
++ sed -e s/.dist//
+ target=/var/www/html/plugins/Authentication/Moodle/Moodle.config.php
++ basename /var/www/html/plugins/Authentication/Moodle/Moodle.config.php
+ cp --no-clobber /var/www/html/plugins/Authentication/Moodle/Moodle.config.dist.php /config/Moodle.config.php
++ basename /var/www/html/plugins/Authentication/Moodle/Moodle.config.php
+ chown www-data:www-data /config/Moodle.config.php
++ basename /var/www/html/plugins/Authentication/Moodle/Moodle.config.php
+ ln -s /config/Moodle.config.php /var/www/html/plugins/Authentication/Moodle/Moodle.config.php
ln: failed to create symbolic link '/var/www/html/plugins/Authentication/Moodle/Moodle.config.php': File exists

Tested on versions: 2.8.5.5, 2.8.6 and develop

colisee commented 1 year ago

Hi @yenyen,

Thank you very much for your PR. I applied it, but I had to make a little correction. Could you please check the new published image(s) librebooking/librebooking:{2.8.5.5, 2.8.6.1, develop}-2.3.2 ?

yenyen commented 1 year ago

Hi @colisee,

Thank you for reactivity, I tried the version 2.8.6.1 and it's working perfectly.

Thank you again.

colisee commented 1 year ago

Excellent!