X-Original-To: alpine-devel@lists.alpinelinux.org Delivered-To: alpine-devel@mail.alpinelinux.org Received: from ncopa-desktop.alpinelinux.org (3.203.202.84.customer.cdi.no [84.202.203.3]) (using SSLv3 with cipher ECDHE-RSA-AES128-SHA (128/128 bits)) (No client certificate requested) (Authenticated sender: n@tanael.org) by mail.alpinelinux.org (Postfix) with ESMTPSA id E4BF1DC00C0; Fri, 11 Jul 2014 12:15:20 +0000 (UTC) Date: Fri, 11 Jul 2014 14:15:16 +0200 From: Natanael Copa To: Paul Kilar Cc: alpine-devel@lists.alpinelinux.org Subject: Re: [alpine-devel] [PATCH] Patch to compile ipmiutil with musl Message-ID: <20140711141516.35a3e448@ncopa-desktop.alpinelinux.org> In-Reply-To: <1405010940-26034-1-git-send-email-pkilar@gmail.com> References: <1405010940-26034-1-git-send-email-pkilar@gmail.com> X-Mailer: Claws Mail 3.10.1 (GTK+ 2.24.23; x86_64-alpine-linux-musl) X-Mailinglist: alpine-devel Precedence: list List-Id: Alpine Development List-Unsubscribe: List-Post: List-Help: List-Subscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit On Thu, 10 Jul 2014 12:49:00 -0400 Paul Kilar wrote: nice! I get compile error on x86_64 though: ipmiutil.c:77:2: warning: (near initialization for 'subcmds[6].desc') [enabled by default] mv -f .deps/ipmiutil.Tpo .deps/ipmiutil.Po In file included from igetevent.c:135:0: imb_api.h:54:14: error: conflicting types for 'wchar_t' typedef long wchar_t; ^ In file included from /usr/include/stdlib.h:19:0, from igetevent.c:109: /usr/include/bits/alltypes.h:18:13: note: previous declaration of 'wchar_t' was here typedef int wchar_t; ^ > --- > testing/ipmiutil/fixups.patch | 33 +++++++++++++++++++++++++++++++++ > 1 file changed, 33 insertions(+) > create mode 100644 testing/ipmiutil/fixups.patch > > diff --git a/testing/ipmiutil/fixups.patch b/testing/ipmiutil/fixups.patch > new file mode 100644 > index 0000000..dea7ada > --- /dev/null > +++ b/testing/ipmiutil/fixups.patch > @@ -0,0 +1,33 @@ > +--- a/util/imb_api.h > ++++ b/util/imb_api.h > +@@ -49,8 +49,8 @@ > + /* DOS defines wchar_t in stdlib.h */ > + #else > + // defined(LINUX) | defined(SOLARIS) > +-#ifndef _WCHAR_T > +-#define _WCHAR_T > ++#ifndef __DEFINED_wchar_t > ++#define __DEFINED_wchar_t > + typedef long wchar_t; > + #endif > + #endif It looks like wchar_t is defined in standard C (since C90 according http://en.wikipedia.org/wiki/Wide_character#C.2FC.2B.2B) so maybe we can just remove the #else clause that typedef wchar_t? -nc --- Unsubscribe: alpine-devel+unsubscribe@lists.alpinelinux.org Help: alpine-devel+help@lists.alpinelinux.org ---