Re: [PATCH v1] env: setenv add resolve value option

2021-11-04 Thread Simon Glass
Hi Art, On Wed, 3 Nov 2021 at 01:41, Art Nikpal wrote: > > > The high level problem I have with this patch is that we keep going back > > to "we really need to update to a modern hush (or other shell) rather > > than patching new features in to our ancient fork". > > Yes it will be fine ! Does

Re: [PATCH v1] env: setenv add resolve value option

2021-11-03 Thread Art Nikpal
> The high level problem I have with this patch is that we keep going back > to "we really need to update to a modern hush (or other shell) rather > than patching new features in to our ancient fork". Yes it will be fine ! Does anybody have information about these plans ? but in any case my patch

Re: [PATCH v1] env: setenv add resolve value option

2021-11-02 Thread Tom Rini
On Tue, Nov 02, 2021 at 03:19:14PM +0800, Artem Lapkin wrote: > Add possibility setup env variable with additional resolving vars inside > value. > > Usage examples > > => setenv a hello > => setenv b world > => setenv c '${a} ${b}' > => setenv -r d '${c}! ${a}...' > => printenv d > d=hello

Re: [PATCH v1] env: setenv add resolve value option

2021-11-02 Thread Simon Glass
Hi Artem, On Tue, 2 Nov 2021 at 01:19, Artem Lapkin wrote: > > Add possibility setup env variable with additional resolving vars inside > value. > > Usage examples > > => setenv a hello > => setenv b world > => setenv c '${a} ${b}' > => setenv -r d '${c}! ${a}...' > => printenv d > d=hello

[PATCH v1] env: setenv add resolve value option

2021-11-02 Thread Artem Lapkin
Add possibility setup env variable with additional resolving vars inside value. Usage examples => setenv a hello => setenv b world => setenv c '${a} ${b}' => setenv -r d '${c}! ${a}...' => printenv d d=hello world! hello... /* internal usage example */ env_resolve("d", "${c}! ${a}..."); /*