openFrameworks icon indicating copy to clipboard operation
openFrameworks copied to clipboard

scripts/linux/ubuntu/install_dependencies.sh: lsb_release might not be installed before it is used in the script

Open shvaikop opened this issue 1 year ago • 1 comments

Hi,

I had an issue with scripts/linux/ubuntu/install_dependencies.sh. The call to lsb_release in

if [ "$ID" = "elementary" ]; then
	# Gets ubuntu base version
	RELEASE=$(lsb_release -r -u)
elif [ "$ID" = "linuxmint" ]; then
	# Gets ubuntu base version
	RELEASE=$(cat /etc/upstream-release/lsb-release | grep DISTRIB_RELEASE | cut -d "=" -f2)
else
	RELEASE=$(lsb_release -r)
fi

returns error if it is not installed and RELEASE ends up being empty resulting in wrong code path later on.

It's not a big deal but I noticed it while trying to automate the build process inside a docker container.

I think it would be a nice idea to try to install the tool with apt-get update && apt-get install -y lsb-release before it is used. What do you think?

Thanks

shvaikop avatar Mar 31 '24 19:03 shvaikop

Sure thing! do you want to submit a PR to address this issue?

dimitre avatar May 09 '24 01:05 dimitre