diff options
author | Kelvin M. Klann <kmk3.code@protonmail.com> | 2023-06-16 15:42:20 -0300 |
---|---|---|
committer | Kelvin M. Klann <kmk3.code@protonmail.com> | 2023-06-28 04:00:13 -0300 |
commit | b963fe41ae2cd669e5819aded531375ddaebc8b2 (patch) | |
tree | a82794efe2b8b925d24a3fd413663cef1e987926 /test/appimage | |
parent | common.h: use __func__ instead of __FUNCTION__ (diff) | |
download | firejail-b963fe41ae2cd669e5819aded531375ddaebc8b2.tar.gz firejail-b963fe41ae2cd669e5819aded531375ddaebc8b2.tar.zst firejail-b963fe41ae2cd669e5819aded531375ddaebc8b2.zip |
Improve errExit error messages
Changes:
* Move msg to the end of errExit (right before perror(3p))
* Include the full file path (within the repository)
* Add "()" to function name for clarity
Before:
Error malloc: main.c:123 main: Cannot allocate memory
After:
Error src/firejail/main.c:123 main(): malloc: Cannot allocate memory
Note: This clarifies which is the exact file that the error message
comes from, as there are many source files with the same name. For
example:
$ git ls-files 'src/*/main.c' | wc -l
20
Diffstat (limited to 'test/appimage')
-rwxr-xr-x | test/appimage/filename.exp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/appimage/filename.exp b/test/appimage/filename.exp index 9d9127fb5..f2b827bb6 100755 --- a/test/appimage/filename.exp +++ b/test/appimage/filename.exp | |||
@@ -24,7 +24,7 @@ after 100 | |||
24 | send -- "firejail --appimage appimage.sh\r" | 24 | send -- "firejail --appimage appimage.sh\r" |
25 | expect { | 25 | expect { |
26 | timeout {puts "TESTING ERROR 2\n";exit} | 26 | timeout {puts "TESTING ERROR 2\n";exit} |
27 | "Error mounting appimage" | 27 | -re "Error .*mounting appimage" |
28 | } | 28 | } |
29 | after 100 | 29 | after 100 |
30 | 30 | ||