Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1432060 > unrolled thread

[PATCH] [perf]: update makefile message for installing required libs

Started byneerajbadlani@gmail.com
First post2016-06-27 16:10 +0200
Last post2016-06-29 11:50 +0200
Articles 3 — 3 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] [perf]: update makefile message for installing required libs neerajbadlani@gmail.com - 2016-06-27 16:10 +0200
    Re: [PATCH] [perf]: update makefile message for installing required  libs Arnaldo Carvalho de Melo <acme@kernel.org> - 2016-06-27 17:40 +0200
    [tip:perf/core] perf tools: Update makefile message for installing  slang devel package tip-bot for Neeraj Badlani <tipbot@zytor.com> - 2016-06-29 11:50 +0200

#1432060 — [PATCH] [perf]: update makefile message for installing required libs

Fromneerajbadlani@gmail.com
Date2016-06-27 16:10 +0200
Subject[PATCH] [perf]: update makefile message for installing required libs
Message-ID<rOFoC-1gZ-9@gated-at.bofh.it>
From: Neeraj Badlani <neerajbadlani@gmail.com>

In case of missing library (libslang), give hint to install library (libslang2-dev)
Since libslang-dev is not provided by Ubuntu's apt-package

Signed-off-by: Neeraj Badlani <neerajbadlani@gmail.com>
---
 tools/perf/config/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/perf/config/Makefile b/tools/perf/config/Makefile
index 5ad0255..e57d4d7 100644
--- a/tools/perf/config/Makefile
+++ b/tools/perf/config/Makefile
@@ -437,7 +437,7 @@ endif
 
 ifndef NO_SLANG
   ifneq ($(feature-libslang), 1)
-    msg := $(warning slang not found, disables TUI support. Please install slang-devel or libslang-dev);
+    msg := $(warning slang not found, disables TUI support. Please install slang-devel or libslang-dev or libslang2-dev);
     NO_SLANG := 1
   else
     # Fedora has /usr/include/slang/slang.h, but ubuntu /usr/include/slang.h
-- 
2.7.4

[toc] | [next] | [standalone]


#1432129 — Re: [PATCH] [perf]: update makefile message for installing required libs

FromArnaldo Carvalho de Melo <acme@kernel.org>
Date2016-06-27 17:40 +0200
SubjectRe: [PATCH] [perf]: update makefile message for installing required libs
Message-ID<rOGNH-27J-5@gated-at.bofh.it>
In reply to#1432060
Em Mon, Jun 27, 2016 at 06:59:57AM -0700, neerajbadlani@gmail.com escreveu:
> From: Neeraj Badlani <neerajbadlani@gmail.com>
> 
> In case of missing library (libslang), give hint to install library (libslang2-dev)
> Since libslang-dev is not provided by Ubuntu's apt-package
> 
> Signed-off-by: Neeraj Badlani <neerajbadlani@gmail.com>
> ---
>  tools/perf/config/Makefile | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tools/perf/config/Makefile b/tools/perf/config/Makefile
> index 5ad0255..e57d4d7 100644
> --- a/tools/perf/config/Makefile
> +++ b/tools/perf/config/Makefile
> @@ -437,7 +437,7 @@ endif
>  
>  ifndef NO_SLANG
>    ifneq ($(feature-libslang), 1)
> -    msg := $(warning slang not found, disables TUI support. Please install slang-devel or libslang-dev);
> +    msg := $(warning slang not found, disables TUI support. Please install slang-devel or libslang-dev or libslang2-dev);

Too many 'or', I'll apply it as:

+    msg := $(warning slang not found, disables TUI support. Please install slang-devel, libslang-dev or libslang2-dev);

Thanks,

- Arnaldo

>      NO_SLANG := 1
>    else
>      # Fedora has /usr/include/slang/slang.h, but ubuntu /usr/include/slang.h
> -- 
> 2.7.4

[toc] | [prev] | [next] | [standalone]


#1433635 — [tip:perf/core] perf tools: Update makefile message for installing slang devel package

Fromtip-bot for Neeraj Badlani <tipbot@zytor.com>
Date2016-06-29 11:50 +0200
Subject[tip:perf/core] perf tools: Update makefile message for installing slang devel package
Message-ID<rPki6-2xi-39@gated-at.bofh.it>
In reply to#1432060
Commit-ID:  9f776ba11c8be311a5c23777bc9f3b96498cc6cc
Gitweb:     http://git.kernel.org/tip/9f776ba11c8be311a5c23777bc9f3b96498cc6cc
Author:     Neeraj Badlani <neerajbadlani@gmail.com>
AuthorDate: Mon, 27 Jun 2016 06:59:57 -0700
Committer:  Arnaldo Carvalho de Melo <acme@redhat.com>
CommitDate: Mon, 27 Jun 2016 12:44:22 -0300

perf tools: Update makefile message for installing slang devel package

In case of missing library (libslang), give hint to install library
(libslang2-dev), since libslang-dev is not provided by Ubuntu.

Signed-off-by: Neeraj Badlani <neerajbadlani@gmail.com>
Link: http://lkml.kernel.org/r/1467035997-9100-1-git-send-email-neerajbadlani@gmail.com
[ removed excessive 'or' usage ]
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
---
 tools/perf/config/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/perf/config/Makefile b/tools/perf/config/Makefile
index 534c811..bf1a0a0 100644
--- a/tools/perf/config/Makefile
+++ b/tools/perf/config/Makefile
@@ -482,7 +482,7 @@ endif
 
 ifndef NO_SLANG
   ifneq ($(feature-libslang), 1)
-    msg := $(warning slang not found, disables TUI support. Please install slang-devel or libslang-dev);
+    msg := $(warning slang not found, disables TUI support. Please install slang-devel, libslang-dev or libslang2-dev);
     NO_SLANG := 1
   else
     # Fedora has /usr/include/slang/slang.h, but ubuntu /usr/include/slang.h

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web