diff mbox series

trace-cmd: Fix lib audit warning message

Message ID CAK7tX=bWn89VxofeyK0hfb4_dQz3tKG2Y2+_47WnaYiWN5UN2w@mail.gmail.com (mailing list archive)
State Superseded
Headers show
Series trace-cmd: Fix lib audit warning message | expand

Commit Message

sameeruddin shaik Jan. 12, 2021, 7:31 a.m. UTC
statically updated the warning message which we will get, when
compiling trace-cmd without the libaudit library.

Following are the the messages which will pop out when compiling

Before:
-------
trace-profile.c:23:3: warning: #warning "lib audit not found, using
raw syscalls " "(install libaudit-devel and try again)"

After:
-------
trace-profile.c:23:3: warning: #warning "lib audit not found, using
raw syscalls " "(install libaudit-devel(for fedora) or
libaudit-dev(for debian/ubuntu) and try again)"

Signed-off-by: Sameeruddin shaik <sameeruddin.shaik8@gmail.com>
---
 tracecmd/trace-profile.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Steven Rostedt Jan. 12, 2021, 5:29 p.m. UTC | #1
On Tue, 12 Jan 2021 13:01:20 +0530
Sameeruddin Shaik <sameeruddin.shaik8@gmail.com> wrote:

Hi Sameer,

> statically updated the warning message which we will get, when
> compiling trace-cmd without the libaudit library.
> 
> Following are the the messages which will pop out when compiling
> 
> Before:
> -------
> trace-profile.c:23:3: warning: #warning "lib audit not found, using
> raw syscalls " "(install libaudit-devel and try again)"
> 
> After:
> -------
> trace-profile.c:23:3: warning: #warning "lib audit not found, using
> raw syscalls " "(install libaudit-devel(for fedora) or
> libaudit-dev(for debian/ubuntu) and try again)"
> 
> Signed-off-by: Sameeruddin shaik <sameeruddin.shaik8@gmail.com>
> ---
>  tracecmd/trace-profile.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/tracecmd/trace-profile.c b/tracecmd/trace-profile.c
> index cfae2a2..4e5d68d 100644
> --- a/tracecmd/trace-profile.c
> +++ b/tracecmd/trace-profile.c
> @@ -21,7 +21,8 @@
> 
>  #ifdef WARN_NO_AUDIT
>  # warning "lib audit not found, using raw syscalls "    \

Your mail client turned tabs into spaces and this doesn't apply properly.
Are you able to fix that?

> -    "(install libaudit-devel and try again)"
> +    "(install libaudit-devel(for fedora) or libaudit-dev(for \
> +debian/ubuntu) and try again)"

Also, the above can stay on one line. Yes, it breaks the 80 char limit, but
that's more of a guideline and not a requirement ;-)


Care to send a v2?

Thanks!

-- Steve

>  #endif
> 
>  #define TASK_STATE_TO_CHAR_STR "RSDTtXZxKWP"
Steven Rostedt Jan. 13, 2021, 2:38 p.m. UTC | #2
On Wed, 13 Jan 2021 21:40:45 +0530
sameeruddin shaik <sameeruddin.shaik8@gmail.com> wrote:

> hi steve,
> 
> please check now.
> 

No, unfortunately its still corrupted. Can you try sending with git?

 https://coderwall.com/p/qcsiew/setting-up-and-using-git-send-email-with-gmail

-- Steve
sameeruddin shaik Jan. 13, 2021, 4:10 p.m. UTC | #3
hi steve,

please check now.

statically updated the warning message which we will get, when
compiling trace-cmd without the libaudit library.

Following are the the messages which will pop out when compiling

Before:
-------
trace-profile.c:23:3: warning: #warning "lib audit not found, using raw 
syscalls " "(install libaudit-devel and try again)"

After:
-------
trace-profile.c:23:3: warning: #warning "lib audit not found, using raw 
syscalls " "(install libaudit-devel(for fedora) or libaudit-dev(for 
debian/ubuntu) and try again)"

Signed-off-by: Sameeruddin shaik <sameeruddin.shaik8@gmail.com>
---
  tracecmd/trace-profile.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tracecmd/trace-profile.c b/tracecmd/trace-profile.c
index cfae2a2..9b5c385 100644
--- a/tracecmd/trace-profile.c
+++ b/tracecmd/trace-profile.c
@@ -21,7 +21,7 @@

  #ifdef WARN_NO_AUDIT
  # warning "lib audit not found, using raw syscalls "    \
-    "(install libaudit-devel and try again)"
+    "(install libaudit-devel(for fedora) or libaudit-dev(for 
debian/ubuntu) and try again)"
  #endif

  #define TASK_STATE_TO_CHAR_STR "RSDTtXZxKWP"
diff mbox series

Patch

diff --git a/tracecmd/trace-profile.c b/tracecmd/trace-profile.c
index cfae2a2..4e5d68d 100644
--- a/tracecmd/trace-profile.c
+++ b/tracecmd/trace-profile.c
@@ -21,7 +21,8 @@ 

 #ifdef WARN_NO_AUDIT
 # warning "lib audit not found, using raw syscalls "    \
-    "(install libaudit-devel and try again)"
+    "(install libaudit-devel(for fedora) or libaudit-dev(for \
+debian/ubuntu) and try again)"
 #endif

 #define TASK_STATE_TO_CHAR_STR "RSDTtXZxKWP"