MariaDB / mariadb-docker

Docker Official Image packaging for MariaDB
https://mariadb.org
GNU General Public License v2.0
759 stars 438 forks source link

set password hash #459

Closed TheAlgorythm closed 1 year ago

TheAlgorythm commented 1 year ago

458

grooverdan commented 1 year ago

Good start, thanks. I made the following changes (which are pushed onto your github fork):

TODO:

TheAlgorythm commented 1 year ago

Somehow ./update.sh doesn't work on my machine:

update.sh: line 8: 10.3: syntax error: invalid arithmetic operator (error token is ".3")
update.sh: line 10: focal: unbound variable

Update

It was caused by MacOS using a old version of bash. But now it has another problem with sed:

sed: -e: No such file or directory

It looks like it was caused by MacOS using BSD-sed by default and not GNU-sed.

TheAlgorythm commented 1 year ago

I am not sure if I will be able to understand the test failures as I am neither a Docker nor a Bash expert.

TheAlgorythm commented 1 year ago

This seems to be a CI problem and not related to my changes as they are also on the MAIN branch. I think that all should be done, besides a new ./update.sh run (see above).

Update

I was able to update the subdirectories.

grooverdan commented 1 year ago

Thanks for trying it out and testing it some more. I'll take it from here and get this finished.

Thanks for describing the update.sh portability issues on MacOS, I'll see if I can correct these.

grooverdan commented 1 year ago

no idea about the CI failures. Where there before this was added. Thanks @TheAlgorythm. Doing a release with this now.