[PATCH] shell: Add rtems_shell_dup_current_env to get a default env.
Chris Johns
chrisj at rtems.org
Wed Jun 4 10:55:41 UTC 2014
On 4/06/2014 4:28 pm, Sebastian Huber wrote:
> On 2014-05-29 07:10, Chris Johns wrote:
>> Remove rtems_current_shell_env as this is dangerous because
>> the env can be NULL if used outside of a valid shell with the
>> POSIX key to an env set up.
>>
>> Clean up the usage of rtems_current_shell_env.
>
> The cleanup looks good, but we should keep the rtems_global_shell_env.
> I underestimated its widespread use, so it was an error that we removed
> it. Now it is a constant, so you can use it only for the initialization
> with default values.
>
I would rather this API cleaned up and only a single way for
applications to do this. It is a simple and clean change.
The change I have uses the environment of a caller if the caller has a
specific environment in the key else the base it used, ie inherits the
environment if present.
Chris
More information about the devel
mailing list