Relax bash and openssl requirements in the setup script

AtomGraph / LinkedDataHub

The low-code Knowledge Graph application platform. Apache license.

https://atomgraph.github.io/LinkedDataHub/

Apache License 2.0

485 stars 122 forks source link

Relax bash and openssl requirements in the setup script #116

Closed namedgraph closed 1 year ago

namedgraph commented 2 years ago

96a8769e7b7e30c8b6dcb845f349fff95a8740dc introduced checks for minimum versions of Bash (4) and OpenSSL (1.1.1) in the setup.sh script.

This will probably backfire by the checks themselves failing... MacOS seems to have libressl instead of openssl.

We should get rid of the associative arrays in the script to stay compliant with Bash 3.x (and other shells). OpenSSL can be replaced by Java's keytool.

namedgraph commented 2 years ago

Disabled the checks for now.

namedgraph commented 2 years ago

The OpenSSL part is now addressed by c869e2450aa899078b1ba4992f016451607077a6 in the develop branch.

The script now also runs on zsh because of the fd5dab1b08e807e007c07a0a904a665bdc2ef862 fix:

I switched the script to zsh (and commented out the bash version check) and it worked fine so if you can detect when you're on osx and use zsh instead it might work out of the box.