Date: Wed, 27 May 2015 09:09:50 +0200
From: Natanael Copa <ncopa@alpinelinux.org>
To: Isaac Dunham <ibid.ag@gmail.com>
Cc: alpine-devel@lists.alpinelinux.org
Subject: Re: [alpine-devel] [PATCH] newapkbuild: add CMake support.
Message-ID: <20150527090950.43816234@ncopa-desktop.alpinelinux.org>
In-Reply-To: <1427855404-5412-1-git-send-email-ibid.ag@gmail.com>
References: <1427855404-5412-1-git-send-email-ibid.ag@gmail.com>
Precedence: list
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit

On Tue, 31 Mar 2015 19:30:04 -0700
Isaac Dunham <ibid.ag@gmail.com> wrote:

> This is a first try that *might* work for cross-compiling packages with
> an absolute bare minimum of requirements, if you're lucky.
> I can't debug that part further, but the references should help with it.
> ---
>  newapkbuild.in | 32 ++++++++++++++++++++++++++++++++
>  1 file changed, 32 insertions(+)

Applied!



> +		-DCMAKE_CXX_COMPILER="\${CXX:-g++}" \\
> +		-DCMAKE_C_COMPILER="\${CC:-gcc}" \\
> +		-DCMAKE_CXX_FLAGS="\$CXXFLAGS" \\
> +		-DCMAKE_CXX_FLAGS="\$CFLAGS" \\


I am slightly in doubt if setting -DCMAKE_CXX_COMPILER is a good idea.
This made building llvm with cmake break with ccache.

I had to do this:
http://git.alpinelinux.org/cgit/aports/commit/main/llvm?id=f38ff631e3623c3a1f8983ae906c86d6c10258a3

I applied your patch as-is anyway, so we can test it.

Thanks!

-nc


---
Unsubscribe:  alpine-devel+unsubscribe@lists.alpinelinux.org
Help:         alpine-devel+help@lists.alpinelinux.org
---