[Tarantool-patches] [PATCH v1 1/2] Correct error message at pwd.lua

Sergey Bronnikov sergeyb at tarantool.org
Mon Jun 1 11:57:33 MSK 2020


LGTM as obvious

On 08:09 Sun 31 May , Alexander V. Tikhonov wrote:
> Durring investigation of the issues found that error message had
> not user friendly format:
>  local pwgr_errstr = "get%s failed [errno %d]: %s"
> produced:
>  [001] LuajitError: builtin/pwd.lua:101: getgetgrall failed [errno 2]: No such file or directory
> while it had to be:
>  [001] LuajitError: builtin/pwd.lua:101: get getgrall failed [errno 2]: No such file or directory
> Fixed the error message.
> ---
> 
> Github: https://github.com/tarantool/tarantool/tree/avtikhon/gh-4592-enoent-pwd-full-ci
> 
>  src/lua/pwd.lua | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/lua/pwd.lua b/src/lua/pwd.lua
> index d59594b46..1ea79b5eb 100644
> --- a/src/lua/pwd.lua
> +++ b/src/lua/pwd.lua
> @@ -86,7 +86,7 @@ ffi.cdef[[
>  
>  -- {{{ Error handling
>  
> -local pwgr_errstr = "get%s failed [errno %d]: %s"
> +local pwgr_errstr = "get %s failed [errno %d]: %s"
>  
>  -- Use it in the following way: set errno to zero, call a passwd /
>  -- group function, then call this function to check whether there
> -- 
> 2.17.1
> 

-- 
sergeyb@


More information about the Tarantool-patches mailing list