linux-kernel - Re: [PATCH 1/1] trace-cmd: use pkg-config instead of python-config

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Fri, 1 Nov 2013 17:19:11 -0400
From:	Steven Rostedt <rostedt@...dmis.org>
To:	Romain Naour <romain.naour@...nwide.fr>
Cc:	linux-kernel@...r.kernel.org,
	Johannes Berg <johannes@...solutions.net>,
	Darren Hart <darren@...art.com>
Subject: Re: [PATCH 1/1] trace-cmd: use pkg-config instead of python-config

Please Cc me on trace-cmd patches.

Johannes and Darren,

Since you handle the python bindings in trace-cmd, can you give me an
ack or nack.

Thanks!

-- Steve


On Thu, 31 Oct 2013 18:39:42 +0100
Romain Naour <romain.naour@...nwide.fr> wrote:

> From: Romain <romain.naour@...nwide.fr>
> 
> trace-cmd use python-config to find out which headers
> and libraries should be used to link against the Python libraries.
> By default, python-config returns paths that are inappropriate for
> cross-compilation.
> 
> Add PYTHON_VERS to build trace-cmd with python2 or python3
> 
> Signed-off-by: Romain Naour <romain.naour@...nwide.fr>
> ---
>  Makefile | 8 +++++---
>  1 file changed, 5 insertions(+), 3 deletions(-)
> 
> diff --git a/Makefile b/Makefile
> index 1964949..2f862ec 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -81,8 +81,10 @@ ifndef NO_PYTHON
>  PYTHON		:= ctracecmd.so
>  PYTHON_GUI	:= ctracecmd.so ctracecmdgui.so
>  
> +PYTHON_VERS ?= python
> +
>  # Can build python?
> -ifeq ($(shell sh -c "python-config --includes > /dev/null 2>&1 && echo y"), y)
> +ifeq ($(shell sh -c "pkg-config --cflags $(PYTHON_VERS) > /dev/null 2>&1 && echo y"), y)
>  	PYTHON_PLUGINS := plugin_python.so
>  	BUILD_PYTHON := $(PYTHON) $(PYTHON_PLUGINS)
>  	PYTHON_SO_INSTALL := ctracecmd.install
> @@ -550,8 +552,8 @@ clean:
>  
>  ##### PYTHON STUFF #####
>  
> -PYTHON_INCLUDES = `python-config --includes`
> -PYTHON_LDFLAGS = `python-config --ldflags` \
> +PYTHON_INCLUDES = `pkg-config --cflags $(PYTHON_VERS)`
> +PYTHON_LDFLAGS = `pkg-config --libs $(PYTHON_VERS)` \
>  		$(shell python -c "import distutils.sysconfig; print distutils.sysconfig.get_config_var('LINKFORSHARED')")
>  PYGTK_CFLAGS = `pkg-config --cflags pygtk-2.0`
>  

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/