[LEDE-DEV] [PATCH 1/5] base-files: introduce /lib/functions/board.sh

Felix Fietkau nbd at nbd.name
Sun Apr 30 02:46:00 PDT 2017


On 2017-04-28 22:51, Roman Yeryomin wrote:
> On 28 April 2017 at 20:12, Felix Fietkau <nbd at nbd.name> wrote:
>> On 2017-04-28 14:56, Roman Yeryomin wrote:
>>> Signed-off-by: Roman Yeryomin <roman at advem.lv>
>>> ---
>>>  package/base-files/files/lib/functions/board.sh | 17 +++++++++++++++++
>>>  1 file changed, 17 insertions(+)
>>>  create mode 100644 package/base-files/files/lib/functions/board.sh
>>>
>>> diff --git a/package/base-files/files/lib/functions/board.sh b/package/base-files/files/lib/functions/board.sh
>>> new file mode 100644
>>> index 0000000..28f3bad
>>> --- /dev/null
>>> +++ b/package/base-files/files/lib/functions/board.sh
>>> @@ -0,0 +1,17 @@
>>> +#!/bin/sh
>>> +
>>> +sysinfo()
>>> +{
>>> +     local file=$1
>>> +     [ -e /tmp/sysinfo/$file ] && cat /tmp/sysinfo/$file || echo "generic"
>>> +}
>>> +
>>> +board_name()
>>> +{
>>> +     sysinfo board_name
>>> +}
>>> +
>>> +board_model()
>>> +{
>>> +     sysinfo model
>>> +}
>> Do we really need board_model() at all? I think it's better to leave the
>> board_name() function where it is instead of adding this extra include file.
>>
> 
> If we don't need it why it exists then?
It exists in various files because of random copy&paste madness. It
hasn't been used in years, so it does not make any sense to carry it
forward.

> I think it's better to leave as separate file because it's mostly used
> separately, sourcing platform file, but for functions.sh it's
> negligible change.
The function is so small that a separate file does not make any sense.
If you don't want it in functions.sh, you could try to find a different
one where it fits better. But please don't add a new file for this tiny
piece of code.

- Felix




More information about the Lede-dev mailing list