Re: [PATCH] ShellPkg: Create a homefilesystem environment variable

["Carsey, Jaben" <jaben.carsey@intel.com>]

Reviewed-by: Jaben Carsey <jaben.carsey@intel.com>

> -----Original Message-----
> From: Jim.Dailey@dell.com [mailto:Jim.Dailey@dell.com]
> Sent: Wednesday, October 03, 2018 9:02 AM
> To: edk2-devel@lists.01.org
> Cc: Carsey, Jaben <jaben.carsey@intel.com>; Ni, Ruiyu <ruiyu.ni@intel.com>
> Subject: [edk2] [PATCH] ShellPkg: Create a homefilesystem environment
> variable
> Importance: High
> 
> Create a homefilesystem environment variable whose value is the file
> system on which the executing shell is located. For example: "FS14:".
> 
> This eliminates the need for people to have to try and find the "boot"
> file system in their startup script.  After this change they can simply
> execute %homefilesystem% to set the cwd to the root of the file system
> where the shell is located.
> 
> A future enhancement could be to add "homefilesystem" to the list of
> predefined, read-only variables listed in the EfiShellSetEnv function of
> file ShellProtocol.c
> 
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Jim Dailey <jim_dailey@dell.com>
> ---
>  ShellPkg/Application/Shell/Shell.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/ShellPkg/Application/Shell/Shell.c
> b/ShellPkg/Application/Shell/Shell.c
> index 3f3bcbb4b0..6185b6ac80 100644
> --- a/ShellPkg/Application/Shell/Shell.c
> +++ b/ShellPkg/Application/Shell/Shell.c
> @@ -1169,6 +1169,8 @@ LocateStartupScript (
>        *TempSpot = CHAR_NULL;
>      }
> 
> +    InternalEfiShellSetEnv(L"homefilesystem", StartupScriptPath, TRUE);
> +
>      StartupScriptPath = StrnCatGrow (&StartupScriptPath, &Size,
> ((FILEPATH_DEVICE_PATH *)FileDevicePath)->PathName, 0);
>      PathRemoveLastItem (StartupScriptPath);
>      StartupScriptPath = StrnCatGrow (&StartupScriptPath, &Size,
> mStartupScript, 0);
> --
> 2.17.0.windows.1