From mboxrd@z Thu Jan  1 00:00:00 1970
From: Jonatan Schlag <jonatan.schlag@ipfire.org>
To: development@lists.ipfire.org
Subject: Re: [PATCH 12/21] initscript functions: add readhash
Date: Sun, 02 Jun 2024 20:05:54 +0200
Message-ID: <965e8b76a9cf3f06ebbcfafbb3e82e910576983e.camel@ipfire.org>
In-Reply-To: <FF86902E-6928-4E2F-B05C-0A94188FDF06@ipfire.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============2465838815059471938=="
List-Id: <development.lists.ipfire.org>

--===============2465838815059471938==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

Hi,

Am Freitag, dem 31.05.2024 um 10:53 +0100 schrieb Michael Tremer:
> Hello,
>=20
> > On 20 May 2024, at 10:06, Jonatan Schlag
> > <jonatan.schlag(a)ipfire.org> wrote:
> >=20
> > To avoid the usage of eval and to store the config in an key value
> > array, we introduce an new function. The tests only check if we
> > read the correct value to the correct variable.
> >=20
> > One comment on the implementation as this has created some
> > headache:
> >=20
> > > From
> > > https://www.gnu.org/software/bash/manual/bash.html#Bourne-Shell-Builtins
> >=20
> > "When used in a function, declare makes each name local, as with
> > the local command, unless the -g option is used."
> >=20
> > So we need to use -g here
> >=20
> > Signed-off-by: Jonatan Schlag <jonatan.schlag(a)ipfire.org>
> > ---
> > src/initscripts/system/functions=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=
=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 | 15 +++++++++++++++
> > tests/src/initscripts/system/functions/data/1=C2=A0 | 17
> > +++++++++++++++++
> > tests/src/initscripts/system/functions/test.sh | 16
> > ++++++++++++++++
> > 3 files changed, 48 insertions(+)
> > create mode 100644 tests/src/initscripts/system/functions/data/1
> > create mode 100755 tests/src/initscripts/system/functions/test.sh
> >=20
> > diff --git a/src/initscripts/system/functions
> > b/src/initscripts/system/functions
> > index b610143ab..44ce999d3 100644
> > --- a/src/initscripts/system/functions
> > +++ b/src/initscripts/system/functions
> > @@ -891,3 +891,18 @@ volume_fs_type() {
> >=20
> > stat -f --format=3D"%T" ${1}
> > }
> > +
> > +readhash() {
> > + local array=3D"${1}"
> > + local file=3D"${2}"
> > +
> > + declare -A -g "${array}"
> > +
> > + local line
> > + while read -r line; do
> > + local key=3D"${line%=3D*}"
> > + local val=3D"${line#*=3D}"
> > +
> > + printf -v "${array}[${key}]" "%s" "${val}"
> > + done < "${file}"
> > +}
>=20
> Okay, so here we are getting to the main bit.
>=20
> I think this is nice and simply. The function creates the array
> regardless and would empty any content which is what we want.

Thank you.
>=20
> You should however check if the file exists and can be read as the
> function would now not return any error. It would write some error
> message, but you cannot programmatically check that.

I will add code for that.
>=20
> Should we have a matching writehash function?

I would add this function when we need it. Less dead code :-).

Jonatan
>=20
> -Michael
>=20
> > diff --git a/tests/src/initscripts/system/functions/data/1
> > b/tests/src/initscripts/system/functions/data/1
> > new file mode 100644
> > index 000000000..8aca9422b
> > --- /dev/null
> > +++ b/tests/src/initscripts/system/functions/data/1
> > @@ -0,0 +1,17 @@
> > +CONFIG_TYPE=3D3
> > +GREEN_DEV=3Dgreen0
> > +GREEN_MACADDR=3D00:c0:08:8a:a0:47
> > +GREEN_DRIVER=3Dr8175
> > +RED_DEV=3Dred0
> > +RED_MACADDR=3D00:c0:08:8a:a0:56
> > +RED_DRIVER=3Dr8283
> > +BLUE_DEV=3D'blue0 net0'
> > +BLUE_MACADDR=3Dbc:30:7d:58:6b:e3
> > +BLUE_DRIVER=3Drt2800
> > +RED_DHCP_HOSTNAME=3Dipfire
> > +RED_DHCP_FORCE_MTU=3D
> > +RED_ADDRESS=3D0.0.0.0
> > +RED_NETMASK=3D0.0.0.0
> > +RED_TYPE=3DPPPOE
> > +RED_NETADDRESS=3D0.0.0.0
> > +
> > diff --git a/tests/src/initscripts/system/functions/test.sh
> > b/tests/src/initscripts/system/functions/test.sh
> > new file mode 100755
> > index 000000000..ec502e199
> > --- /dev/null
> > +++ b/tests/src/initscripts/system/functions/test.sh
> > @@ -0,0 +1,16 @@
> > +#!/usr/bin/bash
> > +
> > +SCRIPT_PATH=3D"$(dirname "$(readlink -f "$0")")"
> > +
> > +ROOT=3D"$(readlink -f "${SCRIPT_PATH}/../../../../..")"
> > +
> > +. ${ROOT}/tests/lib.sh
> > +
> > +. ${ROOT}/src/initscripts/system/functions
> > +
> > +# read the date in
> > +readhash "CONFIG" "${SCRIPT_PATH}/data/1"
> > +
> > +# test if we read the correct data
> > +test_that_key_in_arry_has_value "CONFIG" "RED_DHCP_HOSTNAME"
> > "ipfire"
> > +test_that_key_in_arry_has_value "CONFIG" "BLUE_MACADDR"
> > "bc:30:7d:58:6b:e3"
> > --=20
> > 2.39.2
> >=20
>=20


--===============2465838815059471938==--