Tarantool development patches archive
 help / color / mirror / Atom feed
* [Tarantool-patches] [PATCH v1 1/2] Correct error message at pwd.lua
@ 2020-05-31  5:09 Alexander V. Tikhonov
  2020-05-31  5:09 ` [Tarantool-patches] [PATCH v1 2/2] test: remove skip condition of app-tap/pwd test Alexander V. Tikhonov
                   ` (2 more replies)
  0 siblings, 3 replies; 9+ messages in thread
From: Alexander V. Tikhonov @ 2020-05-31  5:09 UTC (permalink / raw)
  To: Sergey Bronnikov, Alexander Turenko; +Cc: Oleg Piskunov, tarantool-patches

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

^ permalink raw reply	[flat|nested] 9+ messages in thread
* [Tarantool-patches] [PATCH v1 1/2] Correct error message at pwd.lua
@ 2020-05-31  5:12 Alexander V. Tikhonov
  2020-06-01 10:47 ` Alexander Turenko
  0 siblings, 1 reply; 9+ messages in thread
From: Alexander V. Tikhonov @ 2020-05-31  5:12 UTC (permalink / raw)
  To: Sergey Bronnikov, Alexander Turenko; +Cc: Oleg Piskunov, tarantool-patches

During 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

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2020-06-03 11:37 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-05-31  5:09 [Tarantool-patches] [PATCH v1 1/2] Correct error message at pwd.lua Alexander V. Tikhonov
2020-05-31  5:09 ` [Tarantool-patches] [PATCH v1 2/2] test: remove skip condition of app-tap/pwd test Alexander V. Tikhonov
2020-06-01  8:57   ` Sergey Bronnikov
2020-06-01 10:58   ` Alexander Turenko
2020-06-01  8:57 ` [Tarantool-patches] [PATCH v1 1/2] Correct error message at pwd.lua Sergey Bronnikov
2020-06-03 10:13 ` Kirill Yukhin
2020-05-31  5:12 Alexander V. Tikhonov
2020-06-01 10:47 ` Alexander Turenko
2020-06-03 11:37   ` Alexander V. Tikhonov

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox